From 15337cac667ed2c31c5e6ed376f16007e002b9c4 Mon Sep 17 00:00:00 2001 From: Mykola Dvornik Date: Sun, 14 Apr 2024 18:58:12 +0200 Subject: [PATCH] Fix bug that prevents settings changes updates via reverse API Most plugins that use reverse API to PATCH settings updates to remote server only do so when `useReverseAPI` is toggled, but not when the relevant settings are being updated. So lets fix the precondition to use the `m_useReverseAPI` flag instead. --- plugins/channelrx/demodm17/m17demod.cpp | 2 +- plugins/channelrx/freqscanner/freqscanner.cpp | 2 +- plugins/channelrx/localsink/localsink.cpp | 2 +- plugins/channelrx/remotetcpsink/remotetcpsink.cpp | 2 +- plugins/feature/antennatools/antennatools.cpp | 2 +- plugins/feature/aprs/aprs.cpp | 2 +- plugins/feature/demodanalyzer/demodanalyzer.cpp | 2 +- plugins/feature/gs232controller/gs232controller.cpp | 2 +- plugins/feature/jogdialcontroller/jogdialcontroller.cpp | 2 +- plugins/feature/map/map.cpp | 2 +- plugins/feature/pertester/pertester.cpp | 2 +- plugins/feature/radiosonde/radiosonde.cpp | 2 +- plugins/feature/rigctlserver/rigctlserver.cpp | 2 +- plugins/feature/satellitetracker/satellitetracker.cpp | 2 +- plugins/feature/sid/sid.cpp | 2 +- plugins/feature/simpleptt/simpleptt.cpp | 2 +- plugins/feature/skymap/skymap.cpp | 2 +- plugins/feature/startracker/startracker.cpp | 2 +- plugins/feature/vorlocalizer/vorlocalizer.cpp | 2 +- plugins/samplemimo/audiocatsiso/audiocatsiso.cpp | 2 +- plugins/samplemimo/bladerf2mimo/bladerf2mimo.cpp | 2 +- plugins/samplemimo/limesdrmimo/limesdrmimo.cpp | 2 +- plugins/samplemimo/plutosdrmimo/plutosdrmimo.cpp | 2 +- plugins/samplemimo/xtrxmimo/xtrxmimo.cpp | 2 +- plugins/samplesink/aaroniartsaoutput/aaroniartsaoutput.cpp | 2 +- plugins/samplesink/audiooutput/audiooutput.cpp | 2 +- plugins/samplesink/bladerf1output/bladerf1output.cpp | 2 +- plugins/samplesink/bladerf2output/bladerf2output.cpp | 2 +- plugins/samplesink/fileoutput/fileoutput.cpp | 2 +- plugins/samplesink/hackrfoutput/hackrfoutput.cpp | 2 +- plugins/samplesink/limesdroutput/limesdroutput.cpp | 2 +- plugins/samplesink/localoutput/localoutput.cpp | 2 +- plugins/samplesink/plutosdroutput/plutosdroutput.cpp | 2 +- plugins/samplesink/remoteoutput/remoteoutput.cpp | 2 +- plugins/samplesink/xtrxoutput/xtrxoutput.cpp | 2 +- plugins/samplesource/aaroniartsainput/aaroniartsainput.cpp | 2 +- plugins/samplesource/airspy/airspyinput.cpp | 2 +- plugins/samplesource/airspyhf/airspyhfinput.cpp | 2 +- .../androidsdrdriverinput/androidsdrdriverinput.cpp | 2 +- plugins/samplesource/audioinput/audioinput.cpp | 2 +- plugins/samplesource/bladerf2input/bladerf2input.cpp | 2 +- plugins/samplesource/fcdpro/fcdproinput.cpp | 2 +- plugins/samplesource/fcdproplus/fcdproplusinput.cpp | 2 +- plugins/samplesource/fileinput/fileinput.cpp | 2 +- plugins/samplesource/hackrfinput/hackrfinput.cpp | 2 +- plugins/samplesource/kiwisdr/kiwisdrinput.cpp | 2 +- plugins/samplesource/limesdrinput/limesdrinput.cpp | 2 +- plugins/samplesource/localinput/localinput.cpp | 2 +- plugins/samplesource/perseus/perseusinput.cpp | 2 +- plugins/samplesource/plutosdrinput/plutosdrinput.cpp | 2 +- plugins/samplesource/remoteinput/remoteinput.cpp | 2 +- plugins/samplesource/remotetcpinput/remotetcpinput.cpp | 2 +- plugins/samplesource/rtlsdr/rtlsdrinput.cpp | 2 +- plugins/samplesource/sdrplay/sdrplayinput.cpp | 2 +- plugins/samplesource/sdrplayv3/sdrplayv3input.cpp | 2 +- plugins/samplesource/sigmffileinput/sigmffileinput.cpp | 2 +- plugins/samplesource/testsource/testsourceinput.cpp | 2 +- plugins/samplesource/xtrxinput/xtrxinput.cpp | 2 +- 58 files changed, 58 insertions(+), 58 deletions(-) diff --git a/plugins/channelrx/demodm17/m17demod.cpp b/plugins/channelrx/demodm17/m17demod.cpp index 58a2df106..2a6990bbb 100644 --- a/plugins/channelrx/demodm17/m17demod.cpp +++ b/plugins/channelrx/demodm17/m17demod.cpp @@ -295,7 +295,7 @@ void M17Demod::applySettings(const M17DemodSettings& settings, const QListgetInputMessageQueue()->push(msg); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/channelrx/freqscanner/freqscanner.cpp b/plugins/channelrx/freqscanner/freqscanner.cpp index c6c7e64fc..54afdd639 100644 --- a/plugins/channelrx/freqscanner/freqscanner.cpp +++ b/plugins/channelrx/freqscanner/freqscanner.cpp @@ -707,7 +707,7 @@ void FreqScanner::applySettings(const FreqScannerSettings& settings, const QStri m_basebandSink->getInputMessageQueue()->push(msg); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/channelrx/localsink/localsink.cpp b/plugins/channelrx/localsink/localsink.cpp index 3f4a794eb..e8676bf62 100644 --- a/plugins/channelrx/localsink/localsink.cpp +++ b/plugins/channelrx/localsink/localsink.cpp @@ -376,7 +376,7 @@ void LocalSink::applySettings(const LocalSinkSettings& settings, const QListgetInputMessageQueue()->push(msg); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/channelrx/remotetcpsink/remotetcpsink.cpp b/plugins/channelrx/remotetcpsink/remotetcpsink.cpp index 501d621c3..999ee8765 100644 --- a/plugins/channelrx/remotetcpsink/remotetcpsink.cpp +++ b/plugins/channelrx/remotetcpsink/remotetcpsink.cpp @@ -239,7 +239,7 @@ void RemoteTCPSink::applySettings(const RemoteTCPSinkSettings& settings, const Q MsgConfigureRemoteTCPSink *msg = MsgConfigureRemoteTCPSink::create(settings, settingsKeys, force, remoteChange); m_basebandSink->getInputMessageQueue()->push(msg); - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/feature/antennatools/antennatools.cpp b/plugins/feature/antennatools/antennatools.cpp index f897e19d2..6ff591953 100644 --- a/plugins/feature/antennatools/antennatools.cpp +++ b/plugins/feature/antennatools/antennatools.cpp @@ -101,7 +101,7 @@ void AntennaTools::applySettings(const AntennaToolsSettings& settings, const QLi { qDebug() << "AntennaTools::applySettings:" << settings.getDebugString(settingsKeys, force) << " force: " << force; - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/feature/aprs/aprs.cpp b/plugins/feature/aprs/aprs.cpp index ecba7f919..3b889c983 100644 --- a/plugins/feature/aprs/aprs.cpp +++ b/plugins/feature/aprs/aprs.cpp @@ -214,7 +214,7 @@ void APRS::applySettings(const APRSSettings& settings, const QList& set m_worker->getInputMessageQueue()->push(msg); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/feature/demodanalyzer/demodanalyzer.cpp b/plugins/feature/demodanalyzer/demodanalyzer.cpp index 9d9b7cc9c..58f0bb114 100644 --- a/plugins/feature/demodanalyzer/demodanalyzer.cpp +++ b/plugins/feature/demodanalyzer/demodanalyzer.cpp @@ -290,7 +290,7 @@ void DemodAnalyzer::applySettings(const DemodAnalyzerSettings& settings, const Q } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/feature/gs232controller/gs232controller.cpp b/plugins/feature/gs232controller/gs232controller.cpp index c4d47cad5..27e02c5b6 100644 --- a/plugins/feature/gs232controller/gs232controller.cpp +++ b/plugins/feature/gs232controller/gs232controller.cpp @@ -296,7 +296,7 @@ void GS232Controller::applySettings(const GS232ControllerSettings& settings, con m_worker->getInputMessageQueue()->push(msg); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/feature/jogdialcontroller/jogdialcontroller.cpp b/plugins/feature/jogdialcontroller/jogdialcontroller.cpp index 80dbfd207..ac4511efb 100644 --- a/plugins/feature/jogdialcontroller/jogdialcontroller.cpp +++ b/plugins/feature/jogdialcontroller/jogdialcontroller.cpp @@ -174,7 +174,7 @@ void JogdialController::applySettings(const JogdialControllerSettings& settings, { qDebug() << "JogdialController::applySettings:" << settings.getDebugString(settingsKeys, force) << " force: " << force; - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/feature/map/map.cpp b/plugins/feature/map/map.cpp index 7d09e80c8..5c4a1921f 100644 --- a/plugins/feature/map/map.cpp +++ b/plugins/feature/map/map.cpp @@ -145,7 +145,7 @@ void Map::applySettings(const MapSettings& settings, const QList& setti { qDebug() << "Map::applySettings:" << settings.getDebugString(settingsKeys, force) << " force: " << force; - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/feature/pertester/pertester.cpp b/plugins/feature/pertester/pertester.cpp index 2e0333286..6b4b422a6 100644 --- a/plugins/feature/pertester/pertester.cpp +++ b/plugins/feature/pertester/pertester.cpp @@ -196,7 +196,7 @@ void PERTester::applySettings(const PERTesterSettings& settings, const QListgetInputMessageQueue()->push(msg); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/feature/radiosonde/radiosonde.cpp b/plugins/feature/radiosonde/radiosonde.cpp index 7d5f9e06c..0db07161f 100644 --- a/plugins/feature/radiosonde/radiosonde.cpp +++ b/plugins/feature/radiosonde/radiosonde.cpp @@ -139,7 +139,7 @@ void Radiosonde::applySettings(const RadiosondeSettings& settings, const QListgetInputMessageQueue()->push(msg); - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/feature/satellitetracker/satellitetracker.cpp b/plugins/feature/satellitetracker/satellitetracker.cpp index 9aae7946e..e64b6a848 100644 --- a/plugins/feature/satellitetracker/satellitetracker.cpp +++ b/plugins/feature/satellitetracker/satellitetracker.cpp @@ -216,7 +216,7 @@ void SatelliteTracker::applySettings(const SatelliteTrackerSettings& settings, c m_worker->getInputMessageQueue()->push(msg); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/feature/sid/sid.cpp b/plugins/feature/sid/sid.cpp index 80d3f3456..394de6387 100644 --- a/plugins/feature/sid/sid.cpp +++ b/plugins/feature/sid/sid.cpp @@ -166,7 +166,7 @@ void SIDMain::applySettings(const SIDSettings& settings, const QList& s m_worker->getInputMessageQueue()->push(msg); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/feature/simpleptt/simpleptt.cpp b/plugins/feature/simpleptt/simpleptt.cpp index 19a67f7d5..da472263f 100644 --- a/plugins/feature/simpleptt/simpleptt.cpp +++ b/plugins/feature/simpleptt/simpleptt.cpp @@ -214,7 +214,7 @@ void SimplePTT::applySettings(const SimplePTTSettings& settings, const QListgetInputMessageQueue()->push(msg); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/feature/skymap/skymap.cpp b/plugins/feature/skymap/skymap.cpp index 4c5e8a7b9..3c6771d56 100644 --- a/plugins/feature/skymap/skymap.cpp +++ b/plugins/feature/skymap/skymap.cpp @@ -116,7 +116,7 @@ void SkyMap::applySettings(const SkyMapSettings& settings, const QList& { qDebug() << "SkyMap::applySettings:" << settings.getDebugString(settingsKeys, force) << " force: " << force; - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/feature/startracker/startracker.cpp b/plugins/feature/startracker/startracker.cpp index 6f6a22104..c656cb2b0 100644 --- a/plugins/feature/startracker/startracker.cpp +++ b/plugins/feature/startracker/startracker.cpp @@ -253,7 +253,7 @@ void StarTracker::applySettings(const StarTrackerSettings& settings, const QList m_worker->getInputMessageQueue()->push(msg); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/feature/vorlocalizer/vorlocalizer.cpp b/plugins/feature/vorlocalizer/vorlocalizer.cpp index e48c0f5c0..ab9512ae1 100644 --- a/plugins/feature/vorlocalizer/vorlocalizer.cpp +++ b/plugins/feature/vorlocalizer/vorlocalizer.cpp @@ -347,7 +347,7 @@ void VORLocalizer::applySettings(const VORLocalizerSettings& settings, const QLi m_worker->getInputMessageQueue()->push(msg); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplemimo/audiocatsiso/audiocatsiso.cpp b/plugins/samplemimo/audiocatsiso/audiocatsiso.cpp index 208e34ec9..0bae8601d 100644 --- a/plugins/samplemimo/audiocatsiso/audiocatsiso.cpp +++ b/plugins/samplemimo/audiocatsiso/audiocatsiso.cpp @@ -557,7 +557,7 @@ void AudioCATSISO::applySettings(const AudioCATSISOSettings& settings, const QLi forwardTxChange = true; } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplemimo/bladerf2mimo/bladerf2mimo.cpp b/plugins/samplemimo/bladerf2mimo/bladerf2mimo.cpp index 65162c8ca..80415810a 100644 --- a/plugins/samplemimo/bladerf2mimo/bladerf2mimo.cpp +++ b/plugins/samplemimo/bladerf2mimo/bladerf2mimo.cpp @@ -727,7 +727,7 @@ bool BladeRF2MIMO::applySettings(const BladeRF2MIMOSettings& settings, const QLi // Reverse API settings - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplemimo/limesdrmimo/limesdrmimo.cpp b/plugins/samplemimo/limesdrmimo/limesdrmimo.cpp index e4b98eb30..75a4e043e 100644 --- a/plugins/samplemimo/limesdrmimo/limesdrmimo.cpp +++ b/plugins/samplemimo/limesdrmimo/limesdrmimo.cpp @@ -1012,7 +1012,7 @@ bool LimeSDRMIMO::applySettings(const LimeSDRMIMOSettings& settings, const QList } } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplemimo/plutosdrmimo/plutosdrmimo.cpp b/plugins/samplemimo/plutosdrmimo/plutosdrmimo.cpp index 4cfffc4e6..b79009dce 100644 --- a/plugins/samplemimo/plutosdrmimo/plutosdrmimo.cpp +++ b/plugins/samplemimo/plutosdrmimo/plutosdrmimo.cpp @@ -713,7 +713,7 @@ bool PlutoSDRMIMO::applySettings(const PlutoSDRMIMOSettings& settings, const QLi plutoBox->set_params(DevicePlutoSDRBox::DEVICE_PHY, params); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplemimo/xtrxmimo/xtrxmimo.cpp b/plugins/samplemimo/xtrxmimo/xtrxmimo.cpp index 3a4e90166..937a62c9c 100644 --- a/plugins/samplemimo/xtrxmimo/xtrxmimo.cpp +++ b/plugins/samplemimo/xtrxmimo/xtrxmimo.cpp @@ -770,7 +770,7 @@ bool XTRXMIMO::applySettings(const XTRXMIMOSettings& settings, const QListstartWork(); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplesink/bladerf2output/bladerf2output.cpp b/plugins/samplesink/bladerf2output/bladerf2output.cpp index d267dacb6..6be8d6183 100644 --- a/plugins/samplesink/bladerf2output/bladerf2output.cpp +++ b/plugins/samplesink/bladerf2output/bladerf2output.cpp @@ -903,7 +903,7 @@ bool BladeRF2Output::applySettings(const BladeRF2OutputSettings& settings, const } } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplesink/fileoutput/fileoutput.cpp b/plugins/samplesink/fileoutput/fileoutput.cpp index 8181ee7a9..872a1c754 100644 --- a/plugins/samplesink/fileoutput/fileoutput.cpp +++ b/plugins/samplesink/fileoutput/fileoutput.cpp @@ -316,7 +316,7 @@ void FileOutput::applySettings(const FileOutputSettings& settings, const QListstartWork(); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplesink/limesdroutput/limesdroutput.cpp b/plugins/samplesink/limesdroutput/limesdroutput.cpp index afcea2d4e..be4d95fa1 100644 --- a/plugins/samplesink/limesdroutput/limesdroutput.cpp +++ b/plugins/samplesink/limesdroutput/limesdroutput.cpp @@ -1006,7 +1006,7 @@ bool LimeSDROutput::applySettings(const LimeSDROutputSettings& settings, const Q } } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplesink/localoutput/localoutput.cpp b/plugins/samplesink/localoutput/localoutput.cpp index 986ee27ad..eb67dd8a8 100644 --- a/plugins/samplesink/localoutput/localoutput.cpp +++ b/plugins/samplesink/localoutput/localoutput.cpp @@ -214,7 +214,7 @@ void LocalOutput::applySettings(const LocalOutputSettings& settings, const QList QString remoteAddress; QList reverseAPIKeys; - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplesink/plutosdroutput/plutosdroutput.cpp b/plugins/samplesink/plutosdroutput/plutosdroutput.cpp index f60e051fa..56b62d83f 100644 --- a/plugins/samplesink/plutosdroutput/plutosdroutput.cpp +++ b/plugins/samplesink/plutosdroutput/plutosdroutput.cpp @@ -538,7 +538,7 @@ bool PlutoSDROutput::applySettings(const PlutoSDROutputSettings& settings, const plutoBox->set_params(DevicePlutoSDRBox::DEVICE_PHY, params); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplesink/remoteoutput/remoteoutput.cpp b/plugins/samplesink/remoteoutput/remoteoutput.cpp index 23879e404..2abd3775a 100644 --- a/plugins/samplesink/remoteoutput/remoteoutput.cpp +++ b/plugins/samplesink/remoteoutput/remoteoutput.cpp @@ -302,7 +302,7 @@ void RemoteOutput::applySettings(const RemoteOutputSettings& settings, const QLi mutexLocker.unlock(); - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplesink/xtrxoutput/xtrxoutput.cpp b/plugins/samplesink/xtrxoutput/xtrxoutput.cpp index 942c72dd1..d9aac937e 100644 --- a/plugins/samplesink/xtrxoutput/xtrxoutput.cpp +++ b/plugins/samplesink/xtrxoutput/xtrxoutput.cpp @@ -906,7 +906,7 @@ bool XTRXOutput::applySettings(const XTRXOutputSettings& settings, const QListgetDeviceEngineInputMessageQueue()->push(notif); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplesource/androidsdrdriverinput/androidsdrdriverinput.cpp b/plugins/samplesource/androidsdrdriverinput/androidsdrdriverinput.cpp index 70baae819..350e4ac54 100644 --- a/plugins/samplesource/androidsdrdriverinput/androidsdrdriverinput.cpp +++ b/plugins/samplesource/androidsdrdriverinput/androidsdrdriverinput.cpp @@ -248,7 +248,7 @@ void AndroidSDRDriverInput::applySettings(const AndroidSDRDriverInputSettings& s mutexLocker.unlock(); - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplesource/audioinput/audioinput.cpp b/plugins/samplesource/audioinput/audioinput.cpp index d0a246ee0..bc7519986 100644 --- a/plugins/samplesource/audioinput/audioinput.cpp +++ b/plugins/samplesource/audioinput/audioinput.cpp @@ -310,7 +310,7 @@ void AudioInput::applySettings(const AudioInputSettings& settings, QListconfigureCorrections(settings.m_dcBlock, settings.m_iqCorrection); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplesource/fcdproplus/fcdproplusinput.cpp b/plugins/samplesource/fcdproplus/fcdproplusinput.cpp index ee3bce44a..f6b9a69f5 100644 --- a/plugins/samplesource/fcdproplus/fcdproplusinput.cpp +++ b/plugins/samplesource/fcdproplus/fcdproplusinput.cpp @@ -427,7 +427,7 @@ void FCDProPlusInput::applySettings(const FCDProPlusSettings& settings, const QL m_deviceAPI->configureCorrections(settings.m_dcBlock, settings.m_iqImbalance); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplesource/fileinput/fileinput.cpp b/plugins/samplesource/fileinput/fileinput.cpp index 9f932868b..3382ca56e 100644 --- a/plugins/samplesource/fileinput/fileinput.cpp +++ b/plugins/samplesource/fileinput/fileinput.cpp @@ -554,7 +554,7 @@ bool FileInput::applySettings(const FileInputSettings& settings, const QListgetDeviceEngineInputMessageQueue()->push(notif); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplesource/kiwisdr/kiwisdrinput.cpp b/plugins/samplesource/kiwisdr/kiwisdrinput.cpp index b83ad2cf9..8eab12d09 100644 --- a/plugins/samplesource/kiwisdr/kiwisdrinput.cpp +++ b/plugins/samplesource/kiwisdr/kiwisdrinput.cpp @@ -314,7 +314,7 @@ bool KiwiSDRInput::applySettings(const KiwiSDRSettings& settings, const QListgetDeviceEngineInputMessageQueue()->push(notif); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplesource/limesdrinput/limesdrinput.cpp b/plugins/samplesource/limesdrinput/limesdrinput.cpp index 64ff741cb..d15187270 100644 --- a/plugins/samplesource/limesdrinput/limesdrinput.cpp +++ b/plugins/samplesource/limesdrinput/limesdrinput.cpp @@ -1162,7 +1162,7 @@ bool LimeSDRInput::applySettings(const LimeSDRInputSettings& settings, const QLi } } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplesource/localinput/localinput.cpp b/plugins/samplesource/localinput/localinput.cpp index 31b3f8461..33368c390 100644 --- a/plugins/samplesource/localinput/localinput.cpp +++ b/plugins/samplesource/localinput/localinput.cpp @@ -221,7 +221,7 @@ void LocalInput::applySettings(const LocalInputSettings& settings, const QListgetDeviceEngineInputMessageQueue()->push(notif); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplesource/plutosdrinput/plutosdrinput.cpp b/plugins/samplesource/plutosdrinput/plutosdrinput.cpp index 723b2c8c8..ac2f6051b 100644 --- a/plugins/samplesource/plutosdrinput/plutosdrinput.cpp +++ b/plugins/samplesource/plutosdrinput/plutosdrinput.cpp @@ -591,7 +591,7 @@ bool PlutoSDRInput::applySettings(const PlutoSDRInputSettings& settings, const Q plutoBox->set_params(DevicePlutoSDRBox::DEVICE_PHY, params); } - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplesource/remoteinput/remoteinput.cpp b/plugins/samplesource/remoteinput/remoteinput.cpp index 969bdc9e9..4500966b6 100644 --- a/plugins/samplesource/remoteinput/remoteinput.cpp +++ b/plugins/samplesource/remoteinput/remoteinput.cpp @@ -287,7 +287,7 @@ void RemoteInput::applySettings(const RemoteInputSettings& settings, const QList mutexLocker.unlock(); - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplesource/remotetcpinput/remotetcpinput.cpp b/plugins/samplesource/remotetcpinput/remotetcpinput.cpp index afc29a195..06619e3ac 100644 --- a/plugins/samplesource/remotetcpinput/remotetcpinput.cpp +++ b/plugins/samplesource/remotetcpinput/remotetcpinput.cpp @@ -244,7 +244,7 @@ void RemoteTCPInput::applySettings(const RemoteTCPInputSettings& settings, const mutexLocker.unlock(); - if (settingsKeys.contains("useReverseAPI")) + if (settings.m_useReverseAPI) { bool fullUpdate = (settingsKeys.contains("useReverseAPI") && settings.m_useReverseAPI) || settingsKeys.contains("reverseAPIAddress") || diff --git a/plugins/samplesource/rtlsdr/rtlsdrinput.cpp b/plugins/samplesource/rtlsdr/rtlsdrinput.cpp index 8cf3d0081..6edccc2c9 100644 --- a/plugins/samplesource/rtlsdr/rtlsdrinput.cpp +++ b/plugins/samplesource/rtlsdr/rtlsdrinput.cpp @@ -567,7 +567,7 @@ bool RTLSDRInput::applySettings(const RTLSDRSettings& settings, const QList