First rework of php formula to bring it up to standards.

php/apc.sls

@@ -1,10 +1,9 @@
+{% from "php/map.jinja" import php with context %}
+
 include:
   - apt
 
-{% if grains['os']=="Ubuntu" %}
-
 php-apc:
-  pkg.installed:
+  pkg:
-    - order: 180
+    - installed
-
+    - name: {{ php.apc-pkg }}
-{% endif %}

php/curl.sls

@@ -1,11 +1,9 @@
+{% from "php/map.jinja" import php with context %}
+
 include:
   - apt
 
-{% if grains['os']=="Ubuntu" %}
-
 php-curl:
-  pkg.installed:
+  pkg
-    - name: php5-curl
+    - installed
-    - order: 180
+    - name: {{ curl-pkg }}
-
-{% endif %}

php/fpm.sls

@@ -1,15 +1,13 @@
+{{% from "php/map.jinja" import php with context %}
+
 include:
   - apt
 
-{% if grains['os']=="Ubuntu" %}
-
 php-fpm:
-  pkg.installed:
+  pkg:
-    - name: php5-fpm
+    - installed
-    - order: 175
+    - name: {{ fpm-pkg }}
-  service.running:
+  service:
-    - name: php5-fpm
+    - running
+    - name: {{ fpm-service }}
     - enable: True
-    - order: 455
-
-{% endif %}

php/gd.sls

@@ -1,11 +1,9 @@
+{{% from "php/map.jinja" import php with context %}
+
 include:
   - apt
 
-{% if grains['os']=="Ubuntu" %}
-
 php-gd:
-  pkg.installed:
+  pkg:
-    - name: php5-gd
+    - installed
-    - order: 180
+    - name: {{ gd-pkg }}
-
-{% endif %}

php/init.sls

@@ -1,11 +1,9 @@
+{% from "php/map.jinja import php with context %}
+
 include:
   - apt
 
-{% if grains['os']=="Ubuntu" %}
-
 php:
-  pkg.installed:
+  pkg:
-    - name: php5
+    - installed
-    - order: 175
+    - name: {{ php-pkg }}
-
-{% endif %}

php/map.jinja

@@ -0,0 +1,24 @@
+{% set map = salt['grains.filter_by']({
+    'Ubuntu': {
+        'php-pkg': 'php5',
+        'apc-pkg': 'php-apc',
+        'curl-pkg': 'php-curl',
+        'fpm-pkg': 'php5-fpm',
+        'gd-pkg': 'php5-gd',
+        'mcrypt-pkg': 'php5-mcrypt',
+        'mysql-pkg': 'php5-mysql',
+        'pear-pkg': ' php-pear',
+        'fpm-service': 'php5-fpm',
+    },
+    'RedHat': {
+        'php-pkg': 'php5',
+        'apc-pkg': 'php-apc',
+        'curl-pkg': 'php-curl',
+        'fpm-pkg': 'php5-fpm',
+        'gd-pkg': 'php5-gd',
+        'mcrypt-pkg': 'php5-mcrypt',
+        'mysql-pkg': 'php5-mysql',
+        'pear-pkg': ' php-pear',
+        'fpm-service': 'php5-fpm',
+    },
+}, merge=salt['pillar.get']('php:lookup')) $}

php/mcrypt.sls

@@ -1,11 +1,9 @@
+{% from "php/map.jinja" import php with context %}
+
 include:
   - apt
 
-{% if grains['os']=="Ubuntu" %}
-
 php-mcrypt:
-  pkg.installed:
+  pkg:
-    - name: php5-mcrypt
+    - installed
-    - order: 180
+    - name: {{ mcrpyt-pkg }}
-
-{% endif %}

php/mysql.sls

@@ -1,11 +1,9 @@
+{% from "php/map.jinja import php with context %}
+
 include:
   - apt
 
-{% if grains['os']=="Ubuntu" %}
-
 php-mysql:
-  pkg.installed:
+  pkg:
-    - name: php5-mysql
+    - installed
-    - order: 180
+    - name: {{ mysql-pkg }}
-
-{% endif %}

php/pear.sls

@@ -1,10 +1,9 @@
+{% from "php/map.jinja" import php with context %}
+
 include:
   - apt
 
-{% if grains['os']=="Ubuntu" %}
-
 php-pear:
-  pkg.installed:
+  pkg:
-    - order: 180
+    - installed
-
+    - name : {{ pear-pkg }}
-{% endif %}