Merge pull request #277 from kindlehl/php_packages_extra_fix

Php_packages_extra fix

defaults/main.yml

@@ -3,6 +3,9 @@
 # for RHEL/CentOS.
 php_enablerepo: ""
 
+# No extra packages are installed by default
+php_packages_extra: []
+
 # Default PHP version to install on Debian-based OSes.
 php_default_version_debian: "7.0"
 

tasks/main.yml

@@ -8,11 +8,6 @@
     php_packages: "{{ __php_packages | list }}"
   when: php_packages is not defined
 
-- name: Define extra php_packages.
-  set_fact:
-    php_packages: "{{ php_packages | list + php_packages_extra | list }}"
-  when: php_packages_extra is defined
-
 - name: Define php_webserver_daemon.
   set_fact:
     php_webserver_daemon: "{{ __php_webserver_daemon }}"

tasks/setup-Debian.yml

@@ -4,7 +4,7 @@
 
 - name: Ensure PHP packages are installed.
   apt:
-    name: "{{ php_packages }}"
+    name: "{{ php_packages + php_packages_extra }}"
     state: "{{ php_packages_state }}"
     install_recommends: "{{ php_install_recommends }}"
   register: php_package_install

tasks/setup-RedHat.yml

@@ -1,7 +1,7 @@
 ---
 - name: Ensure PHP packages are installed.
   package:
-    name: "{{ php_packages }}"
+    name: "{{ php_packages + php_packages_extra }}"
     state: "{{ php_packages_state }}"
     enablerepo: "{{ php_enablerepo | default(omit, true) }}"
   notify: restart webserver