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
Jan

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 NotificationManager::cancelAllNonPersistent(); and / or do a var_export($view->notifications)

Can reproduce by simply creating a new goal and adding the "Goals overview" widget on the dashboard. (nginx, php 5.6, file sessions)

Commenting NotificationManager::cancelAllNonPersistent(); in Piwik\Plugin\Controller:646 doesn't change anything. var_export($view->notifications) shows an empty array.

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.

if (empty($notifications)) {
   $view->notifications = NotificationManager::getAllNotificationsToDisplay(); //tries to sort the notifications by reference: uasort($notifications, function ($n1, $n2) []
   NotificationManager::cancelAllNonPersistent(); //Calls unset($session->notifications[$id]);
}

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
Maybe it would be better to check with !isset or !is_array at that point?

Just tested that on a local instance. It fixed the exception in the widget

Hi sgiel: what needs to be done?

Checking with !isset instead of empty in the code part linked above. But I don't know if that has any other impact.

After upgrade to 2.11.1 I see this on goals as described but also on Visits Overview widget (with or without graph).

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 core/Notification/Manager.php with this content: https://raw.githubusercontent.com/piwik/piwik/3699f22e0410b39ff4bcb2cafdf2f38e0669b146/core/Notification/Manager.php

@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 ==
If anyone wants to fix it just replace content of the file core/Notification/Manager.php with this content: https://raw.githubusercontent.com/piwik/piwik/7274/core/Notification/Manager.php
== END 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)
I only see "not found".

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/