can't delete monitors - bug in php file *solved*

[acrid]

i couldn't delete monitors from the console view (zm 1.22.0 full)

changed zm_actions.php line 992
from

Code: Select all

if (isset($mid) && canEdit( 'Monitors', $mid ) )

into

Code: Select all

if ( (isset($mid) || isset($mark_mids)) && canEdit( 'Monitors', $mid ) )

works now

[zoneminder]

Yes, this was a late 'tidy up php warnings' fix that ended up breaking it. This whole section needs a bit of a clean up at some point, while I was going through it earlier I found a few others inconsistencies that I'll put into a 1.22.1 release which I'm sure won't be too long coming

[proxorp]

Thanks for the post acrid. This was driving me bonkers. I was thinking I had some permissions set wrong. Your patch did the trick for me.