#130 Trash Folder Exclusion On Courier

closed-fixed
None
5
2006-08-15
2005-11-11
Michael Card
No

There is an error in folders.php for Courier IMAP. The
scripts says to exclude a folder from the list if its name
is 'inbox.trash' but it doesn't get excluded because the
name on this type of IMAP would be 'INBOX.Trash', but
it is above the strtolower function, so a match isn't
made.

Please let me know if you have any questions.

Michael Card

Discussion

  • Tomas Kuliavas
    Tomas Kuliavas
    2005-11-12

    Logged In: YES
    user_id=225877

    can you provide more information about this issue. Which
    courier version can't store folders in INBOX.Trash?

     
  • Tomas Kuliavas
    Tomas Kuliavas
    2005-11-18

    • assigned_to: nobody --> tokul
     
  • Tomas Kuliavas
    Tomas Kuliavas
    2005-11-18

    Logged In: YES
    user_id=225877

    Please provide explanation why courier-imap can't store
    folders in INBOX.Trash. Without such information I'll remove
    all SquirrelMail INBOX.Trash workarounds specific to courier
    and forward all related bug reports to courier.

    INBOX.Trash folder is not reported as \NoInferiors on
    courier-imap 3.0.8 and 4.0.6. SquirrelMail can store folders
    in INBOX.Trash.

     
  • Michael Card
    Michael Card
    2005-11-19

    Logged In: YES
    user_id=1376538

    There is already a workaround for this, if it is going to
    stay it should work correctly. I believe one of the
    reasons for it would be if you empty the trash, it deletes
    those folders, and then the left folder list gives an
    error because it thinks those folders are still in the
    folder list. You should be able to replicate this problem.

    Let me know your questions thoughts. I would agree that it
    should either be taken out or left in there, but left it
    should be corrected.

    Michael Card

     
  • Tomas Kuliavas
    Tomas Kuliavas
    2005-11-20

    1.5.1cvs patch

     
    Attachments
  • Tomas Kuliavas
    Tomas Kuliavas
    2005-11-20

    Logged In: YES
    user_id=225877

    After folder is deleted, interface displays 'refresh left
    folder list' link. If you don't refresh the listing and try
    using outdated left folder listing, it will cause folder
    errors on any imap server.

    If I remove special handling of Trash folder for courier, I
    am able to store folders in Trash, purge link removes Trash
    subfolders without any error messages. I am using courier
    imap 3.0.8, standard courier preset and set $delete_folder
    variable to false.

    Please provide steps how to reproduce error, if left folder
    listing is up to date.

    You can try deleting folder with name that already exists in
    trashfolder. You get folder deletion error, but issue is not
    specific to courier.

     
  • Tomas Kuliavas
    Tomas Kuliavas
    2005-11-20

    1.4.6cvs patch

     
    Attachments
  • Michael Card
    Michael Card
    2005-11-20

    Logged In: YES
    user_id=1376538

    According to the SM docs for Courier, delete folder should
    be set to true? If this is wrong I will change in on my
    setup then retest.

    Michael Card

     
  • Logged In: YES
    user_id=285765

    What's exactly the status of this?

     
  • Tomas Kuliavas
    Tomas Kuliavas
    2006-07-06

    Logged In: YES
    user_id=225877

    SquirrelMail has code that should add special handling to
    trash folder on Courier. There is an error in folder name
    case and code is not active. But Courier documentation does
    not have any information that explains why INBOX.Trash is
    special.

    If courier is used, SquirrelMail does not allow to store
    folders in Trash. I think user can have trash subfolders in
    courier.

    Issues are low priority. In order to fix these issues some
    changes are needed in conf.pl and src/folders.php

     
  • Logged In: YES
    user_id=285765

    Didn't this get fixed recently?

     
  • Tomas Kuliavas
    Tomas Kuliavas
    2006-08-14

    Logged In: YES
    user_id=225877

    cvs log -r1.147 src/folders.php
    cvs diff -u -r 1.146 -r 1.147 src/folders.php

    I like to break things only in HEAD.

     
  • Tomas Kuliavas
    Tomas Kuliavas
    2006-08-14

    • labels: 102902 -->
     
  • Tomas Kuliavas
    Tomas Kuliavas
    2006-08-15

    • status: open --> closed-fixed