diff options
author | David Faure <faure@kde.org> | 2011-10-22 11:33:06 +0200 |
---|---|---|
committer | Qt by Nokia <qt-info@nokia.com> | 2011-10-23 15:28:25 +0200 |
commit | dfa24768a3243eda68d91f573050bf23f96da2dd (patch) | |
tree | 40437c86c50a7ed1557caf8559129724899cd918 | |
parent | 59f91c0194f1ac83ae78e4b22f466e25873e1282 (diff) |
Deprecate QDesktopServices::storageLocation/displayName
Which required porting the related unittests to qstandardpaths
Change-Id: I6eb63c46845f05cd29cc42b59872707526277c90
Reviewed-by: Thiago Macieira (Intel) <thiago.macieira@intel.com>
-rw-r--r-- | examples/qtconcurrent/imagescaling/imagescaling.cpp | 4 | ||||
-rw-r--r-- | src/gui/util/qdesktopservices.cpp | 17 | ||||
-rw-r--r-- | src/gui/util/qdesktopservices.h | 13 | ||||
-rw-r--r-- | tests/auto/gui/util/qdesktopservices/tst_qdesktopservices.cpp | 44 | ||||
-rw-r--r-- | tests/benchmarks/network/access/qnetworkdiskcache/tst_qnetworkdiskcache.cpp | 18 |
5 files changed, 28 insertions, 68 deletions
diff --git a/examples/qtconcurrent/imagescaling/imagescaling.cpp b/examples/qtconcurrent/imagescaling/imagescaling.cpp index 6ed7ae7511..fe2e3f4dad 100644 --- a/examples/qtconcurrent/imagescaling/imagescaling.cpp +++ b/examples/qtconcurrent/imagescaling/imagescaling.cpp @@ -100,9 +100,9 @@ void Images::open() imageScaling->waitForFinished(); } - // Show a file open dialog at QDesktopServices::PicturesLocation. + // Show a file open dialog at QStandardPaths::PicturesLocation. QStringList files = QFileDialog::getOpenFileNames(this, tr("Select Images"), - QDesktopServices::storageLocation(QDesktopServices::PicturesLocation), + QStandardPaths::writableLocation(QStandardPaths::PicturesLocation), "*.jpg *.png"); if (files.count() == 0) diff --git a/src/gui/util/qdesktopservices.cpp b/src/gui/util/qdesktopservices.cpp index ab45c17dd3..4fbf0a59f3 100644 --- a/src/gui/util/qdesktopservices.cpp +++ b/src/gui/util/qdesktopservices.cpp @@ -270,21 +270,16 @@ void QDesktopServices::unsetUrlHandler(const QString &scheme) */ /*! - \deprecated Use QStandardPaths::writableLocation() + \fn QString QDesktopServices::storageLocation(StandardLocation type) + \obsolete + Use QStandardPaths::writableLocation() */ -QString QDesktopServices::storageLocation(StandardLocation type) -{ - return QStandardPaths::writableLocation(static_cast<QStandardPaths::StandardLocation>(type)); -} - /*! - \deprecated Use QStandardPaths::displayName() + \fn QString QDesktopServices::displayName(StandardLocation type) + \obsolete + Use QStandardPaths::displayName() */ -QString QDesktopServices::displayName(StandardLocation type) -{ - return QStandardPaths::displayName(static_cast<QStandardPaths::StandardLocation>(type)); -} QT_END_NAMESPACE diff --git a/src/gui/util/qdesktopservices.h b/src/gui/util/qdesktopservices.h index aba51959bd..ba7702f0d7 100644 --- a/src/gui/util/qdesktopservices.h +++ b/src/gui/util/qdesktopservices.h @@ -43,6 +43,9 @@ #define QDESKTOPSERVICES_H #include <QtCore/qstring.h> +#if QT_DEPRECATED_SINCE(5, 0) +#include <QtCore/qstandardpaths.h> +#endif QT_BEGIN_HEADER @@ -63,6 +66,7 @@ public: static void setUrlHandler(const QString &scheme, QObject *receiver, const char *method); static void unsetUrlHandler(const QString &scheme); +#if QT_DEPRECATED_SINCE(5, 0) enum StandardLocation { DesktopLocation, DocumentsLocation, @@ -77,8 +81,13 @@ public: CacheLocation }; - static QString storageLocation(StandardLocation type); - static QString displayName(StandardLocation type); + QT_DEPRECATED static QString storageLocation(StandardLocation type) { + return QStandardPaths::writableLocation(static_cast<QStandardPaths::StandardLocation>(type)); + } + QT_DEPRECATED static QString displayName(StandardLocation type) { + return QStandardPaths::displayName(static_cast<QStandardPaths::StandardLocation>(type)); + } +#endif }; #endif // QT_NO_DESKTOPSERVICES diff --git a/tests/auto/gui/util/qdesktopservices/tst_qdesktopservices.cpp b/tests/auto/gui/util/qdesktopservices/tst_qdesktopservices.cpp index 270674d31f..64af6c9613 100644 --- a/tests/auto/gui/util/qdesktopservices/tst_qdesktopservices.cpp +++ b/tests/auto/gui/util/qdesktopservices/tst_qdesktopservices.cpp @@ -61,11 +61,6 @@ private slots: void cleanup(); void openUrl(); void handlers(); - void storageLocation_data(); - void storageLocation(); - - void storageLocationDoesNotEndWithSlash_data(); - void storageLocationDoesNotEndWithSlash(); }; tst_qdesktopservices::tst_qdesktopservices() @@ -126,44 +121,5 @@ void tst_qdesktopservices::handlers() QCOMPARE(barHandler.lastHandledUrl.toString(), barUrl.toString()); } -Q_DECLARE_METATYPE(QDesktopServices::StandardLocation) -void tst_qdesktopservices::storageLocation_data() -{ - QTest::addColumn<QDesktopServices::StandardLocation>("location"); - QTest::newRow("DesktopLocation") << QDesktopServices::DesktopLocation; - QTest::newRow("DocumentsLocation") << QDesktopServices::DocumentsLocation; - QTest::newRow("FontsLocation") << QDesktopServices::FontsLocation; - QTest::newRow("ApplicationsLocation") << QDesktopServices::ApplicationsLocation; - QTest::newRow("MusicLocation") << QDesktopServices::MusicLocation; - QTest::newRow("MoviesLocation") << QDesktopServices::MoviesLocation; - QTest::newRow("PicturesLocation") << QDesktopServices::PicturesLocation; - QTest::newRow("TempLocation") << QDesktopServices::TempLocation; - QTest::newRow("HomeLocation") << QDesktopServices::HomeLocation; - QTest::newRow("DataLocation") << QDesktopServices::DataLocation; -} - -void tst_qdesktopservices::storageLocation() -{ - QFETCH(QDesktopServices::StandardLocation, location); - QDesktopServices::storageLocation(location); - QDesktopServices::displayName(location); -} - -void tst_qdesktopservices::storageLocationDoesNotEndWithSlash_data() -{ - storageLocation_data(); -} - -void tst_qdesktopservices::storageLocationDoesNotEndWithSlash() -{ - // Currently all desktop locations return their storage location - // with "Unix-style" paths (i.e. they use a slash, not backslash). - QFETCH(QDesktopServices::StandardLocation, location); - QString loc = QDesktopServices::storageLocation(location); - if (loc.size() > 1) // workaround for unlikely case of locations that return '/' - QCOMPARE(loc.endsWith(QLatin1Char('/')), false); -} - - QTEST_MAIN(tst_qdesktopservices) #include "tst_qdesktopservices.moc" diff --git a/tests/benchmarks/network/access/qnetworkdiskcache/tst_qnetworkdiskcache.cpp b/tests/benchmarks/network/access/qnetworkdiskcache/tst_qnetworkdiskcache.cpp index 3e1168e6fc..a81c71550d 100644 --- a/tests/benchmarks/network/access/qnetworkdiskcache/tst_qnetworkdiskcache.cpp +++ b/tests/benchmarks/network/access/qnetworkdiskcache/tst_qnetworkdiskcache.cpp @@ -47,7 +47,7 @@ #include <QDebug> #include <QtTest/QtTest> #include <QIODevice> -#include <QDesktopServices> +#include <QStandardPaths> @@ -109,8 +109,8 @@ void tst_qnetworkdiskcache::timeInsertion_data() { QTest::addColumn<QString>("cacheRootDirectory"); - QString cacheLoc = QDesktopServices::storageLocation(QDesktopServices::CacheLocation); - QTest::newRow("QDesktopServices Cache Location") << cacheLoc; + QString cacheLoc = QStandardPaths::writableLocation(QStandardPaths::CacheLocation); + QTest::newRow("QStandardPaths Cache Location") << cacheLoc; } //This functions times an insert() operation. @@ -170,8 +170,8 @@ void tst_qnetworkdiskcache::timeRead_data() { QTest::addColumn<QString>("cacheRootDirectory"); - QString cacheLoc = QDesktopServices::storageLocation(QDesktopServices::CacheLocation); - QTest::newRow("QDesktopServices Cache Location") << cacheLoc; + QString cacheLoc = QStandardPaths::writableLocation(QStandardPaths::CacheLocation); + QTest::newRow("QStandardPaths Cache Location") << cacheLoc; } //Times metadata as well payload lookup @@ -229,8 +229,8 @@ void tst_qnetworkdiskcache::timeRemoval_data() { QTest::addColumn<QString>("cacheRootDirectory"); - QString cacheLoc = QDesktopServices::storageLocation(QDesktopServices::CacheLocation); - QTest::newRow("QDesktopServices Cache Location") << cacheLoc; + QString cacheLoc = QStandardPaths::writableLocation(QStandardPaths::CacheLocation); + QTest::newRow("QStandardPaths Cache Location") << cacheLoc; } void tst_qnetworkdiskcache::timeRemoval() @@ -279,8 +279,8 @@ void tst_qnetworkdiskcache::timeExpiration_data() { QTest::addColumn<QString>("cacheRootDirectory"); - QString cacheLoc = QDesktopServices::storageLocation(QDesktopServices::CacheLocation); - QTest::newRow("QDesktopServices Cache Location") << cacheLoc; + QString cacheLoc = QStandardPaths::writableLocation(QStandardPaths::CacheLocation); + QTest::newRow("QStandardPaths Cache Location") << cacheLoc; } void tst_qnetworkdiskcache::timeExpiration() |