@anonymous-piwik-user opened this issue on June 25th 2009

I just tried to use Zend's session namespaces in a plugin, and failed. As it seems, the usage of session_start and of $_SESSION in Piwik prevents plugins from using this. The attached patch should apply cleanly against 0.4.1, I replaced every occurance of $_SESSION with a new Zend session namespace named after the core component/plugin. I tested the Installer plugin, should work fine. Keywords: session handling zend

@anonymous-piwik-user commented on June 25th 2009
@anonymous-piwik-user commented on June 25th 2009
@robocoder commented on June 25th 2009

Yet you skipped core/Config.php and plugins/Dashboard?

@anonymous-piwik-user commented on June 25th 2009

my bad, forgot to add plugins/Dashboard to the patch. The comment in core/Config.php has been updated too.

@robocoder commented on July 1st 2009

Instead of removing session_start(), shouldn't it be replaced with Zend_Session::start()?

@anonymous-piwik-user commented on July 1st 2009

Look at libs/Zend/Session/Namespace.php, line 116 - it's called by the constructor anyway.

@robocoder commented on July 2nd 2009

Right. And Zend_Session is indirectly used by Zend_Auth...

@robocoder commented on July 3rd 2009

Ok, committing with some minor changes.

@robocoder commented on July 3rd 2009

(In [1276]) Fixes #828 - use Zend_Session / Zend_Session_Namespace

@robocoder commented on July 4th 2009

(In [1286]) refs #828 - fix 'Trying to get property of non-object' error

@robocoder commented on August 1st 2009

PHP 5.2.0 bug causes:

Indirect modification of overloaded property Zend_Session_Namespace::$skipThisStep has no effect
in '/home/hostnser/sd/phpmv/plugins/Installation/Controller.php' at the line 212

Reference: http://framework.zend.com/issues/browse/ZF-1743

@robocoder commented on August 1st 2009

(In [1339]) fixes #885, refs #828 - dispatcher starts/resumes session unless PIWIK_ENABLE_SESSION_START=0

@robocoder commented on August 1st 2009

(In [1348]) fixes #828 - workaround ZF1743 and refactor session handling code

This issue was closed on August 1st 2009
Powered by GitHub Issue Mirror