ZoneMinder
/
zoneminder
mirror of https://github.com/ZoneMinder/zoneminder.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Check for ownership of layout before saving

pull/3524/head
Isaac Connor 2022-07-12 17:32:07 -04:00
parent 56828f4d48
commit c6cc23d73c
1 changed files with 19 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
web/skins/classic/views/js/montage.js
View File

@ -145,23 +145,35 @@ function edit_layout(button) {
});
$j('#SaveLayout').show();
$j('#EditLayout').hide();
const layout = layouts[document.getElementById('zmMontageLayout').value];
if (user.Id && (layout.UserId == 0 || layout.UserId != user.Id)) {
alert('You may not edit this layout, but you can create a new one from it. Please give it a name.');
}
} // end function edit_layout
function save_layout(button) {
mode = VIEWING;
var form = button.form;
var name = form.elements['Name'].value;
const form = button.form;
let name = form.elements['Name'].value;
const layout = layouts[form.zmMontageLayout.value];
if ( !name ) {
if (!name) {
name = form.elements['zmMontageLayout'].options[form.elements['zmMontageLayout'].selectedIndex].text;
}
if ( name=='Freeform' || name=='2 Wide' || name=='3 Wide' || name=='4 Wide' || name=='5 Wide' ) {
alert('You cannot edit the built in layouts. Please give the layout a new name.');
if ( name=='Freeform' || name=='2 Wide' || name=='3 Wide' || name=='4 Wide' || name=='5 Wide' ) {
alert('You cannot edit the built in layouts. Please give the layout a new name.');
return;
} else if (user.Id && (layout.UserId != user.Id) && !canEdit('System') && (name != layout.Name)) {
alert('You cannot edit someone else\'s layouts. Please give the layout a new name.');
return;
}
} else if ( name=='Freeform' || name=='2 Wide' || name=='3 Wide' || name=='4 Wide' || name=='5 Wide' ) {
alert('You cannot use that name. It conflicts with the built in layouts. Please give the layout a new name.');
return;
}
// In fixed positioning, order doesn't matter. In floating positioning, it does.
var Positions = {};
for ( var i = 0, length = monitors.length; i < length; i++ ) {