summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPetr Mrázek <peterix@gmail.com>2013-10-09 23:16:10 +0200
committerPetr Mrázek <peterix@gmail.com>2013-10-09 23:16:10 +0200
commiteaf0cbeafc5ff70bd2bb0d66b5f5980a71f824c5 (patch)
treee3a844f6b6c4fa31da606de992a28af45d6695f3
parent595943244ca929e5c566a78efae9a017ebaed69f (diff)
downloadMultiMC-eaf0cbeafc5ff70bd2bb0d66b5f5980a71f824c5.tar
MultiMC-eaf0cbeafc5ff70bd2bb0d66b5f5980a71f824c5.tar.gz
MultiMC-eaf0cbeafc5ff70bd2bb0d66b5f5980a71f824c5.tar.lz
MultiMC-eaf0cbeafc5ff70bd2bb0d66b5f5980a71f824c5.tar.xz
MultiMC-eaf0cbeafc5ff70bd2bb0d66b5f5980a71f824c5.zip
Fix MMC-15
``mod does not delete from jar''
-rw-r--r--gui/mainwindow.cpp5
-rw-r--r--logic/LegacyInstance.cpp125
-rw-r--r--logic/ModList.cpp241
-rw-r--r--logic/OneSixInstance.cpp6
4 files changed, 183 insertions, 194 deletions
diff --git a/gui/mainwindow.cpp b/gui/mainwindow.cpp
index 70d26e02..5784b85a 100644
--- a/gui/mainwindow.cpp
+++ b/gui/mainwindow.cpp
@@ -507,10 +507,11 @@ void MainWindow::onLoginComplete()
}
else
{
- ProgressDialog *tDialog = new ProgressDialog(this);
+ ProgressDialog tDialog(this);
connect(updateTask, SIGNAL(succeeded()), SLOT(onGameUpdateComplete()));
connect(updateTask, SIGNAL(failed(QString)), SLOT(onGameUpdateError(QString)));
- tDialog->exec(updateTask);
+ tDialog.exec(updateTask);
+ delete updateTask;
}
}
diff --git a/logic/LegacyInstance.cpp b/logic/LegacyInstance.cpp
index 205f6873..f741caad 100644
--- a/logic/LegacyInstance.cpp
+++ b/logic/LegacyInstance.cpp
@@ -14,8 +14,9 @@
#define LAUNCHER_FILE "MultiMCLauncher.jar"
-LegacyInstance::LegacyInstance(const QString& rootDir, SettingsObject* settings, QObject* parent)
- :BaseInstance( new LegacyInstancePrivate(),rootDir, settings, parent)
+LegacyInstance::LegacyInstance(const QString &rootDir, SettingsObject *settings,
+ QObject *parent)
+ : BaseInstance(new LegacyInstancePrivate(), rootDir, settings, parent)
{
settings->registerSetting(new Setting("NeedsRebuild", true));
settings->registerSetting(new Setting("ShouldUpdate", false));
@@ -24,50 +25,51 @@ LegacyInstance::LegacyInstance(const QString& rootDir, SettingsObject* settings,
settings->registerSetting(new Setting("IntendedJarVersion", ""));
}
-BaseUpdate* LegacyInstance::doUpdate()
+BaseUpdate *LegacyInstance::doUpdate()
{
+ auto list = jarModList();
return new LegacyUpdate(this, this);
}
-MinecraftProcess* LegacyInstance::prepareForLaunch(LoginResponse response)
+MinecraftProcess *LegacyInstance::prepareForLaunch(LoginResponse response)
{
- MinecraftProcess * proc = new MinecraftProcess(this);
-
+ MinecraftProcess *proc = new MinecraftProcess(this);
+
QIcon icon = MMC->icons()->getIcon(iconKey());
- auto pixmap = icon.pixmap(128,128);
- pixmap.save(PathCombine(minecraftRoot(), "icon.png"),"PNG");
-
+ auto pixmap = icon.pixmap(128, 128);
+ pixmap.save(PathCombine(minecraftRoot(), "icon.png"), "PNG");
+
// extract the legacy launcher
QFile(":/launcher/launcher.jar").copy(PathCombine(minecraftRoot(), LAUNCHER_FILE));
-
+
// set the process arguments
{
QStringList args;
-
+
// window size
QString windowSize;
if (settings().get("LaunchMaximized").toBool())
windowSize = "max";
else
- windowSize = QString("%1x%2").
- arg(settings().get("MinecraftWinWidth").toInt()).
- arg(settings().get("MinecraftWinHeight").toInt());
-
+ windowSize = QString("%1x%2").arg(settings().get("MinecraftWinWidth").toInt()).arg(
+ settings().get("MinecraftWinHeight").toInt());
+
// window title
QString windowTitle;
windowTitle.append("MultiMC: ").append(name());
-
+
// Java arguments
args.append(Util::Commandline::splitArgs(settings().get("JvmArgs").toString()));
-
+
#ifdef OSX
// OSX dock icon and name
args << "-Xdock:icon=icon.png";
args << QString("-Xdock:name=\"%1\"").arg(windowTitle);
#endif
-
- QString lwjgl = QDir(MMC->settings()->get("LWJGLDir").toString() + "/" + lwjglVersion()).absolutePath();
-
+
+ QString lwjgl = QDir(MMC->settings()->get("LWJGLDir").toString() + "/" + lwjglVersion())
+ .absolutePath();
+
// launcher arguments
args << QString("-Xms%1m").arg(settings().get("MinMemAlloc").toInt());
args << QString("-Xmx%1m").arg(settings().get("MaxMemAlloc").toInt());
@@ -80,41 +82,39 @@ MinecraftProcess* LegacyInstance::prepareForLaunch(LoginResponse response)
args << lwjgl;
proc->setMinecraftArguments(args);
}
-
+
// set the process work path
proc->setMinecraftWorkdir(minecraftRoot());
-
+
return proc;
}
void LegacyInstance::cleanupAfterRun()
{
- //FIXME: delete the launcher and icons and whatnot.
+ // FIXME: delete the launcher and icons and whatnot.
}
-std::shared_ptr< ModList > LegacyInstance::coreModList()
+std::shared_ptr<ModList> LegacyInstance::coreModList()
{
I_D(LegacyInstance);
- if(!d->core_mod_list)
+ if (!d->core_mod_list)
{
d->core_mod_list.reset(new ModList(coreModsDir()));
}
- else
- d->core_mod_list->update();
+ d->core_mod_list->update();
return d->core_mod_list;
}
-std::shared_ptr< ModList > LegacyInstance::jarModList()
+std::shared_ptr<ModList> LegacyInstance::jarModList()
{
I_D(LegacyInstance);
- if(!d->jar_mod_list)
+ if (!d->jar_mod_list)
{
auto list = new ModList(jarModsDir(), modListFile());
connect(list, SIGNAL(changed()), SLOT(jarModsChanged()));
d->jar_mod_list.reset(list);
}
- else
- d->jar_mod_list->update();
+ d->jar_mod_list->update();
return d->jar_mod_list;
}
@@ -123,38 +123,33 @@ void LegacyInstance::jarModsChanged()
setShouldRebuild(true);
}
-
-std::shared_ptr< ModList > LegacyInstance::loaderModList()
+std::shared_ptr<ModList> LegacyInstance::loaderModList()
{
I_D(LegacyInstance);
- if(!d->loader_mod_list)
+ if (!d->loader_mod_list)
{
d->loader_mod_list.reset(new ModList(loaderModsDir()));
}
- else
- d->loader_mod_list->update();
+ d->loader_mod_list->update();
return d->loader_mod_list;
}
-std::shared_ptr< ModList > LegacyInstance::texturePackList()
+std::shared_ptr<ModList> LegacyInstance::texturePackList()
{
I_D(LegacyInstance);
- if(!d->texture_pack_list)
+ if (!d->texture_pack_list)
{
d->texture_pack_list.reset(new ModList(texturePacksDir()));
}
- else
- d->texture_pack_list->update();
+ d->texture_pack_list->update();
return d->texture_pack_list;
}
-
-QDialog * LegacyInstance::createModEditDialog ( QWidget* parent )
+QDialog *LegacyInstance::createModEditDialog(QWidget *parent)
{
return new LegacyModEditDialog(this, parent);
}
-
QString LegacyInstance::jarModsDir() const
{
return PathCombine(instanceRoot(), "instMods");
@@ -204,7 +199,6 @@ QString LegacyInstance::instanceConfigFolder() const
return PathCombine(minecraftRoot(), "config");
}
-
/*
bool LegacyInstance::shouldUpdateCurrentVersion() const
{
@@ -215,21 +209,22 @@ bool LegacyInstance::shouldUpdateCurrentVersion() const
void LegacyInstance::updateCurrentVersion(bool keepCurrent)
{
QFileInfo jar(runnableJar());
-
+
if(!jar.exists())
{
setLastCurrentVersionUpdate(0);
setCurrentVersionId("Unknown");
return;
}
-
+
qint64 time = jar.lastModified().toUTC().toMSecsSinceEpoch();
-
+
setLastCurrentVersionUpdate(time);
if (!keepCurrent)
{
// TODO: Implement GetMinecraftJarVersion function.
- QString newVersion = "Unknown";//javautils::GetMinecraftJarVersion(jar.absoluteFilePath());
+ QString newVersion =
+"Unknown";//javautils::GetMinecraftJarVersion(jar.absoluteFilePath());
setCurrentVersionId(newVersion);
}
}
@@ -247,41 +242,41 @@ void LegacyInstance::setLastCurrentVersionUpdate ( qint64 val )
bool LegacyInstance::shouldRebuild() const
{
I_D(LegacyInstance);
- return d->m_settings->get ( "NeedsRebuild" ).toBool();
+ return d->m_settings->get("NeedsRebuild").toBool();
}
-void LegacyInstance::setShouldRebuild ( bool val )
+void LegacyInstance::setShouldRebuild(bool val)
{
I_D(LegacyInstance);
- d->m_settings->set ( "NeedsRebuild", val );
+ d->m_settings->set("NeedsRebuild", val);
}
QString LegacyInstance::currentVersionId() const
{
I_D(LegacyInstance);
- return d->m_settings->get ( "JarVersion" ).toString();
+ return d->m_settings->get("JarVersion").toString();
}
-void LegacyInstance::setCurrentVersionId ( QString val )
+void LegacyInstance::setCurrentVersionId(QString val)
{
I_D(LegacyInstance);
- d->m_settings->set ( "JarVersion", val );
+ d->m_settings->set("JarVersion", val);
}
QString LegacyInstance::lwjglVersion() const
{
I_D(LegacyInstance);
- return d->m_settings->get ( "LwjglVersion" ).toString();
+ return d->m_settings->get("LwjglVersion").toString();
}
-void LegacyInstance::setLWJGLVersion ( QString val )
+void LegacyInstance::setLWJGLVersion(QString val)
{
I_D(LegacyInstance);
- d->m_settings->set ( "LwjglVersion", val );
+ d->m_settings->set("LwjglVersion", val);
}
QString LegacyInstance::intendedVersionId() const
{
I_D(LegacyInstance);
- return d->m_settings->get ( "IntendedJarVersion" ).toString();
+ return d->m_settings->get("IntendedJarVersion").toString();
}
-bool LegacyInstance::setIntendedVersionId ( QString version )
+bool LegacyInstance::setIntendedVersionId(QString version)
{
settings().set("IntendedJarVersion", version);
setShouldUpdate(true);
@@ -290,16 +285,16 @@ bool LegacyInstance::setIntendedVersionId ( QString version )
bool LegacyInstance::shouldUpdate() const
{
I_D(LegacyInstance);
- QVariant var = settings().get ( "ShouldUpdate" );
- if ( !var.isValid() || var.toBool() == false )
+ QVariant var = settings().get("ShouldUpdate");
+ if (!var.isValid() || var.toBool() == false)
{
return intendedVersionId() != currentVersionId();
}
return true;
}
-void LegacyInstance::setShouldUpdate ( bool val )
+void LegacyInstance::setShouldUpdate(bool val)
{
- settings().set ( "ShouldUpdate", val );
+ settings().set("ShouldUpdate", val);
}
QString LegacyInstance::defaultBaseJar() const
@@ -312,7 +307,7 @@ QString LegacyInstance::defaultCustomBaseJar() const
return PathCombine(binDir(), "mcbackup.jar");
}
-bool LegacyInstance::menuActionEnabled ( QString action_name ) const
+bool LegacyInstance::menuActionEnabled(QString action_name) const
{
if (action_name == "actionChangeInstMCVersion")
return false;
@@ -321,7 +316,7 @@ bool LegacyInstance::menuActionEnabled ( QString action_name ) const
QString LegacyInstance::getStatusbarDescription()
{
- if(shouldUpdate())
+ if (shouldUpdate())
return "Legacy : " + currentVersionId() + " -> " + intendedVersionId();
else
return "Legacy : " + currentVersionId();
diff --git a/logic/ModList.cpp b/logic/ModList.cpp
index a600afff..236f0db6 100644
--- a/logic/ModList.cpp
+++ b/logic/ModList.cpp
@@ -1,12 +1,12 @@
-//
+//
// Copyright 2013 MultiMC Contributors
-//
+//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
-//
+//
// http://www.apache.org/licenses/LICENSE-2.0
-//
+//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -23,22 +23,23 @@
#include <QFileSystemWatcher>
#include <logger/QsLog.h>
-ModList::ModList ( const QString& dir, const QString& list_file )
-: QAbstractListModel(), m_dir(dir), m_list_file(list_file)
+ModList::ModList(const QString &dir, const QString &list_file)
+ : QAbstractListModel(), m_dir(dir), m_list_file(list_file)
{
- m_dir.setFilter(QDir::Readable | QDir::NoDotAndDotDot | QDir::Files | QDir::Dirs | QDir::NoSymLinks);
+ m_dir.setFilter(QDir::Readable | QDir::NoDotAndDotDot | QDir::Files | QDir::Dirs |
+ QDir::NoSymLinks);
m_dir.setSorting(QDir::Name);
m_list_id = QUuid::createUuid().toString();
m_watcher = new QFileSystemWatcher(this);
is_watching = false;
- connect(m_watcher,SIGNAL(directoryChanged(QString)), this, SLOT(directoryChanged(QString)));
- update();
+ connect(m_watcher, SIGNAL(directoryChanged(QString)), this,
+ SLOT(directoryChanged(QString)));
}
void ModList::startWatching()
{
is_watching = m_watcher->addPath(m_dir.absolutePath());
- if(is_watching)
+ if (is_watching)
QLOG_INFO() << "Started watching " << m_dir.absolutePath();
else
QLOG_INFO() << "Failed to start watching " << m_dir.absolutePath();
@@ -47,32 +48,31 @@ void ModList::startWatching()
void ModList::stopWatching()
{
is_watching = !m_watcher->removePath(m_dir.absolutePath());
- if(!is_watching)
+ if (!is_watching)
QLOG_INFO() << "Stopped watching " << m_dir.absolutePath();
else
QLOG_INFO() << "Failed to stop watching " << m_dir.absolutePath();
}
-
bool ModList::update()
{
if (!isValid())
return false;
-
+
QList<Mod> newMods;
m_dir.refresh();
auto folderContents = m_dir.entryInfoList();
bool orderWasInvalid = false;
-
+
// first, process the ordered items (if any)
int currentOrderIndex = 0;
QStringList listOrder = readListFile();
- for(auto item: listOrder)
+ for (auto item : listOrder)
{
- QFileInfo info (m_dir.filePath(item));
+ QFileInfo info(m_dir.filePath(item));
int idx = folderContents.indexOf(info);
// if the file from the index file exists
- if(idx != -1)
+ if (idx != -1)
{
// remove from the actual folder contents list
folderContents.takeAt(idx);
@@ -84,26 +84,27 @@ bool ModList::update()
orderWasInvalid = true;
}
}
- for(auto entry: folderContents)
+ for (auto entry : folderContents)
{
newMods.append(Mod(entry));
}
- if(mods.size() != newMods.size())
+ if (mods.size() != newMods.size())
{
orderWasInvalid = true;
}
- else for(int i = 0; i < mods.size(); i++)
- {
- if(!mods[i].strongCompare(newMods[i]))
+ else
+ for (int i = 0; i < mods.size(); i++)
{
- orderWasInvalid = true;
- break;
+ if (!mods[i].strongCompare(newMods[i]))
+ {
+ orderWasInvalid = true;
+ break;
+ }
}
- }
beginResetModel();
mods.swap(newMods);
endResetModel();
- if(orderWasInvalid)
+ if (orderWasInvalid)
{
saveListFile();
emit changed();
@@ -111,22 +112,21 @@ bool ModList::update()
return true;
}
-void ModList::directoryChanged ( QString path )
+void ModList::directoryChanged(QString path)
{
update();
}
-
QStringList ModList::readListFile()
{
QStringList stringList;
- if(m_list_file.isNull() || m_list_file.isEmpty())
+ if (m_list_file.isNull() || m_list_file.isEmpty())
return stringList;
-
+
QFile textFile(m_list_file);
- if(!textFile.open(QIODevice::ReadOnly | QIODevice::Text))
+ if (!textFile.open(QIODevice::ReadOnly | QIODevice::Text))
return QStringList();
-
+
QTextStream textStream(&textFile);
while (true)
{
@@ -144,13 +144,13 @@ QStringList ModList::readListFile()
bool ModList::saveListFile()
{
- if(m_list_file.isNull() || m_list_file.isEmpty())
+ if (m_list_file.isNull() || m_list_file.isEmpty())
return false;
QFile textFile(m_list_file);
- if(!textFile.open(QIODevice::WriteOnly | QIODevice::Text | QIODevice::Truncate))
+ if (!textFile.open(QIODevice::WriteOnly | QIODevice::Text | QIODevice::Truncate))
return false;
QTextStream textStream(&textFile);
- for(auto mod:mods)
+ for (auto mod : mods)
{
auto pathname = mod.filename();
QString filename = pathname.fileName();
@@ -160,29 +160,28 @@ bool ModList::saveListFile()
return false;
}
-
bool ModList::isValid()
{
return m_dir.exists() && m_dir.isReadable();
}
-bool ModList::installMod ( const QFileInfo& filename, int index )
+bool ModList::installMod(const QFileInfo &filename, int index)
{
- if(!filename.exists() || !filename.isReadable() || index < 0)
+ if (!filename.exists() || !filename.isReadable() || index < 0)
{
return false;
}
Mod m(filename);
- if(!m.valid())
+ if (!m.valid())
return false;
-
+
// if it's already there, replace the original mod (in place)
int idx = mods.indexOf(m);
- if(idx != -1)
+ if (idx != -1)
{
- if(mods[idx].replace(m))
+ if (mods[idx].replace(m))
{
-
+
auto left = this->index(index);
auto right = this->index(index, columnCount(QModelIndex()) - 1);
emit dataChanged(left, right);
@@ -192,33 +191,33 @@ bool ModList::installMod ( const QFileInfo& filename, int index )
}
return false;
}
-
+
auto type = m.type();
- if(type == Mod::MOD_UNKNOWN)
+ if (type == Mod::MOD_UNKNOWN)
return false;
- if(type == Mod::MOD_SINGLEFILE || type == Mod::MOD_ZIPFILE)
+ if (type == Mod::MOD_SINGLEFILE || type == Mod::MOD_ZIPFILE)
{
QString newpath = PathCombine(m_dir.path(), filename.fileName());
- if(!QFile::copy(filename.filePath(), newpath))
+ if (!QFile::copy(filename.filePath(), newpath))
return false;
m.repath(newpath);
beginInsertRows(QModelIndex(), index, index);
- mods.insert(index,m);
+ mods.insert(index, m);
endInsertRows();
saveListFile();
emit changed();
return true;
}
- else if(type == Mod::MOD_FOLDER)
+ else if (type == Mod::MOD_FOLDER)
{
-
+
QString from = filename.filePath();
QString to = PathCombine(m_dir.path(), filename.fileName());
- if(!copyPath(from, to))
+ if (!copyPath(from, to))
return false;
m.repath(to);
beginInsertRows(QModelIndex(), index, index);
- mods.insert(index,m);
+ mods.insert(index, m);
endInsertRows();
saveListFile();
emit changed();
@@ -227,12 +226,12 @@ bool ModList::installMod ( const QFileInfo& filename, int index )
return false;
}
-bool ModList::deleteMod ( int index )
+bool ModList::deleteMod(int index)
{
- if(index >= mods.size() || index < 0)
+ if (index >= mods.size() || index < 0)
return false;
- Mod & m = mods[index];
- if(m.destroy())
+ Mod &m = mods[index];
+ if (m.destroy())
{
beginRemoveRows(QModelIndex(), index, index);
mods.removeAt(index);
@@ -244,11 +243,11 @@ bool ModList::deleteMod ( int index )
return false;
}
-bool ModList::deleteMods ( int first, int last )
+bool ModList::deleteMods(int first, int last)
{
- for(int i = first; i <= last; i++)
+ for (int i = first; i <= last; i++)
{
- Mod & m = mods[i];
+ Mod &m = mods[i];
m.destroy();
}
beginRemoveRows(QModelIndex(), first, last);
@@ -259,18 +258,17 @@ bool ModList::deleteMods ( int first, int last )
return true;
}
-
-bool ModList::moveModTo ( int from, int to )
+bool ModList::moveModTo(int from, int to)
{
- if(from < 0 || from >= mods.size())
+ if (from < 0 || from >= mods.size())
return false;
if (to >= rowCount())
to = rowCount() - 1;
if (to == -1)
to = rowCount() - 1;
- if(from == to)
+ if (from == to)
return false;
- int togap = to > from ? to + 1: to;
+ int togap = to > from ? to + 1 : to;
beginMoveRows(QModelIndex(), from, from, QModelIndex(), togap);
mods.move(from, to);
endMoveRows();
@@ -279,83 +277,81 @@ bool ModList::moveModTo ( int from, int to )
return true;
}
-bool ModList::moveModUp ( int from )
+bool ModList::moveModUp(int from)
{
- if(from > 0)
+ if (from > 0)
return moveModTo(from, from - 1);
return false;
}
-bool ModList::moveModsUp ( int first, int last )
+bool ModList::moveModsUp(int first, int last)
{
- if(first == 0)
+ if (first == 0)
return false;
-
+
beginMoveRows(QModelIndex(), first, last, QModelIndex(), first - 1);
- mods.move(first-1, last);
+ mods.move(first - 1, last);
endMoveRows();
saveListFile();
emit changed();
return true;
}
-
-bool ModList::moveModDown ( int from )
+bool ModList::moveModDown(int from)
{
- if(from < 0)
+ if (from < 0)
return false;
- if(from < mods.size() - 1)
+ if (from < mods.size() - 1)
return moveModTo(from, from + 1);
return false;
}
-bool ModList::moveModsDown ( int first, int last )
+bool ModList::moveModsDown(int first, int last)
{
- if(last == mods.size() - 1)
+ if (last == mods.size() - 1)
return false;
-
+
beginMoveRows(QModelIndex(), first, last, QModelIndex(), last + 2);
- mods.move(last+1, first);
+ mods.move(last + 1, first);
endMoveRows();
saveListFile();
emit changed();
return true;
}
-
-int ModList::columnCount ( const QModelIndex& parent ) const
+int ModList::columnCount(const QModelIndex &parent) const
{
return 2;
}
-QVariant ModList::data ( const QModelIndex& index, int role ) const
+QVariant ModList::data(const QModelIndex &index, int role) const
{
- if(!index.isValid())
+ if (!index.isValid())
return QVariant();
-
+
int row = index.row();
int column = index.column();
-
- if(row < 0 || row >= mods.size())
+
+ if (row < 0 || row >= mods.size())
return QVariant();
-
- if(role != Qt::DisplayRole)
+
+ if (role != Qt::DisplayRole)
return QVariant();
-
- switch(column)
+
+ switch (column)
{
- case 0:
- return mods[row].name();
- case 1:
- return mods[row].version();
- case 2:
- return mods[row].mcversion();
- default:
- return QVariant();
+ case 0:
+ return mods[row].name();
+ case 1:
+ return mods[row].version();
+ case 2:
+ return mods[row].mcversion();
+ default:
+ return QVariant();
}
}
-QVariant ModList::headerData ( int section, Qt::Orientation orientation, int role ) const
+QVariant ModList::headerData(int section, Qt::Orientation orientation, int role) const
{
if (role != Qt::DisplayRole || orientation != Qt::Horizontal)
return QVariant();
@@ -370,10 +366,9 @@ QVariant ModList::headerData ( int section, Qt::Orientation orientation, int rol
}
}
-
-Qt::ItemFlags ModList::flags ( const QModelIndex& index ) const
+Qt::ItemFlags ModList::flags(const QModelIndex &index) const
{
- Qt::ItemFlags defaultFlags = QAbstractListModel::flags ( index );
+ Qt::ItemFlags defaultFlags = QAbstractListModel::flags(index);
if (index.isValid())
return Qt::ItemIsDragEnabled | Qt::ItemIsDropEnabled | defaultFlags;
else
@@ -400,36 +395,37 @@ Qt::DropActions ModList::supportedDragActions() const
return Qt::MoveAction;
}
-QMimeData* ModList::mimeData ( const QModelIndexList& indexes ) const
+QMimeData *ModList::mimeData(const QModelIndexList &indexes) const
{
- QMimeData * data = new QMimeData();
-
- if(indexes.size() == 0)
+ QMimeData *data = new QMimeData();
+
+ if (indexes.size() == 0)
return data;
-
+
auto idx = indexes[0];
int row = idx.row();
- if(row <0 || row >= mods.size())
+ if (row < 0 || row >= mods.size())
return data;
-
+
QStringList params;
params << m_list_id << QString::number(row);
data->setText(params.join('|'));
return data;
}
-bool ModList::dropMimeData ( const QMimeData* data, Qt::DropAction action, int row, int column, const QModelIndex& parent )
+bool ModList::dropMimeData(const QMimeData *data, Qt::DropAction action, int row, int column,
+ const QModelIndex &parent)
{
if (action == Qt::IgnoreAction)
- return true;
+ return true;
// check if the action is supported
if (!data || !(action & supportedDropActions()))
return false;
- if(parent.isValid())
+ if (parent.isValid())
{
row = parent.row();
column = parent.column();
}
-
+
if (row > rowCount())
row = rowCount();
if (row == -1)
@@ -437,41 +433,41 @@ bool ModList::dropMimeData ( const QMimeData* data, Qt::DropAction action, int r
if (column == -1)
column = 0;
QLOG_INFO() << "Drop row: " << row << " column: " << column;
-
+
// files dropped from outside?
- if(data->hasUrls())
+ if (data->hasUrls())
{
bool was_watching = is_watching;
- if(was_watching)
+ if (was_watching)
stopWatching();
auto urls = data->urls();
- for(auto url: urls)
+ for (auto url : urls)
{
// only local files may be dropped...
- if(!url.isLocalFile())
+ if (!url.isLocalFile())
continue;
QString filename = url.toLocalFile();
installMod(filename, row);
QLOG_INFO() << "installing: " << filename;
}
- if(was_watching)
+ if (was_watching)
startWatching();
return true;
}
- else if(data->hasText())
+ else if (data->hasText())
{
QString sourcestr = data->text();
auto list = sourcestr.split('|');
- if(list.size() != 2)
+ if (list.size() != 2)
return false;
QString remoteId = list[0];
int remoteIndex = list[1].toInt();
QLOG_INFO() << "move: " << sourcestr;
// no moving of things between two lists
- if(remoteId != m_list_id)
+ if (remoteId != m_list_id)
return false;
// no point moving to the same place...
- if(row == remoteIndex)
+ if (row == remoteIndex)
return false;
// otherwise, move the mod :D
moveModTo(remoteIndex, row);
@@ -479,4 +475,3 @@ bool ModList::dropMimeData ( const QMimeData* data, Qt::DropAction action, int r
}
return false;
}
-
diff --git a/logic/OneSixInstance.cpp b/logic/OneSixInstance.cpp
index a2a6bf17..ad3f9f58 100644
--- a/logic/OneSixInstance.cpp
+++ b/logic/OneSixInstance.cpp
@@ -168,8 +168,7 @@ std::shared_ptr<ModList> OneSixInstance::loaderModList()
{
d->loader_mod_list.reset(new ModList(loaderModsDir()));
}
- else
- d->loader_mod_list->update();
+ d->loader_mod_list->update();
return d->loader_mod_list;
}
@@ -180,8 +179,7 @@ std::shared_ptr<ModList> OneSixInstance::resourcePackList()
{
d->resource_pack_list.reset(new ModList(resourcePacksDir()));
}
- else
- d->resource_pack_list->update();
+ d->resource_pack_list->update();
return d->resource_pack_list;
}