View Issue Details

IDProjectCategoryView StatusLast Update
0003603Composrcore_menuspublic2018-05-21 19:01
ReporterMaster RatAssigned ToChris Graham 
SeverityTrivial-bug 
Status resolvedResolutionfixed 
Product Version 
Fixed in Version 
Summary0003603: When you click on the notification box the link "Mark All Read" seems to not be working...
DescriptionIn the notifications box the link "mark all read" seems or appears not to mark all read even after refresh?
Steps To ReproduceOpen 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....
TagsNo tags attached.
Time estimation (hours)
Sponsorship open

Activities

Adam Edington

2018-05-02 03:21

administrator   ~0005688

Yes I've noticed this too.

Chris Graham

2018-05-21 19:01

administrator   ~0005717

Thanks, will be fixed in next release.

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');

Issue History

Date Modified Username Field Change
2018-05-01 21:35 Master Rat New Issue
2018-05-02 03:21 Adam Edington Note Added: 0005688
2018-05-21 19:01 Chris Graham Note Added: 0005717
2018-05-21 19:01 Chris Graham Status non-assigned => resolved
2018-05-21 19:01 Chris Graham Resolution open => fixed
2018-05-21 19:01 Chris Graham Assigned To => Chris Graham