[dev] ingo procmail script driver

Ben Chavet ben at chavet.net
Mon Aug 18 02:12:19 PDT 2003


Well, I like the structural changes you made.  They make a lot more sense (not
sure why I didn't think of something like that), but it broke the
functionality...

So, following the structure you came up with, I fixed the functionality &
enhanced it a little bit while I was at it:

*  changed the explode() function call to implode(), since that's the one that
   works...

*  The switch/case statement was incomplete and was leaving out some of the
   rule types, so I made sure all of the rules were being generated properly.

*  I changed the script comments to reflect the rule name, since that would
   probably make more sense to the user looking at the script.

*  Added code to check the driver for support of the stop-script function.
   procmail stops searching as soon as it finds a match, so that feature
   doesn't apply to this driver.

Files changed:
  ingo/lib/Script.php
  ingo/lib/Script/procmail.php
  ingo/templates/rule/footer.inc

diffs attached.

--Ben Chavet


Quoting Michael M Slusarz <slusarz at bigworm.colorado.edu>:

> Quoting ben at chavet.net:
>
> | I extended the procmail script driver in ingo to allow user-defined
> | filters.
> | I'll be the first to admit that it's kind of an ugly patch, but it works
> | (ugly,
> | meaning there is a lot of duplicate code).
>
> Committed - thanks.
>
> | As far as I know, procmail cannot change message status flags (read,
> | answered,
> | etc.), so does anybody know if there is a simple way to disable those
> | options
> | on the filter-editing page?
>
> I've added code that doesn't show the status flags if the underlying
> Ingo_Script:: driver doesn't support it.
>
> michael
>
> ______________________________________________
> Michael Slusarz [slusarz at bigworm.colorado.edu]
> The University of Colorado at Boulder
>
> --
> Horde developers mailing list
> Frequently Asked Questions: http://horde.org/faq/
> To unsubscribe, mail: dev-unsubscribe at lists.horde.org
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: procmail-patch.tar.gz
Type: application/x-gzip
Size: 1623 bytes
Desc: not available
Url : http://lists.horde.org/archives/dev/attachments/20030818/021bf5e6/procmail-patch.tar.bin


More information about the dev mailing list