6.0.0-git
2019-03-19

[#1105] sieve begins/ends with error
Summary sieve begins/ends with error
Queue Ingo
Queue Version HEAD
Type Bug
State Resolved
Priority 1. Low
Owners Horde Developers (at) , jan (at) horde (dot) org
Requester tmerritt (at) email (dot) arizona (dot) edu
Created 2005-01-07 (5184 days ago)
Due
Updated 2005-02-07 (5153 days ago)
Assigned 2005-01-27 (5164 days ago)
Resolved 2005-02-07 (5153 days ago)
Milestone
Patch No

History
2005-02-07 10:28:47 Jan Schneider Comment #10
State ⇒ Resolved
Reply to this comment
Weird, works now.



Committed to CVS, thanks.
2005-01-27 13:26:33 tmerritt (at) email (dot) arizona (dot) edu Comment #9 Reply to this comment
Could you please give me a specific case where it is not added, I 
can't reproduce this.  Thanks.
2005-01-27 13:08:51 Jan Schneider Comment #8
State ⇒ Feedback
Reply to this comment
I think you convinced me. But the patch is still broken, because the 
wildcard is not added to all value after the split.
2005-01-27 12:52:52 tmerritt (at) email (dot) arizona (dot) edu Comment #7 Reply to this comment
Please see line 1481 of ingo/lib/Script/sieve.php for an example of 
how separating multiple stings with a , is already supported in ingo.
2005-01-19 15:59:05 tmerritt (at) email (dot) arizona (dot) edu Comment #6 Reply to this comment
They don't support it today because they are broken, every other 
string/address test in ingo supports multiple strings/addresses 
separated by ,'s.  I believe if you look further in the existing code 
you'll see that \, is translated into a ,  allowing ,'s in strings.   
We're running ingo here with that same patch applied and it does not 
break anything.
2005-01-19 15:35:35 Jan Schneider Comment #5
State ⇒ Not A Bug
Reply to this comment
These tests don't support multiple addresses. And your patch doesn't 
work anyway, because it makes "equal" test of the addresses if you 
enter them separated by commas.
2005-01-11 18:21:00 Jan Schneider State ⇒ Assigned
Assigned to Horde DevelopersHorde Developers
 
2005-01-07 17:29:05 Jan Schneider Version ⇒ HEAD
 
2005-01-07 17:20:53 tmerritt (at) email (dot) arizona (dot) edu Comment #4 Reply to this comment
The patch is against HEAD, which I would have sworn that I chose when 
I submitted the bug report.
2005-01-07 17:17:27 Jan Schneider Comment #3
State ⇒ Feedback
Reply to this comment
This patch does not apply to the current RELENG_1 code. Make sure you 
have an up-to-date checkout.
2005-01-07 17:14:00 Jan Schneider Assigned to Jan Schneider
State ⇒ Assigned
 
2005-01-07 16:09:28 tmerritt (at) email (dot) arizona (dot) edu Comment #2
New Attachment: ingo.patch Download
Reply to this comment
attached is a patch that fixes this problem.
2005-01-07 16:07:59 tmerritt (at) email (dot) arizona (dot) edu Comment #1
Type ⇒ Bug
State ⇒ Unconfirmed
Priority ⇒ 1. Low
Summary ⇒ sieve begins/ends with error
Queue ⇒ Ingo
Reply to this comment
The sieve script driver does not properly handle ends/begins with if 
the strings/addresses field has multiple values.

Saved Queries