Opened 3 years ago
Closed 3 years ago
#1486397 closed Bugs (fixed)
[PATCH] rcube_sieve::as_text does not generate correct requires for multiple actions
| Reported by: | FunkyM | Owned by: | |
|---|---|---|---|
| Priority: | 1 - Highest | Milestone: | 0.4-beta |
| Component: | Plugins | Version: | git-master |
| Severity: | major | Keywords: | patch sieve |
| Cc: |
Description
The "rcube_sieve::as_text()" method does not generate the correct sieve "require" field on chained actions.
Example would be a simple rule with first a vacation action followed by a fileinto action.
The current code logic only sets the required extension to the first action found, any following are not considered.
This causes errors on saving from sieve since such a script is invalid.
Patch to fix this is attached.
Attachments (1)
Change History (2)
Changed 3 years ago by FunkyM
comment:1 Changed 3 years ago by alec
- Resolution set to fixed
- Status changed from new to closed
Thx. Fixed in [a2cb3069].
Note: See
TracTickets for help on using
tickets.

[PATCH] Correctly set requires of sieve scripts with multiple actions