Gregory,
I think upstream has a solution available in this thread:
http://bugs.horde.org/ticket/7611
However you might want to review the suggestion made by the most recent
poster on that thread and see if it makes sense and should be implemented.
I've attached the published patch that does not include the numeric
comparison change.
David.
On Sat, 8 Aug 2009, Gregory Colpart wrote:
Hello,
On Wed, Jul 22, 2009 at 03:09:49PM -0500, David Ehle wrote:
This may have been addressed upstream, but it appears that the Destination
and Source filter on choices in Ingo produce lines similar to this:
##### sourcetest #####
if( \
/^From,Sender,Reply-to,Resent-from:\s*.*u...@host\.some\.domain\.edu/:h \
)
exception {
to "${DEFAULT}/.ttt/"
}
The syntax /^From,Sender,Reply-to,Resent-from: while "legal" appears to be
trying to match that string literally rather than anything in the list.
This means that it basically just doesn't work.
I saw some traffic on the INGO1 mailing list about a similar problem for the
"Destination" filter choice having the same problem. From the look of it
both Destination and Participant are both effected in the Debian/Lenny
release:
Thanks for your report.
I'm not a maildrop guru, then could you provide corrected rules?
Regards,
--
Gregory Colpart <r...@evolix.fr> GnuPG:1024D/C1027A0E
Evolix - Informatique et Logiciels Libres http://www.evolix.fr/
--- maildrop.php.org 2008-10-27 10:07:56.000000000 +0100
+++ maildrop.php 2008-10-30 12:47:37.000000000 +0100
@@ -570,8 +570,13 @@
$string .= '! ';
}
- $string .= '/^' . $condition['field'] . ':\\s*';
-
+ // convert 'field' to PCRE pattern matching
+ if (strpos($condition['field'], ',') == false) {
+ $string .= '/^' . $condition['field'] . ':\\s*';
+ } else {
+ $string .= '/^(' . str_replace(',', '|', $condition['field']) . '):\\s*';
+ }
+
switch ($match) {
case 'regex':
$string .= $condition['value'];
