Commit e47246e2 authored by Alexander Lohnau's avatar Alexander Lohnau 💬
Browse files

RemoveDeadEntries option, cleanup files and config

BUG: 416255
FIXED-IN: 5.20

Without deleting the file KNS could not tell if we have the entry manually uninstalled.
Also the config entry gets removed once the uninstallation is successful.
parent 5f95108e
......@@ -36,6 +36,7 @@ Name[zh_TW]=登入畫面
ProvidersUrl=https://autoconfig.kde.org/ocs/providers.xml
Categories=SDDM Theme
StandardResource=tmp
RemoveDeadEntries=true
InstallationCommand=sddmthemeinstaller -i %f
UninstallCommand=sddmthemeinstaller -u %f
......@@ -89,7 +89,11 @@ int main(int argc, char **argv)
action.setHelperId(QStringLiteral("org.kde.kcontrol.kcmsddm"));
action.addArgument(QStringLiteral("filePath"), installedTheme);
KAuth::ExecuteJob *job = action.execute();
job->exec();
// We want KNS to be able to tell if the entry was manually deleted, see BUG: 416255
if (job->exec()) {
QFile::remove(args.first());
cg.deleteEntry(args.first());
}
}
return 0;
}
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment