From dd199fbfad80f97b85d055b91952d31274bcb8ba Mon Sep 17 00:00:00 2001 From: Sunil Mohan Adapa Date: Tue, 6 Jun 2023 09:51:20 -0700 Subject: [PATCH] mediawiki: Utilize purging of packages and don't remove explicitly - Purging mediawiki package removes /etc/mediawiki. There is no need to explicitly remove our two configuration files from /etc/mediawiki. Tests: - Setup mediawiki, change public registrations setting and uninstall. Reinstall and observe that public registrations setting is reset. Signed-off-by: Sunil Mohan Adapa Reviewed-by: James Valleroy --- plinth/modules/mediawiki/privileged.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/plinth/modules/mediawiki/privileged.py b/plinth/modules/mediawiki/privileged.py index 93c05b274..d9994ad32 100644 --- a/plinth/modules/mediawiki/privileged.py +++ b/plinth/modules/mediawiki/privileged.py @@ -185,5 +185,4 @@ def set_default_language(language: str): def uninstall(): """Remove Mediawiki's database and local config file.""" shutil.rmtree('/var/lib/mediawiki-db', ignore_errors=True) - pathlib.Path(LOCAL_SETTINGS_CONF).unlink(missing_ok=True) - pathlib.Path(CONF_FILE).unlink(missing_ok=True) + # /etc/mediawiki is removed on purge of mediawiki package