diff options
author | Dominik Riebeling <Dominik.Riebeling@gmail.com> | 2009-05-19 21:10:19 +0000 |
---|---|---|
committer | Dominik Riebeling <Dominik.Riebeling@gmail.com> | 2009-05-19 21:10:19 +0000 |
commit | f1fadb2b60ab534929dedbcf3d200a7d3bdc1e49 (patch) | |
tree | 27b86aa4f623a1bcac6df7aa64548a1ce569bc91 | |
parent | 792a7a608656be94d8c8c01c79a02ae0b3e47612 (diff) | |
download | rockbox-f1fadb2b60ab534929dedbcf3d200a7d3bdc1e49.tar.gz rockbox-f1fadb2b60ab534929dedbcf3d200a7d3bdc1e49.zip |
Clean up and improve themes install window debug messages.
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@20989 a1c6a512-1295-4272-9138-f99709370657
-rw-r--r-- | rbutil/rbutilqt/themesinstallwindow.cpp | 33 |
1 files changed, 17 insertions, 16 deletions
diff --git a/rbutil/rbutilqt/themesinstallwindow.cpp b/rbutil/rbutilqt/themesinstallwindow.cpp index 8b525546b3..9a9d5957ba 100644 --- a/rbutil/rbutilqt/themesinstallwindow.cpp +++ b/rbutil/rbutilqt/themesinstallwindow.cpp | |||
@@ -56,16 +56,15 @@ void ThemesInstallWindow::downloadInfo() | |||
56 | // try to get the current build information | 56 | // try to get the current build information |
57 | getter = new HttpGet(this); | 57 | getter = new HttpGet(this); |
58 | 58 | ||
59 | qDebug() << "downloading themes info"; | ||
60 | themesInfo.open(); | 59 | themesInfo.open(); |
61 | qDebug() << "file:" << themesInfo.fileName(); | 60 | qDebug() << "[Themes] downloading info to" << themesInfo.fileName(); |
62 | themesInfo.close(); | 61 | themesInfo.close(); |
63 | 62 | ||
64 | QUrl url; | 63 | QUrl url; |
65 | url = QUrl(RbSettings::value(RbSettings::ThemesUrl).toString() + "/rbutilqt.php?target=" | 64 | url = QUrl(RbSettings::value(RbSettings::ThemesUrl).toString() |
65 | + "/rbutilqt.php?target=" | ||
66 | + RbSettings::value(RbSettings::CurConfigureModel).toString()); | 66 | + RbSettings::value(RbSettings::CurConfigureModel).toString()); |
67 | qDebug() << "downloadInfo()" << url; | 67 | qDebug() << "[Themes] Info URL:" << url << "Query:" << url.queryItems(); |
68 | qDebug() << url.queryItems(); | ||
69 | if(RbSettings::value(RbSettings::CacheOffline).toBool()) | 68 | if(RbSettings::value(RbSettings::CacheOffline).toBool()) |
70 | getter->setCache(true); | 69 | getter->setCache(true); |
71 | getter->setFile(&themesInfo); | 70 | getter->setFile(&themesInfo); |
@@ -79,13 +78,13 @@ void ThemesInstallWindow::downloadInfo() | |||
79 | void ThemesInstallWindow::downloadDone(int id, bool error) | 78 | void ThemesInstallWindow::downloadDone(int id, bool error) |
80 | { | 79 | { |
81 | downloadDone(error); | 80 | downloadDone(error); |
82 | qDebug() << "downloadDone(bool) =" << id << error; | 81 | qDebug() << "[Themes] Download" << id << "done, error:" << error; |
83 | } | 82 | } |
84 | 83 | ||
85 | 84 | ||
86 | void ThemesInstallWindow::downloadDone(bool error) | 85 | void ThemesInstallWindow::downloadDone(bool error) |
87 | { | 86 | { |
88 | qDebug() << "downloadDone(bool) =" << error; | 87 | qDebug() << "[Themes] Download done, error:" << error; |
89 | 88 | ||
90 | disconnect(logger, SIGNAL(aborted()), getter, SLOT(abort())); | 89 | disconnect(logger, SIGNAL(aborted()), getter, SLOT(abort())); |
91 | disconnect(logger, SIGNAL(aborted()), this, SLOT(close())); | 90 | disconnect(logger, SIGNAL(aborted()), this, SLOT(close())); |
@@ -93,9 +92,10 @@ void ThemesInstallWindow::downloadDone(bool error) | |||
93 | 92 | ||
94 | QSettings iniDetails(themesInfo.fileName(), QSettings::IniFormat, this); | 93 | QSettings iniDetails(themesInfo.fileName(), QSettings::IniFormat, this); |
95 | QStringList tl = iniDetails.childGroups(); | 94 | QStringList tl = iniDetails.childGroups(); |
96 | qDebug() << iniDetails.value("error/code").toString() | 95 | qDebug() << "[Themes] Theme site result:" |
97 | << iniDetails.value("error/description").toString() | 96 | << iniDetails.value("error/code").toString() |
98 | << iniDetails.value("error/query").toString(); | 97 | << iniDetails.value("error/description").toString() |
98 | << iniDetails.value("error/query").toString(); | ||
99 | 99 | ||
100 | if(error) { | 100 | if(error) { |
101 | logger->addItem(tr("Network error: %1.\n" | 101 | logger->addItem(tr("Network error: %1.\n" |
@@ -109,7 +109,8 @@ void ThemesInstallWindow::downloadDone(bool error) | |||
109 | } | 109 | } |
110 | // handle possible error codes | 110 | // handle possible error codes |
111 | if(iniDetails.value("error/code").toInt() != 0 || !iniDetails.contains("error/code")) { | 111 | if(iniDetails.value("error/code").toInt() != 0 || !iniDetails.contains("error/code")) { |
112 | qDebug() << "error!"; | 112 | qDebug() << "[Themes] Theme site returned an error:" |
113 | << iniDetails.value("error/code"); | ||
113 | logger->addItem(tr("the following error occured:\n%1") | 114 | logger->addItem(tr("the following error occured:\n%1") |
114 | .arg(iniDetails.value("error/description", "unknown error").toString()), LOGERROR); | 115 | .arg(iniDetails.value("error/description", "unknown error").toString()), LOGERROR); |
115 | logger->setFinished(); | 116 | logger->setFinished(); |
@@ -128,7 +129,7 @@ void ThemesInstallWindow::downloadDone(bool error) | |||
128 | iniDetails.endGroup(); | 129 | iniDetails.endGroup(); |
129 | continue; | 130 | continue; |
130 | } | 131 | } |
131 | qDebug() << "adding theme:" << tl.at(i); | 132 | qDebug() << "[Themes] adding to list:" << tl.at(i); |
132 | // convert to unicode and replace HTML-specific entities | 133 | // convert to unicode and replace HTML-specific entities |
133 | QByteArray raw = iniDetails.value("name").toByteArray(); | 134 | QByteArray raw = iniDetails.value("name").toByteArray(); |
134 | QTextCodec* codec = QTextCodec::codecForHtml(raw); | 135 | QTextCodec* codec = QTextCodec::codecForHtml(raw); |
@@ -231,7 +232,7 @@ void ThemesInstallWindow::updateDetails(QListWidgetItem* cur, QListWidgetItem* p | |||
231 | 232 | ||
232 | void ThemesInstallWindow::updateImage(bool error) | 233 | void ThemesInstallWindow::updateImage(bool error) |
233 | { | 234 | { |
234 | qDebug() << "updateImage(bool) =" << error; | 235 | qDebug() << "[Themes] Updating image:"<< !error; |
235 | 236 | ||
236 | if(error) { | 237 | if(error) { |
237 | ui.themePreview->clear(); | 238 | ui.themePreview->clear(); |
@@ -257,7 +258,7 @@ void ThemesInstallWindow::updateImage(bool error) | |||
257 | 258 | ||
258 | void ThemesInstallWindow::resizeEvent(QResizeEvent* e) | 259 | void ThemesInstallWindow::resizeEvent(QResizeEvent* e) |
259 | { | 260 | { |
260 | qDebug() << "resizeEvent(QResizeEvent*) =" << e; | 261 | qDebug() << "[Themes]" << e; |
261 | 262 | ||
262 | QPixmap p, q; | 263 | QPixmap p, q; |
263 | QSize img; | 264 | QSize img; |
@@ -318,12 +319,12 @@ void ThemesInstallWindow::accept() | |||
318 | QDate().currentDate().toString("yyyyMMdd")).toString()); | 319 | QDate().currentDate().toString("yyyyMMdd")).toString()); |
319 | iniDetails.endGroup(); | 320 | iniDetails.endGroup(); |
320 | } | 321 | } |
321 | qDebug() << "installing themes:" << themes; | 322 | qDebug() << "[Themes] installing:" << themes; |
322 | 323 | ||
323 | logger = new ProgressLoggerGui(this); | 324 | logger = new ProgressLoggerGui(this); |
324 | logger->show(); | 325 | logger->show(); |
325 | QString mountPoint = RbSettings::value(RbSettings::Mountpoint).toString(); | 326 | QString mountPoint = RbSettings::value(RbSettings::Mountpoint).toString(); |
326 | qDebug() << "mountpoint:" << mountPoint; | 327 | qDebug() << "[Themes] mountpoint:" << mountPoint; |
327 | // show dialog with error if mount point is wrong | 328 | // show dialog with error if mount point is wrong |
328 | if(!QFileInfo(mountPoint).isDir()) { | 329 | if(!QFileInfo(mountPoint).isDir()) { |
329 | logger->addItem(tr("Mount point is wrong!"),LOGERROR); | 330 | logger->addItem(tr("Mount point is wrong!"),LOGERROR); |