diff options
| author |  Alexandre Alapetite <alexandre@alapetite.fr>
| 2020-10-05 19:03:33 +0200 |
|---|---|---|
| committer |  GitHub <noreply@github.com>
| 2020-10-05 19:03:33 +0200 |
| commit | 76523693592a58c7b15c2860ad95133f551d86a5 (patch) | |
| tree | a0fc3605396921dbbb225b2ec04559a014d6f156 /app/Controllers/feedController.php | |
| parent | 2f88b267805c71e9859c41bb50c348b1566cfc06 (diff) | |
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
Diffstat (limited to 'app/Controllers/feedController.php')
| -rwxr-xr-x | app/Controllers/feedController.php | 6 |
1 files changed, 1 insertions, 5 deletions
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 { |