From d9a818525d4f0b04447dfa13e741b107e341347e Mon Sep 17 00:00:00 2001
From: Jakob Ketterl <jakob.ketterl@gmx.de>
Date: Sat, 2 May 2020 01:07:44 +0200
Subject: [PATCH] refactor demodulator classes, part 2

---
 htdocs/lib/BookmarkBar.js      |   3 +-
 htdocs/lib/DemodulatorPanel.js | 106 +++++++++++++++++++++------------
 htdocs/lib/Modes.js            |   2 +-
 3 files changed, 70 insertions(+), 41 deletions(-)

diff --git a/htdocs/lib/BookmarkBar.js b/htdocs/lib/BookmarkBar.js
index 402e3d4..fadf00a 100644
--- a/htdocs/lib/BookmarkBar.js
+++ b/htdocs/lib/BookmarkBar.js
@@ -13,7 +13,8 @@ function BookmarkBar() {
         if (b.modulation) {
             me.getDemodulatorPanel().setMode(b.modulation);
         } else if (b.digital_modulation) {
-            me.getDemodulatorPanel().setDigiMode(b.digital_modulation);
+            // TODO: update bookmarks so they don't contain digital_modulation
+            me.getDemodulatorPanel().setMode(b.digital_modulation);
         }
         $bookmark.addClass('selected');
     });
diff --git a/htdocs/lib/DemodulatorPanel.js b/htdocs/lib/DemodulatorPanel.js
index 09c2cfd..8d37085 100644
--- a/htdocs/lib/DemodulatorPanel.js
+++ b/htdocs/lib/DemodulatorPanel.js
@@ -10,13 +10,22 @@ function DemodulatorPanel(el) {
     });
 
     Modes.registerModePanel(this);
