[sam] CVS HEAD problems

[cyko at cykotix.com]

Quoting Chuck Hagenbuch <chuck at horde.org>:

> > [163]    $this->_options[$pref] = array($this->_options[$pref]);
> > [164]  }
> > [165]  var_dump($this->_defaults);
> > [166]    foreach ($this->_defaults[$pref] as $email_address) {
>
> Ah, I think I got it - 163 should have been modifying $this->_defaults, not
> $this->_options. Try with what I just committed.

I also had to change a few lines down from _options to _defaults as well in
order to get it to work entirely after your update.

--- sql.php.orig        Mon May 12 11:20:06 2003
+++ sql.php     Mon May 12 11:15:02 2003
@@ -165,8 +165,8 @@
                     $this->_defaults[$pref] = array($this->_defaults[$pref]);
                 }
                 foreach ($this->_defaults[$pref] as $email_address) {
-                    if (!in_array($email_address, $this->_options[$pref])) {
-                        array_push($this->_options[$pref], $email_address);
+                    if (!in_array($email_address, $this->_defaults[$pref])) {
+                        array_push($this->_defaults[$pref], $email_address);
                     }
                 }
             }

Thanks for the input.  It seems to have gotten rid of all the error messages.

-Patrick Lahni
 cyko at cykotix.com

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