You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When the script is saved, the resulting sieve script is:
<some rules before nested-if rule>
fileinto "Tagged";
This seems to be a similar issue to #5540 where the proposal was to drop nested ifs rules on save. Something went wrong here as the rules are dropped, but rules following the dropped one are damaged and deleted as well.
I'd propose an alternative solution instead:
When a rule cannot be parsed to display it in the visual editor, display it as a text field with source code instead
Please at least warn the user that the editor could not load all the rules, as otherwise the user will save a damaged script.
Blocking the visual editor until the script can be parsed correctly would also be beneficial.
The text was updated successfully, but these errors were encountered:
I don't like the alternative solution. Managesieve plugin is not a sieve editor. It is a simple interface not requiring knowledge about sieve syntax. So, it has some limitations. The same as we don't support "else" we do not support nesting. If you need all these features use the "Edit filter set" action.
I have the following sieve script:
When the script is saved, the resulting sieve script is:
This seems to be a similar issue to #5540 where the proposal was to drop nested ifs rules on save. Something went wrong here as the rules are dropped, but rules following the dropped one are damaged and deleted as well.
I'd propose an alternative solution instead:
When a rule cannot be parsed to display it in the visual editor, display it as a text field with source code instead
Please at least warn the user that the editor could not load all the rules, as otherwise the user will save a damaged script.
Blocking the visual editor until the script can be parsed correctly would also be beneficial.
The text was updated successfully, but these errors were encountered: