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

6.0.0-RC7

6/21/26

History
Attachments
Comment
Watch
Download

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/16 (3468 days ago)
Due
Updated	10/20/17 (3166 days ago)
Assigned
Resolved	1/13/17 (3446 days ago)
Github Issue Link
Github Pull Request
Milestone	3.2.14
Patch	No

498	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

498	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

315	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

551	Jan Schneider	Assigned to Jan Schneider State ⇒ Resolved Milestone ⇒ 3.2.14

461	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

461	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

201	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

513	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"