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)

0001-Correctly-set-requires-of-sieve-scripts-with-multiple-actions.patch (845 bytes) - added by FunkyM 3 years ago.
[PATCH] Correctly set requires of sieve scripts with multiple actions

Download all attachments as: .zip

Change History (2)

Changed 3 years ago by FunkyM

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

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.