6.0.0-beta1
7/4/25

[#8804] Procmail script generation error
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/2009 (5667 days ago)
Due
Updated 01/13/2010 (5651 days ago)
Assigned 12/30/2009 (5665 days ago)
Resolved 12/30/2009 (5665 days ago)
Github Issue Link
Github Pull Request
Milestone 1.2.4
Patch No

History
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
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 = ' ';
           }

Saved Queries