Tickets :: [#6736] Sieve script fileinto clause should have "INBOX.folder", not only "folder"

6.0.0-beta6

4/10/26

Summary	Sieve script fileinto clause should have "INBOX.folder", not only "folder"
Queue	Ingo
Queue Version	1.2-RC3
Type	Bug
State	Not A Bug
Priority	2. Medium
Owners
Requester	janne.peltonen (at) helsinki (dot) fi
Created	5/21/08 (6533 days ago)
Due
Updated	5/28/08 (6526 days ago)
Assigned	5/22/08 (6532 days ago)
Resolved	5/28/08 (6526 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

05/28/2008 01:39:00 PM	Jan Schneider	State ⇒ Not A Bug

05/28/2008 01:20:07 PM	janne (dot) peltonen (at) helsinki (dot) fi	Comment #7	Reply to this comment
This was apparently a failure in our configuration (default value for IMP spam_folder), not a bug. My apologies for the confusion.

05/23/2008 07:11:28 PM	Chuck Hagenbuch	Comment #6	Reply to this comment
See ~~bug 6527~~ - how are both of these possible open bugs at the same time?

05/23/2008 06:32:37 AM	janne (dot) peltonen (at) helsinki (dot) fi	Comment #5	Reply to this comment
And where are you getting the folder name? If you're using IMP to get the folder list I believe INBOX. should be included automatically. I used IMP's Options -> Server and Folder Information -> Spam folder drop-down menu. And I also used INGO's Spam Filter -> Folder to receive spam menu. Both resulted in a sieve script without any prefix. I was also able to create new spam folders with both (with some trouble, see ~~ticket #6640~~ - IMP spam_folder appears to override INGO spam['folder'] upon login) - and the resulting script had no prefix in the folder name.

05/22/2008 04:41:05 PM	Chuck Hagenbuch	Comment #4	Reply to this comment
The patch was inappropriate for a general-purpose sieve generation app. Folder paths should never be hardcoded.

05/22/2008 04:40:33 PM	Chuck Hagenbuch	Deleted Original Message

05/22/2008 04:40:08 PM	Chuck Hagenbuch	Comment #3 Priority ⇒ 2. Medium State ⇒ Feedback	Reply to this comment
And where are you getting the folder name? If you're using IMP to get the folder list I believe INBOX. should be included automatically.

05/21/2008 12:23:30 PM	janne (dot) peltonen (at) helsinki (dot) fi	Comment #2 New Attachment: ingo_sieve_folder_under_inbox.patch	Reply to this comment
If it is enough to disallow spam folders outside one's INBOX, then this patch might suffice...

05/21/2008 12:02:50 PM	janne (dot) peltonen (at) helsinki (dot) fi	Comment #1 Milestone ⇒ State ⇒ Unconfirmed Patch ⇒ No Queue ⇒ Ingo Summary ⇒ Sieve script fileinto clause should have "INBOX.folder", not only "folder" Type ⇒ Bug Priority ⇒ 3. High	Reply to this comment
Sieve script fileinto clause should have "INBOX.folder", not only "folder". If there is a line in a sieve script such as 'fileinto "junk4";', the script tries to insert the mail into a folder "junk4" in the global namespace, not in "user.$userid.junk4". The line should be 'fileinto "INBOX.junk4";'.