Ticket #249 (new defect)

Opened 4 years ago

Last modified 4 years ago

Making a new folder hides inbox (maildir)

Reported by: hanne Owned by: avel
Priority: normal Milestone: 1.9.10
Component: Main Version: 1.9.7
Severity: major Keywords:
Cc:

Description

When making a folder from avelsieve, what ends up in the script is not the expected, which is:

fileinto "test";

but instead

fileinto "INBOX.test";

On a system I admin, the actual inbox is ~username/Maildir/ and there exists no file or folder named INBOX. The folders are stored directly inside ~username/Maildir/.

If there isn't already a folder INBOX, having sieve make one will effectively hide the folder for incoming mail in some imap-clients, rendering all the real inbox, and therefore new mail, invisible. New mail can again be read by deleting the .INBOX.test-folder that is in ~username/Maildir/.

The non-existent INBOX is added by the following lines in include/support.inc.php, see source:main_plugin/trunk/include/support.inc.php@935#90 :

if(empty($subfolder)) { 
    $subfolder = "INBOX"; 
}


There are two obvious ways to fix this:

  1. a config-parameter $disallow_empty_subfolder that is true by default:
if(empty($subfolder) and $disallow_empty_subfolder) {
    $subfolder = "INBOX";
}
  1. using $default_folder_prefix instead of hardcoding INBOX:
if(empty($subfolder) and $disallow_empty_subfolder) {
    $subfolder = $default_folder_prefix;
}

Me, I'm deleting those lines altogether.

Change History

comment:1 Changed 4 years ago by avel

  • Milestone set to 1.9.9

comment:2 Changed 4 years ago by avel

  • Milestone changed from 1.9.9 to 1.9.10
Note: See TracTickets for help on using tickets.