View Issue Details

IDProjectCategoryView StatusLast Update
0002902Composrcorepublic2016-10-28 16:47
ReporterPatrick SchmalstigAssigned ToChris Graham 
SeverityMinor-bug 
Status closedResolutionunable to reproduce 
Product Version 
Fixed in Version 
Summary0002902: If change detection returns 1, but refresh not to happen, notification sound plays repeatedly
DescriptionIf change detection script detects a change on the page you're on (eg. tickets, or in my case the forums), and due to some circumstance, the content is not to be refreshed (such as activating quick reply), the new notification sound keeps playing every time change detection script runs.
Additional InformationChrome Windows 10 lovinity.org Composr CMS 10 RC20
TagsNo tags attached.
Time estimation (hours)
Sponsorship open

Relationships

related to 0002905 resolvedChris Graham Show message on change detection refresh 

Activities

Chris Graham

2016-10-28 16:47

administrator   ~0004498

I can't reproduce or see an issue in the code.

INTERNALISED_AJAX_SCREEN.tpl has changed now. Maybe you need to update your code to reflect that?

INTERNALISED_AJAX_SCREEN.tpl
 setup interval calling detect_change in internalised_ajax_screen.js
internalised_ajax_screen.js
 detects change
 clears interval
 calls callback in INTERNALISED_AJAX_SCREEN.tpl
INTERNALISED_AJAX_SCREEN.tpl
 if active quick reply post, stops
 calls block refresh
 calls _detected_change in internalised_ajax_screen.js
internalised_ajax_screen.js
 makes sound

As you can see, the interval gets cleared (so no loop), and the sound only ever happens if there's no active quick reply (so no sound).

Issue History

Date Modified Username Field Change
2016-10-26 01:58 Patrick Schmalstig New Issue
2016-10-26 01:59 Patrick Schmalstig Additional Information Updated View Revisions
2016-10-27 21:41 Patrick Schmalstig Relationship added related to 0002905
2016-10-28 16:47 Chris Graham Note Added: 0004498
2016-10-28 16:47 Chris Graham Status non-assigned => closed
2016-10-28 16:47 Chris Graham Assigned To => Chris Graham
2016-10-28 16:47 Chris Graham Resolution open => unable to reproduce