From 76523693592a58c7b15c2860ad95133f551d86a5 Mon Sep 17 00:00:00 2001 From: Alexandre Alapetite Date: Mon, 5 Oct 2020 19:03:33 +0200 Subject: Minz: Attach a notification to a request (#3208) * Minz: Attach a notification to a request Notifications should be attached to a request, not to a global session. Fix https://github.com/FreshRSS/FreshRSS/pull/3096#issuecomment-654891906 Prepare https://github.com/FreshRSS/FreshRSS/pull/3096 * Rename array * Avoid string constants Implement https://github.com/FreshRSS/FreshRSS/pull/3208#issuecomment-703243863 * Improved logic * Simplify storage https://github.com/FreshRSS/FreshRSS/pull/3208#discussion_r499511213 * Fix notification bug in configuration/system --- app/Controllers/feedController.php | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) (limited to 'app/Controllers/feedController.php') diff --git a/app/Controllers/feedController.php b/app/Controllers/feedController.php index 2fc0734db..73faf8dce 100755 --- a/app/Controllers/feedController.php +++ b/app/Controllers/feedController.php @@ -544,11 +544,7 @@ class FreshRSS_feed_Controller extends Minz_ActionController { // Most of the time, ajax request is for only one feed. But since // there are several parallel requests, we should return that there // are several updated feeds. - $notif = array( - 'type' => 'good', - 'content' => _t('feedback.sub.feed.actualizeds') - ); - Minz_Session::_param('notification', $notif); + Minz_Request::setGoodNotification(_t('feedback.sub.feed.actualizeds')); // No layout in ajax request. $this->view->_layout(false); } else { -- cgit v1.2.3