[sam] CVS HEAD problems

[cyko at cykotix.com]

> > I'm getting the following messages at the top of my window when using the
> > sam module:
> >
> > Warning: Invalid argument supplied for foreach() in
> > /usr/local/www2/mail/webmail/horde/sam/lib/Driver/sql.php on line 166
>
> You might want to try the bit of cleanup that I just committed, but it
> doesn't really look like it should have an effect. Could you put a
> var_dump($this->_defaults) ($this->defaults without my update) right before
> that foreach(), to see what it's trying to loop on?

I placed the var_dump($this->_defaults) (I did the update) right before the
foreach().  This is the output I receive:

array(13) { ["required_hits"]=> string(3) "5.0" ["subject_tag"]=> string(14)
"*****SPAM*****" ["rewrite_subject"]=> string(1) "1" ["spam_level_stars"]=>
string(1) "1" ["report_header"]=> string(1) "1" ["use_terse_report"]=>
string(1) "1" ["defang_mime"]=> string(1) "0" ["skip_rbl_checks"]=> string(1)
"0" ["blacklist_from"]=> string(17) "*@topsites-us.com" ["score
RCVD_IN_BL_SPAMCOP_NET"]=> string(1) "5" ["use_razor2"]=> string(1) "1" ["score
RCVD_IN_SBL"]=> string(1) "5" ["score RCVD_IN_OSIRUSOFT_COM"]=> string(2) "10"
}
Warning: Invalid argument supplied for foreach() in
/usr/local/www2/mail/webmail/horde/sam/lib/Driver/sql.php on line 166

Warning: Cannot modify header information - headers already sent by (output
started at /usr/local/www2/mail/webmail/horde/sam/lib/Driver/sql.php:165) in
/usr/local/www2/mail/webmail/horde/sam/templates/common-header.inc on line 3

Warning: Cannot modify header information - headers already sent by (output
started at /usr/local/www2/mail/webmail/horde/sam/lib/Driver/sql.php:165) in
/usr/local/www2/mail/webmail/horde/sam/templates/common-header.inc on line 4


The code looks like this:

[163]    $this->_options[$pref] = array($this->_options[$pref]);
[164]  }
[165]  var_dump($this->_defaults);
[166]    foreach ($this->_defaults[$pref] as $email_address) {

Hope this helps in figuring out what is wrong.

Thanks.

-Patrick Lahni
 cyko at cykotix.com

-------------------------------------------------
This mail sent through IMP: http://horde.org/imp/