Fixed #377 - Callbook query does not work

foldynl · May 29, 2024 · d7bc821 · d7bc821
1 parent ac7295d
commit d7bc821
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/core/HamQTH.cpp b/core/HamQTH.cpp
@@ -97,8 +97,6 @@ void HamQTH::queryCallsign(const QString &callsign)
         return;
     }
 
-    queuedCallsign = "";
-
     QUrlQuery query;
     query.addQueryItem("id", sessionId);
     query.addQueryItem("callsign", callsign);
@@ -114,6 +112,10 @@ void HamQTH::queryCallsign(const QString &callsign)
 
     currentReply = nam->get(QNetworkRequest(url));
     currentReply->setProperty("queryCallsign", callsign);
+
+    // Attention, variable callsign and queuedCallsign point to the same object
+    // queuedCallsign must be cleared after the last use of the callsign variable
+    queuedCallsign = QString();
 }
 
 void HamQTH::abortQuery()

diff --git a/core/QRZ.cpp b/core/QRZ.cpp
@@ -57,8 +57,6 @@ void QRZ::queryCallsign(const QString &callsign)
         return;
     }
 
-    queuedCallsign = "";
-
     QUrlQuery query;
     query.addQueryItem("s", sessionId);
 
@@ -86,6 +84,10 @@ void QRZ::queryCallsign(const QString &callsign)
     currentReply = nam->get(QNetworkRequest(url));
     currentReply->setProperty("queryCallsign", QVariant(callsign));
     currentReply->setProperty("messageType", QVariant("callsignInfoQuery"));
+
+    // Attention, variable callsign and queuedCallsign point to the same object
+    // queuedCallsign must be cleared after the last use of the callsign variable
+    queuedCallsign = QString();
 }
 
 void QRZ::abortQuery()