[sam] Re: Development status of SAM module? Need a developer?

[John Dalbec]

:sigh: Mutatis mutandis.
John

--- sam/blacklist.php   Wed May 28 22:33:28 2003
+++ blacklist.php       Mon Nov 17 11:06:56 2003
@@ -13,7 +13,7 @@

  require_once SAM_BASE . '/lib/base.php';
  require_once SAM_BASE . '/config/conf.php';
-$sam = new SAM( 'sql', $conf['storage']['params'] );
+$sam = new SAM( $conf['storage']['driver'], $conf['storage']['params'] );


  /* Request retrieval of related user data. */
--- sam/enabledisable.php       Tue Aug 19 11:57:10 2003
+++ enabledisable.php   Mon Nov 17 11:06:52 2003
@@ -12,7 +12,7 @@

  require_once SAM_BASE . '/lib/base.php';
  require_once SAM_BASE . '/config/conf.php';
-$sam = new SAM( 'sql', $conf['storage']['params'] );
+$sam = new SAM( $conf['storage']['driver'], $conf['storage']['params'] );

  /* Request retrieval of related user data. */
  $sam->retrieve();
--- sam/whitelist.php   Wed May 28 22:33:28 2003
+++ whitelist.php       Mon Nov 17 11:07:00 2003
@@ -13,7 +13,7 @@

  require_once SAM_BASE . '/lib/base.php';
  require_once SAM_BASE . '/config/conf.php';
-$sam = new SAM( 'sql', $conf['storage']['params'] );
+$sam = new SAM( $conf['storage']['driver'], $conf['storage']['params'] );

  /* Request retrieval of related user data. */
  $sam->retrieve();


John Dalbec wrote:

> One more patch.  I'm writing my own driver to update SQL _and_ flat 
> files together since I currently have SpamAssassin configured to use 
> flat files and I don't want to break the existing setup.  I'm calling it 
> "sqlftp".  Imagine my surprise when changing the driver name in conf.php 
> didn't work!
> Thanks,
> John
> 
> --- sam/spamassassin.php        Mon Nov 17 09:45:05 2003
> +++ spamassassin.php    Mon Nov 17 10:06:36 2003
> @@ -12,7 +12,7 @@
> 
>  require_once SAM_BASE . '/lib/base.php';
>  require_once SAM_BASE . '/config/conf.php';
> -$sam = new SAM( 'sql', $conf['storage']['params'] );
> +$sam = new SAM( $conf['storage']['driver'], $conf['storage']['params'] );
> 
>  /* Request retrieval of related user data. */
>  $sam->retrieve();
> 
> 
> 
> Damian Gerow wrote:
> 
>> Thus spake John Dalbec (jpdalbec at ysu.edu) [05/11/03 17:10]:
>>
>>> I forgot to mention that my patch is against the Horde 2.x backport, 
>>> in which you hard-code "mail.example.org" as the FTP host in 
>>> enabledisable.php.  I presume the behavior you're describing is that 
>>> of CVS HEAD?
>>
>>
>>
>> Well, it would appear that the version we're actually using is slightly
>> different than the version I have up for distribution...  :)  I figured
>> yours was agains the 2.x backport, as that's what you were replying 
>> to.  I
>> don't follow CVS for sam.
>>
>> I'll apply all three patches later tonight, and start working on the
>> configurable-by-realm tidbit as well.
>>
> 
>