From 7c22e4d805f3ae23938fdfedcb02e53d26966dfd Mon Sep 17 00:00:00 2001 From: Berthold Stoeger Date: Sat, 29 Sep 2018 22:47:19 +0200 Subject: Cleanup: don't derive UserSurveyServices from WebServices UserSurveyServices derives from WebServices and therefore has to define three pure virtual functions [startDownload(), startUpload(), buttonClicked()] as no-ops. Interestingly, a comment in the header says "need to declare them as no ops or Qt4 is unhappy" which is of course not true as these functions are not declared by Qt. There seems to be no point in deriving from WebServices, therefore don't do it. These function definitions can then be removed. Signed-off-by: Berthold Stoeger --- desktop-widgets/subsurfacewebservices.cpp | 9 ++++----- desktop-widgets/subsurfacewebservices.h | 8 +------- 2 files changed, 5 insertions(+), 12 deletions(-) diff --git a/desktop-widgets/subsurfacewebservices.cpp b/desktop-widgets/subsurfacewebservices.cpp index 2dd39bffd..a7586c128 100644 --- a/desktop-widgets/subsurfacewebservices.cpp +++ b/desktop-widgets/subsurfacewebservices.cpp @@ -793,17 +793,16 @@ void DivelogsDeWebServices::buttonClicked(QAbstractButton *button) } } -UserSurveyServices::UserSurveyServices(QWidget *parent, Qt::WindowFlags f) : WebServices(parent, f) +UserSurveyServices::UserSurveyServices(QWidget *parent, Qt::WindowFlags f) : QDialog(parent, f) { - } -QNetworkReply* UserSurveyServices::sendSurvey(QString values) +QNetworkReply *UserSurveyServices::sendSurvey(QString values) { QNetworkRequest request; request.setUrl(QString("http://subsurface-divelog.org/survey?%1").arg(values)); request.setRawHeader("Accept", "text/xml"); - request.setRawHeader("User-Agent", userAgent.toUtf8()); - reply = manager()->get(request); + request.setRawHeader("User-Agent", getUserAgent().toUtf8()); + QNetworkReply *reply = manager()->get(request); return reply; } diff --git a/desktop-widgets/subsurfacewebservices.h b/desktop-widgets/subsurfacewebservices.h index ad68f08f3..b51d81271 100644 --- a/desktop-widgets/subsurfacewebservices.h +++ b/desktop-widgets/subsurfacewebservices.h @@ -77,17 +77,11 @@ private: bool uploadMode; }; -class UserSurveyServices : public WebServices { +class UserSurveyServices : public QDialog { Q_OBJECT public: QNetworkReply* sendSurvey(QString values); explicit UserSurveyServices(QWidget *parent = 0, Qt::WindowFlags f = 0); -private -slots: - // need to declare them as no ops or Qt4 is unhappy - void startDownload() { } - void startUpload() { } - void buttonClicked(QAbstractButton *button) { Q_UNUSED(button) } }; #endif // SUBSURFACEWEBSERVICES_H -- cgit v1.2.3-70-g09d2