holm/egroupware
1
1
Fork 0
forked from extern/egroupware
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Test that the autorefresh timer is still going before binding, tab may have been closed

Browse Source
This commit is contained in:
Nathan Gray 2014-03-19 20:11:03 +00:00
parent 133c4a317c
commit 69f4d67d38
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
etemplate/js/et2_extension_nextmatch.js
View File

@ -187,6 +187,7 @@ var et2_nextmatch = et2_DOMWidget.extend([et2_IResizeable, et2_IInput],
if(this._autorefresh_timer) if(this._autorefresh_timer)
{ {
window.clearInterval(this._autorefresh_timer); window.clearInterval(this._autorefresh_timer);
this._autorefresh_timer = null;
} }
// Unbind handler used for toggling autorefresh // Unbind handler used for toggling autorefresh
$j(this.getInstanceManager().DOMContainer.parentNode).off('show.et2_nextmatch'); $j(this.getInstanceManager().DOMContainer.parentNode).off('show.et2_nextmatch');
@ -1340,6 +1341,9 @@ var et2_nextmatch = et2_DOMWidget.extend([et2_IResizeable, et2_IInput],
// If the autorefresh time is up, bind once to trigger a refresh // If the autorefresh time is up, bind once to trigger a refresh
// (if needed) when tab is activated again // (if needed) when tab is activated again
this._autorefresh_timer = setTimeout(jQuery.proxy(function() { this._autorefresh_timer = setTimeout(jQuery.proxy(function() {
// Check in case it was stopped / destroyed since
if(!this._autorefresh_timer) return;
$j(this.getInstanceManager().DOMContainer.parentNode).one('show.et2_nextmatch', $j(this.getInstanceManager().DOMContainer.parentNode).one('show.et2_nextmatch',
// Important to use anonymous function instead of just 'this.refresh' because // Important to use anonymous function instead of just 'this.refresh' because
// of the parameters passed // of the parameters passed