From 7b6d4952bd3bb4814c9b03759ceaa4f57102053e Mon Sep 17 00:00:00 2001
From: Isaac Connor <isaac@zoneminder.com>
Date: Mon, 16 Dec 2024 10:45:30 -0500
Subject: [PATCH] Revert "Don't allow a scale of 0.  Default to 100.  Call
 changeStreamQuality after changeScale so that we always pass the right value.
  Prevents Errors in logs about bogus scale value"

This reverts commit 223dc4b70f4c29f365982666b1ca2ecaf4d09c29.
---
 web/skins/classic/views/js/event.js | 17 ++++++++---------
 1 file changed, 8 insertions(+), 9 deletions(-)

diff --git a/web/skins/classic/views/js/event.js b/web/skins/classic/views/js/event.js
index 1cff4e2f4..d12824580 100644
--- a/web/skins/classic/views/js/event.js
+++ b/web/skins/classic/views/js/event.js
@@ -42,7 +42,7 @@ var coordinateMouse = {
 };
 var leftBtnStatus = {Down: false, UpAfterDown: false};
 var updateScale = false; //Scale needs to be updated
-var currentScale = 100; // Temporarily, because need to put things in order with the "scale" variable = "select" block
+var currentScale = 0; // Temporarily, because need to put things in order with the "scale" variable = "select" block
 
 $j(document).on("keydown", "", function(e) {
   e = e || window.event;
@@ -309,13 +309,13 @@ function changeScale() {
     newSize = scaleToFit(eventData.Width, eventData.Height, eventViewer, bottomEl, $j('#wrapperEventVideo'));
     newWidth = newSize.width;
     newHeight = newSize.height;
-    currentScale = newSize.autoScale ? newSize.autoScale : 100;
+    currentScale = newSize.autoScale;
   } else if (scaleSel == 'fit_to_width') {
     //Fit to screen width
     newSize = scaleToFit(eventData.Width, eventData.Height, eventViewer, false, $j('#wrapperEventVideo'));
     newWidth = newSize.width;
     newHeight = newSize.height;
-    currentScale = newSize.autoScale ? newSize.autoScale : 100;
+    currentScale = newSize.autoScale;
   } else if (scaleSel.indexOf("px") > -1) {
     newSize = scaleToFit(eventData.Width, eventData.Height, eventViewer, false, $j('#wrapperEventVideo')); // Only for calculating the maximum width!
     let w = 0;
@@ -1347,6 +1347,7 @@ function initPage() {
   // Load the event stats
   getStat();
   zmPanZoom.init();
+  changeStreamQuality();
 
   if (getEvtStatsCookie() != 'on') {
     eventStats.toggle(false);
@@ -1355,12 +1356,6 @@ function initPage() {
     onStatsResize(eventData.Width);
     wrapperEventVideo.removeClass('col-sm-12').addClass('col-sm-8');
   }
-  console.log('progress');
-  progressBarNav();
-  console.log('changescale');
-  changeScale();
-  console.log('changeStreamQality');
-  changeStreamQuality();
 
   //FIXME prevent blocking...not sure what is happening or best way to unblock
   if (document.getElementById('videoobj')) {
@@ -1416,12 +1411,16 @@ function initPage() {
       }
     }
   } // end if videojs or mjpeg stream
+  progressBarNav();
+  //if (scale == '0') changeScale();
+  changeScale();
   nearEventsQuery(eventData.Id);
   initialAlarmCues(eventData.Id); //call ajax+renderAlarmCues
   document.querySelectorAll('select[name="rate"]').forEach(function(el) {
     el.onchange = window['changeRate'];
   });
 
+
   // enable or disable buttons based on current selection and user rights
   renameBtn.prop('disabled', !canEdit.Events);
   archiveBtn.prop('disabled', !(!eventData.Archived && canEdit.Events));