#3603 - When you click on the notification box the link "Mark All Read" seems to not be working...
| Identifier | #3603 |
|---|---|
| Issue type | Trivial issue (does not break functionality) |
| Title | When you click on the notification box the link "Mark All Read" seems to not be working... |
| Status | Completed |
| Handling member | Chris Graham |
| Addon | core_menus |
| Description | In the notifications box the link "mark all read" seems or appears not to mark all read even after refresh? |
| Steps to reproduce | Open Notification by clicking icon if there are unread notifications and you just want to mark all as read the link "Mark All Read" has no effect even after a screen refresh.... |
| Funded? | No |
The system will post a comment when this issue is modified (e.g., status changes). To be notified of this, click "Enable comment notifications".


Comments
Patch is
diff --git a/sources/global2.php b/sources/global2.php
index a198b592a..040339a3b 100644
--- a/sources/global2.php
+++ b/sources/global2.php
@@ -157,7 +157,7 @@ function init__global2()
require_code('chat_poller');
chat_poller();
}
- if ((running_script('notifications')) && (@filemtime(get_custom_file_base() . '/data_custom/modules/web_notifications/latest.dat') <= get_param_integer('time_barrier'))) {
+ if ((running_script('notifications')) && (@filemtime(get_custom_file_base() . '/data_custom/modules/web_notifications/latest.dat') <= get_param_integer('time_barrier')) && (get_param_string('type', '') == 'poller')) {
prepare_for_known_ajax_response();
header('Content-Type: application/xml');