Merge pull request #397 from BartOpitz/bugfix/with_flattened

fix with_flattened

defaults/main.yml

@@ -43,7 +43,7 @@ php_fpm_pools:
     pool_pm_start_servers: "{{ php_fpm_pm_start_servers }}"
     pool_pm_min_spare_servers: "{{ php_fpm_pm_min_spare_servers }}"
     pool_pm_max_spare_servers: "{{ php_fpm_pm_max_spare_servers }}"
-    pool_php_fpm_pm_max_requests: "{{ php_fpm_pm_max_requests }}"
+    pool_pm_max_requests: "{{ php_fpm_pm_max_requests }}"
 
 # The executable to run when calling PHP from the command line.
 php_executable: "php"

handlers/main.yml

@@ -13,3 +13,4 @@
   when:
     - php_enable_php_fpm
     - php_fpm_state == 'started'
+    - not ansible_check_mode

tasks/configure-fpm.yml

@@ -50,4 +50,7 @@
     name: "{{ php_fpm_daemon }}"
     state: "{{ php_fpm_state }}"
     enabled: "{{ php_fpm_enabled_on_boot }}"
-  when: php_enable_php_fpm and ansible_distribution != "Debian"
+  when:
+    - php_enable_php_fpm
+    - ansible_distribution != "Debian"
+    - not ansible_check_mode

tasks/configure.yml

@@ -1,16 +1,16 @@
 ---
 - name: Ensure configuration directories exist.
-  file:
+  ansible.builtin.file:
     path: "{{ item }}"
     state: directory
     follow: true
     mode: 0755
-  with_flattened:
-    - "{{ php_conf_paths }}"
-    - "{{ php_extension_conf_paths }}"
+  with_items:
+    - "{{ php_conf_paths | flatten }}"
+    - "{{ php_extension_conf_paths | flatten }}"
 
 - name: Place PHP configuration file in place.
-  template:
+  ansible.builtin.template:
     src: php.ini.j2
     dest: "{{ item }}/php.ini"
     owner: root