[PR #400/939c2cd6 backport][stable-1] Fix postgresql_set module if single-value param contains comma in value

changelogs/fragments/0-postgresql_set_avoid_wrong_values.yml

@@ -0,0 +1,2 @@
+bugfixes:
+- postgresql_set - avoid wrong values for single-value parameters containing commas (https://github.com/ansible-collections/community.postgresql/pull/400).

plugins/modules/postgresql_set.py

@@ -308,7 +308,7 @@ def param_set(cursor, module, name, value, context):
         if str(value).lower() == 'default':
             query = "ALTER SYSTEM SET %s = DEFAULT" % name
         else:
-            if isinstance(value, str) and ',' in value:
+            if isinstance(value, str) and ',' in value and not name.endswith(('_command', '_prefix')):
                 # Issue https://github.com/ansible-collections/community.postgresql/issues/78
                 # Change value from 'one, two, three' -> "'one','two','three'"
                 value = ','.join(["'" + elem.strip() + "'" for elem in value.split(',')])

tests/integration/targets/postgresql_set/tasks/options_coverage.yml

@@ -30,6 +30,7 @@
         log_statement: mod
         track_functions: none
         shared_preload_libraries: 'pg_stat_statements, pgaudit'
+        log_line_prefix: 'db=%d,user=%u,app=%a,client=%h '
 
   # Check mode:
   - name: Set settings in check mode
@@ -59,3 +60,13 @@
   - assert:
       that:
       - result.stdout == "shared_preload_libraries = 'pg_stat_statements, pgaudit'"
+
+  # Test for single-value params with commas and spaces in value
+  - name: Test single-value param with commas and spaces in value
+    <<: *task_parameters
+    shell: "grep log_line_prefix {{ pg_auto_conf }}"
+    register: result
+
+  - assert:
+      that:
+      - result.stdout == "log_line_prefix = 'db=%d,user=%u,app=%a,client=%h '"