[sam] CVS HEAD problems

[cyko at cykotix.com]

Quoting Chuck Hagenbuch <chuck at horde.org>:

> Quoting cyko at cykotix.com:
>
> > 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);
> >                      }
> >                  }
> >              }
>
> This doesn't look right to me; what happens without this change?
>
> -chuck
>


I get this output without those changes:

Warning: in_array() [function.in-array]: Wrong datatype for second argument in
/usr/local/www2/mail/webmail/horde/sam/lib/Driver/sql.php on line 168

Warning: array_push() [function.array-push]: First argument should be an array
in /usr/local/www2/mail/webmail/horde/sam/lib/Driver/sql.php on line 169

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


-Patrick




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