Merge pull request #10108 from saito-hideki/issue/10057

Fix to handle ask_* parameters correctly when set false

awx_collection/plugins/modules/workflow_job_template.py

@@ -748,7 +748,7 @@ def main():
         'webhook_service',
     ):
         field_val = module.params.get(field_name)
-        if field_val:
+        if field_val is not None:
             new_fields[field_name] = field_val
 
     if 'extra_vars' in new_fields:

awx_collection/tests/integration/targets/workflow_job_template/tasks/main.yml

@@ -159,12 +159,30 @@
     extra_vars: {'foo': 'bar', 'another-foo': {'barz': 'bar2'}}
     labels:
       - "{{ lab1 }}"
+    ask_inventory_on_launch: true
+    ask_scm_branch_on_launch: true
+    ask_limit_on_launch: true
+    ask_variables_on_launch: true
   register: result
 
 - assert:
     that:
       - "result is changed"
 
+# Turn off ask_ * settings to test that the issue/10057 has been fixed
+- name: Turn ask_* settings OFF
+  tower_workflow_job_template:
+    name: "{{ wfjt_name }}"
+    ask_inventory_on_launch: false
+    ask_scm_branch_on_launch: false
+    ask_limit_on_launch: false
+    ask_variables_on_launch: false
+    state: present
+
+- assert:
+    that:
+      - "result is changed"
+
 # Node actions do what this schema command used to do
 # schema: [{"success": [{"job_template": "{{ jt1_name }}"}], "job_template": "{{ jt2_name }}"}]
 - name: Create leaf node