tyler.durden
/
icingaweb2
mirror of https://github.com/Icinga/icingaweb2.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

ui.js: Don't require jQuery objects in `getUniqueContainerId`

This commit is contained in:
Johannes Meyer 2020-07-29 09:55:36 +02:00
parent fbf3a1f757
commit ca74e14812
1 changed files with 14 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
public/js/icinga/ui.js
View File

@ -536,24 +536,29 @@
$('#layout').toggleClass('fullscreen-layout');
},
getUniqueContainerId: function ($cont) {
if (typeof $cont === 'undefined' || !$cont.length) {
getUniqueContainerId: function (container) {
if (typeof container.jquery !== 'undefined') {
if (! container.length) {
return null;
}
container = container[0];
} else if (typeof container === 'undefined') {
return null;
}
var containerId = $cont.data('icingaContainerId');
if (typeof containerId === 'undefined') {
var containerId = container.dataset.icingaContainerId || null;
if (containerId === null) {
/**
* Only generate an id if it's not for col1 or the menu (which are using the non-suffixed window id).
* This is based on the assumption that the server only knows about the menu and first column
* and therefore does not need to protect its ids. (As the menu is most likely part of the sidebar)
*/
if ($cont.attr('id') === 'menu' || $cont.attr('id') === 'col1' || $cont.closest('#col1').length) {
return null;
var col1 = document.getElementById('col1');
if (container.id !== 'menu' && col1 !== null && ! col1.contains(container)) {
containerId = this.icinga.utils.generateId(6); // Random because the content may move
container.dataset.icingaContainerId = containerId;
}
containerId = this.icinga.utils.generateId(6); // Random because the content may move
$cont.data('icingaContainerId', containerId);
}
return containerId;