Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sieve fileinto extension does not support :create flag #33

Closed
aletheia7 opened this issue Mar 25, 2014 · 4 comments
Closed

Sieve fileinto extension does not support :create flag #33

aletheia7 opened this issue Mar 25, 2014 · 4 comments
Assignees
Milestone

Comments

@aletheia7
Copy link

The Graphical Build window produces a dialog error when "fileinto" is used with ":create". The Source window allows ":create" and the sieve backend allows :create .

The following dialog is shown when ":create" is used:
screenshot from 2014-03-24 19 08 02

Here is the source:

#
#2014-3-20
#
require "fileinto";
require "mailbox";
require "imap4flags";

if allof(
anyof(
    address :localpart "From" "<omitted>" 
    , address :localpart "From" "<omitted>" 
  )
, header :contains "Subject" "Fresh Coffee"
    ) {

    fileinto :create "Trash";
}

Workaround

Use the Source window to save the script.

@aletheia7
Copy link
Author

Version: sieve-0.2.3d.xpi

@thsmi
Copy link
Owner

thsmi commented Apr 16, 2014

The graphical editor currently only supports the commands defined in RFC 5228, no extensions.
Only imap flags are supported as read only.

An Error Message when it stumbles upon an unsupported require statement would be better.

Support for the mailbox extension is unlikely to happen. There are too many more important tasks.
So i'll tag this for a future milestone.

But anyhow feel free to submit a patch. Implementing this addon should not be too hard.

@thsmi thsmi added this to the Future milestone Apr 16, 2014
@AntiSol
Copy link

AntiSol commented Apr 27, 2017

Hi

Is this the reason why I get a similar message when using "fileinto :flags"? Or should I raise that as a separate issue?

Thanks!

@thsmi thsmi self-assigned this Oct 13, 2018
@thsmi
Copy link
Owner

thsmi commented Nov 17, 2018

Fixed in the new grahical editor

@thsmi thsmi closed this as completed Nov 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants