Tickets :: [#8804] Procmail script generation error

6.0.0-beta6

4/9/26

Summary	Procmail script generation error
Queue	Ingo
Queue Version	1.2.3
Type	Bug
State	Resolved
Priority	1. Low
Owners	jan (at) horde (dot) org
Requester	rene.plattner (at) uibk (dot) ac (dot) at
Created	12/28/09 (5946 days ago)
Due
Updated	1/13/10 (5930 days ago)
Assigned	12/30/09 (5944 days ago)
Resolved	12/30/09 (5944 days ago)
Github Issue Link
Github Pull Request
Milestone	1.2.4
Patch	No

01/13/2010 12:25:03 AM	CVS Commit	Comment #7	Reply to this comment
Changes have been made in Git for this ticket: MFB: Remove stray slash (~~Bug #8804~~). http://git.horde.org/diff.php/ingo/docs/CHANGES?rt=horde-git&r1=b4ad4f8663a37182188cbffd960651e089893e8a&r2=7d76106c3960377c978520367292805c234e4fb1 http://git.horde.org/diff.php/ingo/lib/Script/Procmail.php?rt=horde-git&r1=a04d974faf3fd7f46e2036a8917d2fab4091fbdb&r2=7d76106c3960377c978520367292805c234e4fb1

12/30/2009 11:22:07 PM	CVS Commit	Comment #6	Reply to this comment
Changes have been made in Git for this ticket: MFB: Remove stray slash (~~Bug #8804~~). http://git.horde.org/diff.php/ingo/docs/CHANGES?rt=horde-hatchery&r1=b4ad4f8663a37182188cbffd960651e089893e8a&r2=7d76106c3960377c978520367292805c234e4fb1 http://git.horde.org/diff.php/ingo/lib/Script/Procmail.php?rt=horde-hatchery&r1=a04d974faf3fd7f46e2036a8917d2fab4091fbdb&r2=7d76106c3960377c978520367292805c234e4fb1

12/30/2009 11:12:49 PM	Jan Schneider	Comment #5 State ⇒ Resolved Milestone ⇒ 1.2.4 Assigned to Jan Schneider	Reply to this comment
Fixed, thanks!

12/30/2009 11:11:06 PM	CVS Commit	Comment #4	Reply to this comment
Changes have been made in CVS for this ticket: Remove stray slash (~~Bug #8804~~). http://cvs.horde.org/diff.php/ingo/lib/Script/procmail.php?rt=horde&r1=1.46.10.33&r2=1.46.10.34&ty=u

12/30/2009 06:44:54 PM	Matt Selsky	Comment #3	Reply to this comment
No. We want the final result to look like (from "man procmailex": * ^From.*peter There shouldn't be any slash characters.

12/30/2009 05:22:28 PM	Jan Schneider	Comment #2 State ⇒ Feedback	Reply to this comment
Hm, no, it's actually translated to: * /^(To\|Cc\|Bcc\|Resent-to\|From\|Sender\|Reply-to\|Resent-from):.someaddress and this looks like it's supposed to be a regular expression that's missing the closing slash. Can anybody confirm, that this is the correct the rule: /^(To\|Cc\|Bcc\|Resent-to\|From\|Sender\|Reply-to\|Resent-from):.*someaddress/

12/28/2009 10:46:44 AM	rene (dot) plattner (at) uibk (dot) ac (dot) at	Comment #1 Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ Procmail script generation error Queue ⇒ Ingo Milestone ⇒ Patch ⇒ No State ⇒ Unconfirmed	Reply to this comment
Dear Horde Team! I noticed a problem at procmail script generation. 'To,Cc,Bcc,Resent-to' is translated to * \^To,Cc,Bcc,Resent-to: ... should be * ^To,Cc,Bcc,Resent-to: ... I solved this as follows: --- ingo/lib/Script/procmail.php_orig 2009-11-11 08:55:31.000000000 +0100 +++ ingo/lib/Script/procmail.php 2009-11-11 08:59:06.000000000 +0100 @@ -640,7 +640,7 @@ if (strpos($condition['field'], ',') == false) { $string = '^' . $condition['field'] . ':'; } else { - $string .= '/^(' . str_replace(',', '\|', $condition['field']) . '):'; + $string .= '^(' . str_replace(',', '\|', $condition['field']) . '):'; } $prefix = ' '; }