Extra code in DooSession::isStarted()

[roman]

Revision r696 has this code:

Code: Select all

if (!empty($_SESSION[$this->_namespace]) && $this->_sessionStarted)

The first check is unnecessary because both are set in the start() method. The second condition is sufficient and probably faster than the first one.

[roman]

It looks like this framework doesn't have much to look forward to. The committers are busy, bug reports go unnoticed for weeks... It's a pity.

[donkapone]

I'm also saddened by this situation. I'd be happy to commit some code fixing some bugs, if I had the opportunity...

[roman]

Then let's ask Leng and/or Richard to add us to the list of committers.

[donkapone]

Let's.

[leng]

done. added you both.