Tickets :: [#14546] Filter rule with something contains a text with comma generates wrong sieve rule

6.0.0-alpha14

7/2/25

Summary	Filter rule with something contains a text with comma generates wrong sieve rule
Queue	Ingo
Queue Version	3.2.12
Type	Bug
State	Resolved
Priority	1. Low
Owners	jan (at) horde (dot) org
Requester	ibon.igartua (at) ehu (dot) eus
Created	12/22/2016 (3114 days ago)
Due
Updated	10/20/2017 (2812 days ago)
Assigned
Resolved	01/13/2017 (3092 days ago)
Github Issue Link
Github Pull Request
Milestone	3.2.14
Patch	No

10/20/2017 08:33:49 PM	Git Commit	Comment #7	Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2): commit 8f4ee501db462da6febedf2914a1344f0d75fe3e Author: Jan Schneider <jan@horde.org> Date: Fri, 13 Jan 2017 14:58:25 +0100 [jan] Don't split sieve body tests on commas (~~Bug #14546~~). M docs/CHANGES M package.xml https://github.com/horde/ingo/commit/8f4ee501db462da6febedf2914a1344f0d75fe3e

10/20/2017 08:33:49 PM	Git Commit	Comment #6	Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2): commit 786e7611a6a792e84cb78cff2bbb01bbed5eefb8 Author: Jan Schneider <jan@horde.org> Date: Fri, 13 Jan 2017 14:57:16 +0100 Don't split body tests on commas (~~Bug #14546~~). M lib/Script/Sieve.php https://github.com/horde/ingo/commit/786e7611a6a792e84cb78cff2bbb01bbed5eefb8

01/13/2017 05:29:31 PM	Git Commit	Comment #5	Reply to this comment
Changes have been made in Git (master): commit d0ab6b4118063e48f9e03a074d88a0592069f1d9 Author: Jan Schneider <jan@horde.org> Date: Fri Jan 13 14:58:25 2017 +0100 [jan] Don't split sieve body tests on commas (~~Bug #14546~~). ingo/package.xml \| 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-) http://github.com/horde/horde/commit/d0ab6b4118063e48f9e03a074d88a0592069f1d9

01/13/2017 01:58:55 PM	Jan Schneider	Assigned to Jan Schneider State ⇒ Resolved Milestone ⇒ 3.2.14

01/13/2017 01:58:46 PM	Git Commit	Comment #4	Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2): commit 5447463b04c38b9b1e81030c9219b07ed61f536a Author: Jan Schneider <jan@horde.org> Date: Fri Jan 13 14:58:25 2017 +0100 [jan] Don't split sieve body tests on commas (~~Bug #14546~~). ingo/docs/CHANGES \| 1 + ingo/package.xml \| 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) http://github.com/horde/horde/commit/5447463b04c38b9b1e81030c9219b07ed61f536a

01/13/2017 01:58:46 PM	Git Commit	Comment #3	Reply to this comment
Changes have been made in Git (FRAMEWORK_5_2): commit d0bf964cf87a328234ba16b41b6d9f7952c47f76 Author: Jan Schneider <jan@horde.org> Date: Fri Jan 13 14:55:34 2017 +0100 Don't split body tests on commas (~~Bug #14546~~). ingo/lib/Script/Sieve.php \| 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) http://github.com/horde/horde/commit/d0bf964cf87a328234ba16b41b6d9f7952c47f76

01/13/2017 01:56:20 PM	Git Commit	Comment #2	Reply to this comment
Changes have been made in Git (master): commit 86dcf10a6b6039a7ddbe13a46ce7ec69038396bf Author: Jan Schneider <jan@horde.org> Date: Fri Jan 13 14:55:34 2017 +0100 Don't split body tests on commas (~~Bug #14546~~). ingo/lib/Script/Sieve.php \| 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) http://github.com/horde/horde/commit/86dcf10a6b6039a7ddbe13a46ce7ec69038396bf

12/22/2016 03:19:51 PM	ibon (dot) igartua (at) ehu (dot) eus	Comment #1 Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ Filter rule with something contains a text with comma generates wrong sieve rule Queue ⇒ Ingo Milestone ⇒ Patch ⇒ No State ⇒ Unconfirmed	Reply to this comment
If we create a filter rule as: Body -> Contains -> first, second then Flag the message as Seen The generated sieve rule results as follows: 185: if body :comparator "i;ascii-casemap" :contains ["first", " second"] { 186: addflag ["\\Seen"]; 187: stop; 188: } It matches any body containing "first" or " second"