[Tickets #9156] Re: vacation/maildrop single quote in subject

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/9156
------------------------------------------------------------------------------
  Ticket             | 9156
  Updated By         | rsalmon at mbpgroup.com
  Summary            | vacation/maildrop single quote in subject
  Queue              | Ingo
  Version            | Git master
  Type               | Bug
  State              | Feedback
  Priority           | 1. Low
  Milestone          |
  Patch              | 1
  Owners             |
+New Attachment     | ingo.patch
------------------------------------------------------------------------------


rsalmon at mbpgroup.com (2010-08-11 11:04) wrote:

> Ah, understood. And no, they don't work both the same way, that's  
> the problem. Maildrop requires to upload 2 files through VFS which  
> is not supported by the current infrastructure in Ingo. Thus there  
> is some direct VFS access hacked into the maildop script classes,  
> where they don't belong at all.

procmail needs (or may need) 2 files as well. And I think that  
procmail driver doesn't work either since ingo Vfs driver calls  
VFS::singleton.

I nearly find a way to fix maildrop (and may be procmail) drivers, see  
attach patch file.

The main problem I guess is that I couldn't find a way to create a  
Horde_Vfs object, since parameters are not stored like in other apps :  
in ingo, settings are set in backend file and in other apps, settings  
are set in app main config file.

So I modified function Horde/Core/Factory/Vfs.php:getVfs to get an  
extra parameter (see patch file).

The only *problem* is that I don't know how to get yet the actual  
reason inside function ingo/lib/Driver/Vfs.php:setScriptActive

Are you Ok with this approach ?