diff --git a/changelogs/fragments/2185-xfconf-absent-check-mode.yml b/changelogs/fragments/2185-xfconf-absent-check-mode.yml new file mode 100644 index 0000000000..059f4acd9a --- /dev/null +++ b/changelogs/fragments/2185-xfconf-absent-check-mode.yml @@ -0,0 +1,2 @@ +bugfixes: + - xfconf - module was not honoring check mode when ``state`` was ``absent`` (https://github.com/ansible-collections/community.general/pull/2185). diff --git a/plugins/modules/system/xfconf.py b/plugins/modules/system/xfconf.py index 8d0700ae11..5dc87f539c 100644 --- a/plugins/modules/system/xfconf.py +++ b/plugins/modules/system/xfconf.py @@ -219,8 +219,9 @@ class XFConfProperty(CmdMixin, StateMixin, ModuleHelper): self.update_xfconf_output(value=self.vars.value) def state_absent(self): + if not self.module.check_mode: + self.run_command(params=('channel', 'property', 'reset'), extra_params={"reset": True}) self.vars.value = None - self.run_command(params=('channel', 'property', 'reset'), extra_params={"reset": True}) self.update_xfconf_output(previous_value=self.vars.previous_value, value=None)