[stable-4] Fix pylint errors (#5939)

Fix pylint errors (#5933)

* Fix pylint errors.

* Also adjust to https://github.com/ansible/ansible/pull/79909.

(cherry picked from commit b1d9507cd2)

changelogs/fragments/5933-linting.yml

@@ -0,0 +1,2 @@
+bugfixes:
+  - "terraform and timezone - slight refactoring to avoid linter reporting potentially undefined variables (https://github.com/ansible-collections/community.general/pull/5933)."

plugins/modules/cloud/misc/terraform.py

@@ -508,9 +508,9 @@ def main():
 
     outputs_command = [command[0], 'output', '-no-color', '-json'] + _state_args(state_file)
     rc, outputs_text, outputs_err = module.run_command(outputs_command, cwd=project_path)
+    outputs = {}
     if rc == 1:
         module.warn("Could not get Terraform outputs. This usually means none have been defined.\nstdout: {0}\nstderr: {1}".format(outputs_text, outputs_err))
-        outputs = {}
     elif rc != 0:
         module.fail_json(
             msg="Failure when getting Terraform outputs. "

plugins/modules/net_tools/pritunl/pritunl_org.py

@@ -180,7 +180,7 @@ def main():
                 required=False, choices=["present", "absent"], default="present"
             ),
         )
-    ),
+    )
 
     module = AnsibleModule(argument_spec=argument_spec)
 

plugins/modules/net_tools/pritunl/pritunl_org_info.py

@@ -115,7 +115,7 @@ def main():
         dict(
             organization=dict(required=False, type="str", default=None, aliases=["org"])
         )
-    ),
+    )
 
     module = AnsibleModule(argument_spec=argument_spec, supports_check_mode=True)
 

plugins/modules/net_tools/pritunl/pritunl_user.py

@@ -324,7 +324,7 @@ def main():
             user_disabled=dict(required=False, type="bool", default=None),
             user_gravatar=dict(required=False, type="bool", default=None),
         )
-    ),
+    )
 
     module = AnsibleModule(argument_spec=argument_spec)
 

plugins/modules/net_tools/pritunl/pritunl_user_info.py

@@ -157,7 +157,7 @@ def main():
                 default="client",
             ),
         )
-    ),
+    )
 
     module = AnsibleModule(argument_spec=argument_spec, supports_check_mode=True)
 

plugins/modules/system/timezone.py

@@ -355,6 +355,7 @@ class NosystemdTimezone(Timezone):
     def __init__(self, module):
         super(NosystemdTimezone, self).__init__(module)
         # Validate given timezone
+        planned_tz = ''
         if 'name' in self.value:
             tzfile = self._verify_timezone()
             planned_tz = self.value['name']['planned']

plugins/modules/system/xfconf.py

@@ -172,8 +172,8 @@ class XFConfException(Exception):
 
 
 class XFConfProperty(CmdStateModuleHelper):
-    change_params = 'value',
+    change_params = ('value', )
-    diff_params = 'value',
+    diff_params = ('value', )
     output_params = ('property', 'channel', 'value')
     facts_params = ('property', 'channel', 'value')
     module = dict(