extern/egroupware_official
1
0
Fork 0
mirror of https://github.com/EGroupware/egroupware.git synced 2025-01-03 04:29:28 +01:00
Code Issues Packages Projects Releases Wiki Activity

only bind queue event-handler for the first queued item, not of every queued item

Browse Source 
jQuery.one() only makes sure each event-handler is called only once, but it does NOT stop installing multiple ones
This commit is contained in:
ralf 2024-03-18 16:14:44 +02:00
parent 9cffe8cbde
commit 05efd753fe
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
api/js/etemplate/et2_extension_nextmatch.ts
View File

@ -1112,8 +1112,11 @@ export class et2_nextmatch extends et2_DOMWidget implements et2_IResizeable, et2
} }
// Bind so we can get the queued data when tab is re-activated // Bind so we can get the queued data when tab is re-activated
let tab = jQuery(this.getInstanceManager().DOMContainer.parentNode) // only do it for this._queued_refreshes === [], to not install multiple event-handlers (jQuery.one() does NOT help here!)
.one('show.et2_nextmatch', this._queue_refresh_callback.bind(this)); if (Array.isArray(this._queued_refreshes) && !this._queued_refreshes.length)
{
jQuery(this.getInstanceManager().DOMContainer.parentNode).one('show.et2_nextmatch', this._queue_refresh_callback.bind(this));
}
// Edit means refresh everything, so no need to keep queueing // Edit means refresh everything, so no need to keep queueing