From 3deeff1713205511ea8b3b3cdf6112caf1b1f950 Mon Sep 17 00:00:00 2001
From: demvlad <demvlad2@yandex.ru>
Date: Sat, 7 Jun 2025 22:01:03 +0300
Subject: [PATCH 1/2] Removed superfluous dataBuffer in graph_spectrum to
 prevent the random spectrum type change bug

---
 src/graph_spectrum.js      | 26 ++++++--------------------
 src/graph_spectrum_calc.js |  8 ++++----
 2 files changed, 10 insertions(+), 24 deletions(-)

diff --git a/src/graph_spectrum.js b/src/graph_spectrum.js
index 2f4b5c75..e5dc3ed9 100644
--- a/src/graph_spectrum.js
+++ b/src/graph_spectrum.js
@@ -100,8 +100,10 @@ export function FlightLogAnalyser(flightLog, canvas, analyserCanvas) {
       });
     };
 
-    let dataLoad = function () {
-      GraphSpectrumCalc.setDataBuffer(dataBuffer);
+    const dataLoad = function (fieldIndex, curve, fieldName) {
+      if (fieldIndex > 0 && curve != null && fieldName != null) {
+        GraphSpectrumCalc.setDataBuffer(fieldIndex, curve, fieldName);
+      }
 
       switch (userSettings.spectrumType) {
         case SPECTRUM_TYPE.FREQ_VS_THROTTLE:
@@ -127,15 +129,10 @@ export function FlightLogAnalyser(flightLog, canvas, analyserCanvas) {
            It is only used to record the current curve positions, collect the data and draw the
            analyser on screen*/
     this.plotSpectrum = function (fieldIndex, curve, fieldName) {
-      // Store the data pointers
-      dataBuffer.fieldIndex = fieldIndex;
-      dataBuffer.curve = curve;
-      dataBuffer.fieldName = fieldName;
-
       // Detect change of selected field.... reload and redraw required.
       if (fftData == null || fieldIndex != fftData.fieldIndex || dataReload) {
         dataReload = false;
-        dataLoad();
+        dataLoad(fieldIndex, curve, fieldName);
         GraphSpectrumPlot.setData(fftData, userSettings.spectrumType);
       }
 
@@ -206,20 +203,9 @@ export function FlightLogAnalyser(flightLog, canvas, analyserCanvas) {
           userSettings.spectrumType = optionSelected;
           saveOneUserSetting("spectrumType", userSettings.spectrumType);
 
-          // Restore dataBuffer if it was corrupted
-          if (!dataBuffer.curve) {
-            dataBuffer.curve = GraphSpectrumCalc._dataBuffer.curve;
-            dataBuffer.fieldName = GraphSpectrumCalc._dataBuffer.fieldName;
-            dataBuffer.fieldIndex = GraphSpectrumCalc._dataBuffer.fieldIndex;
-            console.warn("The dataBuffer was corrupted (set to default zeroes) in FlightLogAnalyser.spectrumTypeElem.change event");
-          }
           // Recalculate the data, for the same curve than now, and draw it
           dataReload = true;
-          that.plotSpectrum(
-            dataBuffer.fieldIndex,
-            dataBuffer.curve,
-            dataBuffer.fieldName,
-          );
+          that.plotSpectrum(-1, null, null); // Update fft data only
         }
 
         // Hide overdraw and zoomY if needed
diff --git a/src/graph_spectrum_calc.js b/src/graph_spectrum_calc.js
index 14e51eec..15e085ef 100644
--- a/src/graph_spectrum_calc.js
+++ b/src/graph_spectrum_calc.js
@@ -84,10 +84,10 @@ GraphSpectrumCalc.setOutTime = function(time) {
   return this._analyserTimeRange.out;
 };
 
-GraphSpectrumCalc.setDataBuffer = function(dataBuffer) {
-  this._dataBuffer.curve = dataBuffer.curve;
-  this._dataBuffer.fieldName = dataBuffer.fieldName;
-  this._dataBuffer.fieldIndex = dataBuffer.fieldIndex;
+GraphSpectrumCalc.setDataBuffer = function(fieldIndex, curve, fieldName) {
+  this._dataBuffer.curve = curve;
+  this._dataBuffer.fieldName = fieldName;
+  this._dataBuffer.fieldIndex = fieldIndex;
   return undefined;
 };
 

From 71e3bbcf80a453574e7a56376df4c81b5235d686 Mon Sep 17 00:00:00 2001
From: demvlad <demvlad2@yandex.ru>
Date: Sat, 7 Jun 2025 22:17:01 +0300
Subject: [PATCH 2/2] Removed useless dataBuffer variable

---
 src/graph_spectrum.js | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/src/graph_spectrum.js b/src/graph_spectrum.js
index e5dc3ed9..843fa284 100644
--- a/src/graph_spectrum.js
+++ b/src/graph_spectrum.js
@@ -19,12 +19,7 @@ export function FlightLogAnalyser(flightLog, canvas, analyserCanvas) {
     analyserZoomY = 1.0 /* 100% */,
     dataReload = false,
     fftData = null,
-    prefs = new PrefStorage(),
-    dataBuffer = {
-      fieldIndex: 0,
-      curve: null,
-      fieldName: null,
-    };
+    prefs = new PrefStorage();
 
   try {
     let isFullscreen = false;