diff --git a/src/blackmisc/network/url.cpp b/src/blackmisc/network/url.cpp index 2b6a25473..ba13481d8 100644 --- a/src/blackmisc/network/url.cpp +++ b/src/blackmisc/network/url.cpp @@ -95,19 +95,18 @@ namespace BlackMisc return !m_query.isEmpty(); } - QString CUrl::appendQuery(const QString &queryToAppend) + void CUrl::appendQuery(const QString &queryToAppend) { - if (queryToAppend.isEmpty()) { return m_query; } + if (queryToAppend.isEmpty()) { return; } const QString q(stripQueryString(queryToAppend)); - if (q.isEmpty()) { return m_query; } + if (q.isEmpty()) { return; } m_query += hasQuery() ? "&" + q : q; - return m_query; } - QString CUrl::appendQuery(const QString &key, const QString &value) + void CUrl::appendQuery(const QString &key, const QString &value) { - if (key.isEmpty()) { return m_query; } - return this->appendQuery(key + "=" + value); + if (key.isEmpty()) { return; } + this->appendQuery(key + "=" + value); } QString CUrl::getFullUrl(bool withQuery) const diff --git a/src/blackmisc/network/url.h b/src/blackmisc/network/url.h index 243347764..5bc128dc3 100644 --- a/src/blackmisc/network/url.h +++ b/src/blackmisc/network/url.h @@ -106,10 +106,10 @@ namespace BlackMisc bool hasQuery() const; //! Append query - QString appendQuery(const QString &queryToAppend); + void appendQuery(const QString &queryToAppend); //! Append query - QString appendQuery(const QString &key, const QString &value); + void appendQuery(const QString &key, const QString &value); //! Empty bool isEmpty() const;