#117 LSUB command needs % to show \NOSELECT "folders"

closed-fixed
Jason Munro
Folders (53)
9
2002-06-11
2001-08-17
Nick Sayer
No

This is in 1.2.0-rc1.

The colapsable folder view doesn't work very well if
mailboxes don't exist at all of the folder branch
points. That is, if a.b and a.c exist, but not a,
then you don't get to see a +/- colapse point for a
like you should.

The fix is to append % to the LSUB command's 2nd
argument. This will report \NOSELECT nodes. I've
reported this before and it's been fixed in older
versions.

The attached file is the patch to fix it.

Discussion

1 2 > >> (Page 1 of 2)
  • Nick Sayer
    Nick Sayer
    2001-08-17

    Patch against 1.2.0-RC1

     
    Attachments
    p
  • Wouter Teepe
    Wouter Teepe
    2002-01-26

    • status: open --> closed-fixed
     
  • Nick Sayer
    Nick Sayer
    2002-02-23

    • priority: 5 --> 9
    • status: closed-fixed --> open-fixed
     
  • Nick Sayer
    Nick Sayer
    2002-02-23

    Logged In: YES
    user_id=185025

    This is still not fixed in 1.2.5.

     
  • Nick Sayer
    Nick Sayer
    2002-02-23

    • summary: LSUB command needs % to show \NOSELECT f --> LSUB command needs % to show \NOSELECT "folders"
     
  • Nick Sayer
    Nick Sayer
    2002-02-23

    • summary: LSUB command needs % to show \NOSELECT "folders" --> LSUB command needs % to show \NOSELECT "folders"
     
  • Nick Sayer
    Nick Sayer
    2002-02-23

    Logged In: YES
    user_id=185025

    It should be mentioned that it is less likely one would see
    this sort of thing happen if cyrus was not their imap server.

    To see it, create mailboxes foo.bar and foo.baz, but do NOT
    create just plain foo. In the folder view, you will see the
    names bar and baz, but no indication that they live under
    foo. If you add the patch, the folder foo will show up with
    an option to colapse at that point, but foo will not be
    selectable because it's not a mailbox.

     
  • Wouter Teepe
    Wouter Teepe
    2002-03-01

    • assigned_to: nobody --> ullgren
    • summary: LSUB command needs % to show \NOSELECT "folders" --> LSUB command needs % to show \NOSELECT "folders"
     
  • Pontus Ullgren
    Pontus Ullgren
    2002-03-03

    • status: open-fixed --> closed-fixed
     
  • Pontus Ullgren
    Pontus Ullgren
    2002-03-03

    Logged In: YES
    user_id=61176

    Patch applied also added check so that it is impossible to
    delete a \Noselect node since this will generate an error.

    (OT: Deleteing any folder generates error in the current CVS
    (2002-03-03) due to the filter plugin)

     
1 2 > >> (Page 1 of 2)