0xf8/pollymc
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: don't try to delete mods multiple times

Browse Source 
Shows a more helpful message if there's a parsing error when reading the
index file.

Also fixes a clazy warning with using the `.data()` method in a
temporary QByteArray object.
This commit is contained in:
flow
2022-04-21 15:47:46 -03:00
committed by flow
parent e17b6804a7
commit 67e0214fa5
2 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
launcher/minecraft/mod/ModFolderModel.cpp
View File

@@ -339,6 +339,9 @@ bool ModFolderModel::deleteMods(const QModelIndexList& indexes)
for (auto i: indexes)
{
if(i.column() != 0) {
continue;
}
Mod &m = mods[i.row()];
auto index_dir = indexDir();
m.destroy(index_dir);

6
launcher/modplatform/packwiz/Packwiz.cpp
View File

@@ -162,12 +162,14 @@ auto V1::getIndexForMod(QDir& index_dir, QString& index_file_name) -> Mod
// NOLINTNEXTLINE(modernize-avoid-c-arrays)
char errbuf[200];
table = toml_parse(index_file.readAll().data(), errbuf, sizeof(errbuf));
auto file_bytearray = index_file.readAll();
table = toml_parse(file_bytearray.data(), errbuf, sizeof(errbuf));
index_file.close();
if (!table) {
qCritical() << QString("Could not open file %1!").arg(indexFileName(mod.name));
qWarning() << QString("Could not open file %1!").arg(indexFileName(index_file_name));
qWarning() << "Reason: " << QString(errbuf);
return {};
}