From 1fff760722e6a44311373648b074b5c6a491b00a Mon Sep 17 00:00:00 2001 From: sledgehammer999 Date: Thu, 27 Dec 2018 22:04:48 +0200 Subject: [PATCH] Drop upgrade code --- src/app/main.cpp | 21 +-- src/app/upgrade.h | 274 +-------------------------------------- src/base/preferences.cpp | 25 ---- src/base/preferences.h | 2 - 4 files changed, 9 insertions(+), 313 deletions(-) diff --git a/src/app/main.cpp b/src/app/main.cpp index 77e6110cc..90a87397f 100644 --- a/src/app/main.cpp +++ b/src/app/main.cpp @@ -27,8 +27,14 @@ * exception statement from your version. */ +#include + #include +#if !defined Q_OS_WIN && !defined Q_OS_HAIKU +#include +#endif + #include #include #include @@ -50,9 +56,6 @@ Q_IMPORT_PLUGIN(QICOPlugin) #else // NoGUI-only includes #include -#ifdef Q_OS_UNIX -#include "unistd.h" -#endif #endif // DISABLE_GUI #include @@ -110,23 +113,11 @@ int main(int argc, char *argv[]) // We must save it here because QApplication constructor may change it bool isOneArg = (argc == 2); -#ifdef Q_OS_MAC - // On macOS 10.12 Sierra, Apple changed the behaviour of CFPreferencesSetValue() https://bugreports.qt.io/browse/QTBUG-56344 - // Due to this, we have to move from native plist to IniFormat - macMigratePlists(); -#endif - try { // Create Application QString appId = QLatin1String("qBittorrent-") + Utils::Misc::getUserIDString(); QScopedPointer app(new Application(appId, argc, argv)); -#ifndef DISABLE_GUI - // after the application object creation because we need a profile to be set already - // for the migration - migrateRSS(); -#endif - const QBtCommandLineParameters params = app->commandLineArgs(); if (!params.unknownParameter.isEmpty()) { diff --git a/src/app/upgrade.h b/src/app/upgrade.h index 3e18c8c46..570bb2651 100644 --- a/src/app/upgrade.h +++ b/src/app/upgrade.h @@ -26,279 +26,11 @@ * exception statement from your version. */ -#ifndef UPGRADE_H -#define UPGRADE_H +#pragma once -#include -#include -#include - -#if LIBTORRENT_VERSION_NUM >= 10100 -#include -#else -#include -#endif - -#include -#include -#include -#include - -#ifndef DISABLE_GUI -#include -#endif -#ifdef Q_OS_MAC -#include -#endif - -#include "base/logger.h" -#include "base/preferences.h" -#include "base/profile.h" -#include "base/utils/fs.h" -#include "base/utils/misc.h" -#include "base/utils/string.h" - -bool userAcceptsUpgrade() -{ -#ifdef DISABLE_GUI - printf("\n*** %s ***\n", qUtf8Printable(QObject::tr("Upgrade"))); - char ret = '\0'; - do { - printf("%s\n" - , qUtf8Printable(QObject::tr("You updated from an older version that saved things differently. You must migrate to the new saving system. You will not be able to use an older version than v3.3.0 again. Continue? [y/n]"))); - ret = getchar(); // Read pressed key - } - while ((ret != 'y') && (ret != 'Y') && (ret != 'n') && (ret != 'N')); - - if ((ret == 'y') || (ret == 'Y')) - return true; -#else - QMessageBox msgBox; - msgBox.setText(QObject::tr("You updated from an older version that saved things differently. You must migrate to the new saving system. If you continue, you will not be able to use an older version than v3.3.0 again.")); - msgBox.setWindowTitle(QObject::tr("Upgrade")); - msgBox.addButton(QMessageBox::Abort); - msgBox.addButton(QMessageBox::Ok); - msgBox.setDefaultButton(QMessageBox::Abort); - msgBox.show(); // Need to be shown or to moveToCenter does not work - msgBox.move(Utils::Misc::screenCenter(&msgBox)); - if (msgBox.exec() == QMessageBox::Ok) - return true; -#endif // DISABLE_GUI - - return false; -} - -bool upgradeResumeFile(const QString &filepath, const QVariantHash &oldTorrent = QVariantHash()) -{ - QFile file1(filepath); - if (!file1.open(QIODevice::ReadOnly)) - return false; - - QByteArray data = file1.readAll(); - file1.close(); - - libtorrent::error_code ec; -#if LIBTORRENT_VERSION_NUM < 10100 - libtorrent::lazy_entry fastOld; - libtorrent::lazy_bdecode(data.constData(), data.constData() + data.size(), fastOld, ec); - if (ec || (fastOld.type() != libtorrent::lazy_entry::dict_t)) return false; -#else - libtorrent::bdecode_node fastOld; - libtorrent::bdecode(data.constData(), data.constData() + data.size(), fastOld, ec); - if (ec || (fastOld.type() != libtorrent::bdecode_node::dict_t)) return false; -#endif - - libtorrent::entry fastNew; - fastNew = fastOld; - - bool v3_3 = false; - int queuePosition = 0; - QString outFilePath = filepath; - static const QRegularExpression rx(QLatin1String("([A-Fa-f0-9]{40})\\.fastresume\\.(.+)$")); - const QRegularExpressionMatch rxMatch = rx.match(filepath); - if (rxMatch.hasMatch()) { - // Old v3.3.x format had a number at the end indicating the queue position. - // The naming scheme was '.fastresume.'. - // However, QSaveFile, which uses QTemporaryFile internally, might leave - // non-commited files behind eg after a crash. These files have the - // naming scheme '.fastresume.XXXXXX' where each X is a random - // character. So we detect if the last part is present. Then check if it - // is 6 chars long. If all the 6 chars are digits we assume it is an old - // v3.3.x format. Otherwise it is considered a non-commited fastresume - // and is deleted, because it may be a corrupted/incomplete fastresume. - // NOTE: When the upgrade code is removed, we must continue to perform - // cleanup of non-commited QSaveFile/QTemporaryFile fastresumes - queuePosition = rxMatch.captured(2).toInt(); - if ((rxMatch.captured(2).size() == 6) && (queuePosition <= 99999)) { - Utils::Fs::forceRemove(filepath); - return true; - } - - v3_3 = true; - outFilePath.replace(QRegularExpression("\\.fastresume\\..+$"), ".fastresume"); - } - else { - queuePosition = fastOld.dict_find_int_value("qBt-queuePosition", 0); - fastNew["qBt-name"] = oldTorrent.value("name").toString().toStdString(); - fastNew["qBt-tempPathDisabled"] = false; - } - - // in versions < 3.3 we have -1 for seeding torrents, so we convert it to 0 - fastNew["qBt-queuePosition"] = (queuePosition >= 0 ? queuePosition : 0); - - if (v3_3) { - QFileInfo oldFile(filepath); - QFileInfo newFile(outFilePath); - if (newFile.exists() - && (oldFile.lastModified() < newFile.lastModified())) { - Utils::Fs::forceRemove(filepath); - return true; - } - } - QFile file2(outFilePath); - QVector out; - libtorrent::bencode(std::back_inserter(out), fastNew); - if (file2.open(QIODevice::WriteOnly)) { - if (file2.write(&out[0], out.size()) == out.size()) { - if (v3_3) - Utils::Fs::forceRemove(filepath); - return true; - } - } - - return false; -} - -bool upgrade(bool ask = true) +bool upgrade(bool /*ask*/ = true) { - // Upgrade preferences - Preferences::instance()->upgrade(); - - QString backupFolderPath = Utils::Fs::expandPathAbs(specialFolderLocation(SpecialFolder::Data) + "BT_backup"); - QDir backupFolderDir(backupFolderPath); - - // **************************************************************************************** - // Silently converts old v3.3.x .fastresume files - const QStringList backupFiles_3_3 = backupFolderDir.entryList( - QStringList(QLatin1String("*.fastresume.*")), QDir::Files, QDir::Unsorted); - for (const QString &backupFile : backupFiles_3_3) - upgradeResumeFile(backupFolderDir.absoluteFilePath(backupFile)); - // **************************************************************************************** - - SettingsPtr oldResumeSettings = Profile::instance().applicationSettings(QLatin1String("qBittorrent-resume")); - QString oldResumeFilename = oldResumeSettings->fileName(); - QVariantHash oldResumeData = oldResumeSettings->value("torrents").toHash(); - oldResumeSettings.reset(); - - if (oldResumeData.isEmpty()) { - Utils::Fs::forceRemove(oldResumeFilename); - return true; - } - - if (ask && !userAcceptsUpgrade()) return false; - - const QStringList backupFiles = backupFolderDir.entryList( - QStringList(QLatin1String("*.fastresume")), QDir::Files, QDir::Unsorted); - const QRegularExpression rx(QLatin1String("^([A-Fa-f0-9]{40})\\.fastresume$")); - for (const QString &backupFile : backupFiles) { - const QRegularExpressionMatch rxMatch = rx.match(backupFile); - if (rxMatch.hasMatch()) { - const QString hashStr = rxMatch.captured(1); - if (upgradeResumeFile(backupFolderDir.absoluteFilePath(backupFile), oldResumeData[hashStr].toHash())) - oldResumeData.remove(hashStr); - else - Logger::instance()->addMessage(QObject::tr("Couldn't migrate torrent with hash: %1").arg(hashStr), Log::WARNING); - } - else { - Logger::instance()->addMessage(QObject::tr("Couldn't migrate torrent. Invalid fastresume file name: %1").arg(backupFile), Log::WARNING); - } - } - - for (auto i = oldResumeData.cbegin(); i != oldResumeData.cend(); ++i) { - const QVariantHash oldTorrent = i.value().toHash(); - if (oldTorrent.value("is_magnet", false).toBool()) { - const QString &hash = i.key(); - libtorrent::entry resumeData; - resumeData["qBt-magnetUri"] = oldTorrent.value("magnet_uri").toString().toStdString(); - resumeData["qBt-paused"] = false; - resumeData["qBt-forced"] = false; - - resumeData["qBt-savePath"] = oldTorrent.value("save_path").toString().toStdString(); - resumeData["qBt-ratioLimit"] = static_cast(oldTorrent.value("max_ratio", -2).toReal() * 1000); - resumeData["qBt-label"] = oldTorrent.value("label").toString().toStdString(); - resumeData["qBt-name"] = oldTorrent.value("name").toString().toStdString(); - resumeData["qBt-seedStatus"] = oldTorrent.value("seed").toBool(); - resumeData["qBt-tempPathDisabled"] = false; - - int queuePosition = oldTorrent.value("priority", 0).toInt(); - resumeData["qBt-queuePosition"] = (queuePosition >= 0 ? queuePosition : 0); - - QString filename = QString("%1.fastresume").arg(hash); - QString filepath = backupFolderDir.absoluteFilePath(filename); - - QFile resumeFile(filepath); - QVector out; - libtorrent::bencode(std::back_inserter(out), resumeData); - if (resumeFile.open(QIODevice::WriteOnly)) - resumeFile.write(&out[0], out.size()); - } - } - - int counter = 0; - QString backupResumeFilename = oldResumeFilename + ".bak"; - while (QFile::exists(backupResumeFilename)) { - ++counter; - backupResumeFilename = oldResumeFilename + ".bak" + QString::number(counter); - } - QFile::rename(oldResumeFilename, backupResumeFilename); + // Intentionally left no-op as a placeholder for future use return true; } - -#ifdef Q_OS_MAC -void migratePlistToIni(const QString &application) -{ - QSettings iniFile(QSettings::IniFormat, QSettings::UserScope, "qBittorrent", application); - if (!iniFile.allKeys().isEmpty()) return; // We copy the contents of plist, only if inifile does not exist(is empty). - - QSettings *plistFile = new QSettings("qBittorrent", application); - plistFile->setFallbacksEnabled(false); - const QStringList plist = plistFile->allKeys(); - if (!plist.isEmpty()) { - for (const QString &key : plist) - iniFile.setValue(key, plistFile->value(key)); - plistFile->clear(); - } - - QString plistPath = plistFile->fileName(); - delete plistFile; - Utils::Fs::forceRemove(plistPath); -} - -void macMigratePlists() -{ - migratePlistToIni("qBittorrent-data"); - migratePlistToIni("qBittorrent-rss"); - migratePlistToIni("qBittorrent"); -} -#endif // Q_OS_MAC - -#ifndef DISABLE_GUI -void migrateRSS() -{ - // Copy old feed items to new file if needed - SettingsPtr qBTRSS = Profile::instance().applicationSettings(QLatin1String("qBittorrent-rss-feeds")); - if (!qBTRSS->allKeys().isEmpty()) return; // We move the contents of RSS old_items only if inifile does not exist (is empty). - - SettingsPtr qBTRSSLegacy = Profile::instance().applicationSettings(QLatin1String("qBittorrent-rss")); - QHash allOldItems = qBTRSSLegacy->value("old_items", QHash()).toHash(); - - if (!allOldItems.empty()) { - qDebug("Moving %d old items for feeds to qBittorrent-rss-feeds", allOldItems.size()); - qBTRSS->setValue("old_items", allOldItems); - qBTRSSLegacy->remove("old_items"); - } -} -#endif // DISABLE_GUI - -#endif // UPGRADE_H diff --git a/src/base/preferences.cpp b/src/base/preferences.cpp index c0b68594d..7758d483b 100644 --- a/src/base/preferences.cpp +++ b/src/base/preferences.cpp @@ -1453,31 +1453,6 @@ void Preferences::setSpeedWidgetGraphEnable(int id, const bool enable) setValue("SpeedWidget/graph_enable_" + QString::number(id), enable); } -void Preferences::upgrade() -{ - SettingsStorage *settingsStorage = SettingsStorage::instance(); - - const QStringList labels = value("TransferListFilters/customLabels").toStringList(); - if (!labels.isEmpty()) { - QVariantMap categories = value("BitTorrent/Session/Categories").toMap(); - for (const QString &label : labels) { - if (!categories.contains(label)) - categories[label] = ""; - } - setValue("BitTorrent/Session/Categories", categories); - settingsStorage->removeValue("TransferListFilters/customLabels"); - } - - settingsStorage->removeValue("Preferences/Downloads/AppendLabel"); - - // Inhibit sleep based on running downloads/available seeds rather than network activity. - if (value("Preferences/General/PreventFromSuspend", false).toBool()) { - setPreventFromSuspendWhenDownloading(true); - setPreventFromSuspendWhenSeeding(true); - } - settingsStorage->removeValue("Preferences/General/PreventFromSuspend"); -} - void Preferences::apply() { if (SettingsStorage::instance()->save()) diff --git a/src/base/preferences.h b/src/base/preferences.h index a4dbe4d32..ba842f119 100644 --- a/src/base/preferences.h +++ b/src/base/preferences.h @@ -379,8 +379,6 @@ public: bool getSpeedWidgetGraphEnable(int id) const; void setSpeedWidgetGraphEnable(int id, const bool enable); - void upgrade(); - public slots: void setStatusFilterState(bool checked); void setCategoryFilterState(bool checked);