New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Goals Overview widget: Zend_Session is currently marked as read-only. #7274
Comments
In Piwik\Plugin\Controller, the method "setGeneralVariablesView" (called from Goals\Controller.php) tries to retrieve and reset the notifications in the session. This triggers the read-only session error. |
I can confirm this "Goals Overview widget: Zend_Session is currently marked as read-only." |
+1, having this issue as well. |
Hi guys, thanks for the report. Please comment if you experience the issue. Btw have you configured the session to use DB sessions in the config.ini.php or do you maybe use the default "file" session? |
I've tried both ways, and am seeing the same error in either case. |
Hi there, I have this error too! Kind regards |
Just upgraded. Same issue with Goals |
Same issue here, nothing else seems amiss - just the Goals Overview. Using the default file sessions. |
Same problem after upgrading to 2.11.1 |
This is most likely caused by https://github.com/piwik/piwik/pull/7252/files Can you explain what you do step by step? Just opening Step By Step works fine here |
@peterbo can you maybe check which notifications it wants to show? Just comment the line with |
Please do not directly merge. First I want to understand which notification it wants to trigger in Goals overview. Haven't tested it!
Can reproduce by simply creating a new goal and adding the "Goals overview" widget on the dashboard. (nginx, php 5.6, file sessions) Commenting |
Same here after updating to 2.11.1 |
Same issue after upgrading to 2.11.1 |
+1 |
@tsteur $notifications is NULL, that's why it's running into the if clause. It is not trying to add a message, just to get all pending and then reset the notification stack. So both (or better all methods in /core/Notification/Manager.php) should check if the session is writeable.
|
Same here after updating to 2.11.1 |
I can confirm this issue too. |
+1 |
Same problem here. |
same problem after Update... |
Problem is imho that the NotificationManager always tries to write the session if no notifications are available. See https://github.com/piwik/piwik/blob/master/core/Notification/Manager.php#L134-L136 |
Just tested that on a local instance. It fixed the exception in the widget |
Hi sgiel: what needs to be done? |
Checking with |
thanks @sgiehl that's it! Didn't see it would always try to write something there. I fixed it in #7283 If anyone wants to fix it just replace content of the file |
@tsteur +1, fixed it for me, thanks. |
@tsteur +1, works for me too :) |
Worked for me as well. |
perfect! |
Perfect fix for me too 👍 |
Fixed it for me! |
thanks, it work fine. |
Had same problem below info worked for me as well... == QUOTE == |
thanks for fixing, looking forward to the next release :) |
fixed in #7283 |
Can you guys reupload the fixed version of Manager.php ? (or provide a working link) Thanks ! 👍 |
Sure, thx for letting us know. I updated the link in previous comment: https://raw.githubusercontent.com/piwik/piwik/3699f22e0410b39ff4bcb2cafdf2f38e0669b146/core/Notification/Manager.php |
Btw to get the fix, it's recommended to use the latest beta which includes several fixes: http://piwik.org/faq/how-to-update/faq_159/ |
Session handling refactoring introduced an error in the goal widget: Goals Overview widget: Zend_Session is currently marked as read-only.
Reproduced in 2 different instances after updating to 2.11.1.
The text was updated successfully, but these errors were encountered: