the patch looks ok to me but I'm wondering how this can even work:
$this->setValue( "browser", $this->getSessionValue["phpSniffObject"] );
I mean, shouldn't it be $this->getSessionValue("phpSniffObject") ???
(note the brackets instead of the square brackets)
Oscar
> + // send the headers we've been assigned if any, alognside
> the conten-type header
> + foreach( $this->_headers as $header )
> + header( $header );
> +
> + $this->sendContentType();
> +
> + // 'browser' object that all views will have, so that we
> can gather information
> + // about the client...
> + if ($this->getSessionValue("phpSniffObject") == "") {
> + // as long as we're part of a session, the client
> won't change :)
> + $this->setSessionValue("phpSniffObject", new
> phpSniff());
> + }
> + $this->setValue( "browser",
> $this->getSessionValue["phpSniffObject"] );
> }
> - }
> -?>
> \ No newline at end of file
> + }
> +?>
>
> _______________________________________________
> pLog-svn mailing list
> pLog-svn at devel.plogworld.net
> http://devel.plogworld.net/mailman/listinfo/plog-svn
>