Replace system_cron with disable_wp_cron

Update to match Bedrock's new setting

CHANGELOG.md

@@ -1,4 +1,5 @@
 ### HEAD
+* Update cron variable to match Bedrock ([#394](https://github.com/roots/trellis/pull/394))
 * Use curl instead of dig for IP lookups ([#390](https://github.com/roots/trellis/pull/390))
 * Update SSL cipher suite ([#386](https://github.com/roots/trellis/pull/386))
 * Support for other Vagrant providers (VirtualBox, VMWare, Parallels) ([#340](https://github.com/roots/trellis/pull/340))

README.md

@@ -111,7 +111,6 @@ For example: configure the sites on your Vagrant development VM by editing `grou
 * `site_title` - WP site title (*development* only, default: project name)
 * `db_create` - whether to auto create a database or not (default: `true`)
 * `db_import` - Path to local `sql` dump file which will be imported (optional)
-* `system_cron` - Disable WP cron and use system's (default: `true`)
 * `admin_user` - WP admin user name (*development* only, required)
 * `admin_email` - WP admin email address (*development* only, required)
 * `admin_password` - WP admin user password (*development* only, required)
@@ -123,6 +122,7 @@ For example: configure the sites on your Vagrant development VM by editing `grou
   * `enabled` - Cache enabled flag (required, set to `false`)
   * `duration` - Duration of the cache (default: `30s`)
 * `env` - environment variables
+  * `disable_wp_cron` - Disable WP cron and use system's (default: `true`)
   * `wp_home` - `WP_HOME` constant (required)
   * `wp_siteurl` - `WP_SITEURL` constant (required)
   * `wp_env` - environment (required, matches group name: `development`, `staging`, `production`)

group_vars/development/wordpress_sites.yml

@@ -18,8 +18,8 @@ wordpress_sites:
     cache:
       enabled: false
       duration: 30s
-    system_cron: true
     env:
+      disable_wp_cron: true
       wp_home: http://example.dev
       wp_siteurl: http://example.dev/wp
       wp_env: development

group_vars/production/wordpress_sites.yml

@@ -15,8 +15,8 @@ wordpress_sites:
     cache:
       enabled: false
       duration: 30s
-    system_cron: true
     env:
+      disable_wp_cron: true
       wp_home: http://example.com
       wp_siteurl: http://example.com/wp
       wp_env: production

group_vars/staging/wordpress_sites.yml

@@ -15,8 +15,8 @@ wordpress_sites:
     cache:
       enabled: false
       duration: 30s
-    system_cron: true
     env:
+      disable_wp_cron: true
       wp_home: http://staging.example.com
       wp_siteurl: http://staging.example.com/wp
       wp_env: staging

roles/wordpress-setup/tasks/main.yml

@@ -4,26 +4,29 @@
 - include: nginx.yml
 
 - name: Create web root
-  file: path="{{ www_root }}"
-        owner="{{ web_user }}"
-        group="{{ web_group }}"
-        mode=0755
-        state=directory
+  file:
+    path: "{{ www_root }}"
+    owner: "{{ web_user }}"
+    group: "{{ web_group }}"
+    mode: 0755
+    state: directory
   with_dict: wordpress_sites
 
 - name: Create logs folder of sites
-  file: path="{{ www_root }}/{{ item.key }}/logs"
-        owner="{{ web_user }}"
-        group="{{ web_group }}"
-        mode=0755
-        state=directory
+  file:
+    path: "{{ www_root }}/{{ item.key }}/logs"
+    owner: "{{ web_user }}"
+    group: "{{ web_group }}"
+    mode: 0755
+    state: directory
   with_dict: wordpress_sites
 
-- name: Setup system cron
-  cron: name="{{ item.key }} WordPress cron"
-        minute="*/15"
-        user="{{ web_user }}"
-        job="curl -s {{ item.value.env.wp_siteurl }}/wp-cron.php"
-        cron_file="wordpress-{{ item.key | replace('.', '_') }}"
+- name: Setup WP system cron
+  cron:
+    name: "{{ item.key }} WordPress cron"
+    minute: "*/15"
+    user: "{{ web_user }}"
+    job: "curl -s {{ item.value.env.wp_siteurl }}/wp-cron.php"
+    cron_file: "wordpress-{{ item.key | replace('.', '_') }}"
   with_dict: wordpress_sites
-  when: item.value.system_cron | default(False) == True
+  when: item.value.env.disable_wp_cron | default(false) and not item.value.multisite.enabled | default(false)