From 43bcd2ee4b46a4c904a389c6e715e5cc22d04abc Mon Sep 17 00:00:00 2001
From: Isaac Connor <isaac@zoneminder.com>
Date: Wed, 22 May 2024 09:14:49 -0400
Subject: [PATCH] Use parseInt to make MaxImageBufferCount and PreEventCount
 comparison work.

---
 web/skins/classic/views/js/monitor.js.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/web/skins/classic/views/js/monitor.js.php b/web/skins/classic/views/js/monitor.js.php
index 4744c49a1..e51bb3b82 100644
--- a/web/skins/classic/views/js/monitor.js.php
+++ b/web/skins/classic/views/js/monitor.js.php
@@ -156,7 +156,7 @@ function validateForm() {
       errors[errors.length] = "<?php echo translate('BadPostEventCount') ?>";
     if ( !form.elements['newMonitor[StreamReplayBuffer]'].value || !(parseInt(form.elements['newMonitor[StreamReplayBuffer]'].value) >= 0 ) )
       errors[errors.length] = "<?php echo translate('BadStreamReplayBuffer') ?>";
-    if (form.elements['newMonitor[PreEventCount]'].value > form.elements['newMonitor[MaxImageBufferCount]'].value)
+    if (parseInt(form.elements['newMonitor[MaxImageBufferCount]'].value) && (parseInt(form.elements['newMonitor[PreEventCount]'].value) > parseInt(form.elements['newMonitor[MaxImageBufferCount]'].value)))
       errors[errors.length] = "<?php echo translate('BadPreEventCountMaxImageBufferCount') ?>";
 
     if ( !form.elements['newMonitor[AlarmFrameCount]'].value || !(parseInt(form.elements['newMonitor[AlarmFrameCount]'].value) > 0 ) )