-    el.on('click', '.openwebrx-demodulator-button[data-modulation]', function() {
-        self.setMode($(this).data('modulation'));
+    el.on('click', '.openwebrx-demodulator-button', function() {
+        var modulation = $(this).data('modulation');
+        if (modulation) {
+            self.setMode(modulation);
+        } else {
+            self.disableDigiMode();
+        }
     });
     el.on('change', '.openwebrx-secondary-demod-listbox', function() {
-        self.setDigiMode($(this).val());
+        var value = $(this).val();
+        if (value === 'none') {
+            self.disableDigiMode();
+        } else {
+            self.setMode(value);
+        }
     });
-    // TODO: disable digimodes
 };
 
 DemodulatorPanel.prototype.render = function() {
@@ -52,7 +61,7 @@ DemodulatorPanel.prototype.render = function() {
 
     html.push($(
         '<div class="openwebrx-panel-line openwebrx-panel-flex-line">' +
-            '<div class="openwebrx-button openwebrx-demodulator-button" id="openwebrx-button-dig" onclick="demodulator_digital_replace_last();">DIG</div>' +
+            '<div class="openwebrx-button openwebrx-demodulator-button openwebrx-button-dig">DIG</div>' +
             '<select class="openwebrx-secondary-demod-listbox">' +
                 '<option value="none"></option>' +
                 digiModes.map(function(m){
@@ -65,45 +74,60 @@ DemodulatorPanel.prototype.render = function() {
     this.el.find(".openwebrx-modes").html(html);
 };
 
-DemodulatorPanel.prototype.setMode = function(mode) {
-    var offset_frequency = 0;
-    if (this.demodulator) {
-        if (this.demodulator.get_modulation() === mode) {
-            return;
-        }
-        offset_frequency = this.demodulator.get_offset_frequency();
-        this.demodulator.stop();
-    }
-    this.demodulator = new Demodulator(offset_frequency, mode);
-    var self = this;
-    this.demodulator.on("frequencychange", function(freq) {
-        self.tuneableFrequencyDisplay.setFrequency(center_freq + freq);
-    });
-    this.demodulator.start();
-    this.updateButtons();
-};
-
-DemodulatorPanel.prototype.setDigiMode = function(modulation) {
+DemodulatorPanel.prototype.setMode = function(modulation) {
     var mode = Modes.findByModulation(modulation);
     if (!mode) {
         return;
     }
     if (!mode.isAvailable()) {
-        divlog('Digital mode "' + mode.name + '" not supported. Please check requirements', true);
+        divlog('Modulation "' + mode.name + '" not supported. Please check requirements', true);
         return;
     }
-    this.setMode(mode.underlying[0]);
-    this.getDemodulator().set_secondary_demod(modulation);
-    if (mode.bandpass) {
-        this.getDemodulator().setBandpass(mode.bandpass);
+
+    if (mode.type === 'digimode') {
+        modulation = mode.underlying[0];
     }
+
+    var current_offset_frequency = 0;
+    var current_modulation = false;
+    if (this.demodulator) {
+        current_modulation = this.demodulator.get_modulation();
+        current_offset_frequency = this.demodulator.get_offset_frequency();
+    }
+
+    if (current_modulation !== modulation) {
+        if (this.demodulator) this.demodulator.stop();
+        this.demodulator = new Demodulator(current_offset_frequency, modulation);
+        var self = this;
+        this.demodulator.on("frequencychange", function(freq) {
+            self.tuneableFrequencyDisplay.setFrequency(center_freq + freq);
+        });
+    }
+    if (mode.type === 'digimode') {
+        this.demodulator.set_secondary_demod(mode.modulation);
+    } else {
+        this.demodulator.set_secondary_demod(false);
+    }
+    this.demodulator.start();
+
+    this.updateButtons();
+    this.updatePanels();
+    updateHash();
+};
+
+DemodulatorPanel.prototype.disableDigiMode = function() {
+    var modulation = this.el.find('.openwebrx-button.highlighted[data-modulation]').data('modulation');
+    this.setMode(modulation);
+};
+
+DemodulatorPanel.prototype.updatePanels = function() {
+    var modulation = this.getDemodulator().get_secondary_demod();
     $('#openwebrx-panel-digimodes').attr('data-mode', modulation);
-    toggle_panel("openwebrx-panel-digimodes", true);
+    toggle_panel("openwebrx-panel-digimodes", !!modulation);
     toggle_panel("openwebrx-panel-wsjt-message", ['ft8', 'wspr', 'jt65', 'jt9', 'ft4'].indexOf(modulation) >= 0);
     toggle_panel("openwebrx-panel-js8-message", modulation == "js8");
     toggle_panel("openwebrx-panel-packet-message", modulation === "packet");
     toggle_panel("openwebrx-panel-pocsag-message", modulation === "pocsag");
-    updateHash();
 };
 
 DemodulatorPanel.prototype.getDemodulator = function() {
@@ -126,7 +150,10 @@ DemodulatorPanel.prototype.setInitialParams = function(params) {
 };
 
 DemodulatorPanel.prototype.setHashParams = function(params) {
-    this._apply(params);
+    this._apply({
+        mod: params.secondary_mod || params.mod,
+        offset_frequency: params.offset_frequency
+    });
 };
 
 DemodulatorPanel.prototype.updateButtons = function() {
@@ -134,19 +161,20 @@ DemodulatorPanel.prototype.updateButtons = function() {
     $buttons.removeClass("highlighted").removeClass('disabled');
     var demod = this.getDemodulator()
     if (!demod) return;
-    var mod = demod.get_modulation();
-    this.el.find('[data-modulation=' + mod + ']').addClass("highlighted");
-    var secondary_demod = this.getDemodulator().get_secondary_demod()
+    this.el.find('[data-modulation=' + demod.get_modulation() + ']').addClass("highlighted");
+    var secondary_demod = demod.get_secondary_demod()
     if (secondary_demod) {
-        this.el.find("#openwebrx-button-dig").addClass("highlighted");
-        this.el.find('#openwebrx-secondary-demod-listbox').val(secondary_demod);
+        this.el.find(".openwebrx-button-dig").addClass("highlighted");
+        this.el.find('.openwebrx-secondary-demod-listbox').val(secondary_demod);
         var mode = Modes.findByModulation(secondary_demod);
         if (mode) {
-            var active = mode.underlying.map(function(u){ return 'openwebrx-button-' + u; });
             $buttons.filter(function(){
-                return this.id !== "openwebrx-button-dig" && active.indexOf(this.id) < 0;
+                var mod = $(this).data('modulation');
+                return mod && mode.underlying.indexOf(mod) < 0;
             }).addClass('disabled');
         }
+    } else {
+        this.el.find('.openwebrx-secondary-demod-listbox').val('none');
     }
 }
 
diff --git a/htdocs/lib/Modes.js b/htdocs/lib/Modes.js
index 6e7f928..6c6fa80 100644
--- a/htdocs/lib/Modes.js
+++ b/htdocs/lib/Modes.js
@@ -21,7 +21,7 @@ var Modes = {
         this.panels.push(el);
     },
     updatePanels: function() {
-        var init_complete = this.modes && this.features;
+        var init_complete = this.modes.length && Object.keys(this.features).length;
         this.panels.forEach(function(p) {
             p.render();
             if (init_complete) {