diff --git a/logic/minecraft/MinecraftProfile.cpp b/logic/minecraft/MinecraftProfile.cpp index e9ff87d89..e87ece006 100644 --- a/logic/minecraft/MinecraftProfile.cpp +++ b/logic/minecraft/MinecraftProfile.cpp @@ -99,10 +99,14 @@ bool MinecraftProfile::canRemove(const int index) const bool MinecraftProfile::remove(const int index) { if (!canRemove(index)) + { + qDebug() << "Patch" << index << "is non-removable"; return false; + } if(!m_strategy->removePatch(VersionPatches.at(index))) { + qCritical() << "Patch" << index << "could not be removed"; return false; } diff --git a/logic/minecraft/OneSixProfileStrategy.cpp b/logic/minecraft/OneSixProfileStrategy.cpp index 4f596d12b..12503a80d 100644 --- a/logic/minecraft/OneSixProfileStrategy.cpp +++ b/logic/minecraft/OneSixProfileStrategy.cpp @@ -186,6 +186,15 @@ bool OneSixProfileStrategy::removePatch(ProfilePatchPtr patch) bool ok = true; // first, remove the patch file. this ensures it's not used anymore auto fileName = patch->getPatchFilename(); + if(fileName.size()) + { + QFile patchFile(fileName); + if(patchFile.exists() && !patchFile.remove()) + { + qCritical() << "File" << fileName << "could not be removed because:" << patchFile.errorString(); + return false; + } + } auto preRemoveJarMod = [&](JarmodPtr jarMod) -> bool @@ -194,7 +203,13 @@ bool OneSixProfileStrategy::removePatch(ProfilePatchPtr patch) QFileInfo finfo (fullpath); if(finfo.exists()) { - return QFile::remove(fullpath); + QFile jarModFile(fullpath); + if(!jarModFile.remove()) + { + qCritical() << "File" << fullpath << "could not be removed because:" << jarModFile.errorString(); + return false; + } + return true; } return true; };