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

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 (dot) peltonen (at) helsinki (dot) fi
Created	05/21/2008 (109 days ago)
Due
Updated	05/28/2008 (102 days ago)
Assigned	05/22/2008 (108 days ago)
Resolved	05/28/2008 (102 days ago)
Attachments
Milestone
Patch	No

05/28/2008	Jan Schneider	State ⇒ Not A Bug

05/28/2008	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	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	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	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	Chuck Hagenbuch	Deleted Attachment: ingo_sieve_folder_under_inbox.patch

05/22/2008	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	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	janne (dot) peltonen (at) helsinki (dot) fi	Comment #1 Patch ⇒ Milestone ⇒ Queue ⇒ Ingo Summary ⇒ Sieve script fileinto clause should have "INBOX.folder", not only "folder" Type ⇒ Bug Priority ⇒ 3. High State ⇒ Unconfirmed	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";'.