From 07ee21e4cb699f474c3edd6e4491713603b17b91 Mon Sep 17 00:00:00 2001 From: Phil Nelson Date: Thu, 20 Jul 2017 15:51:06 -0600 Subject: [PATCH] Deploys: --skip-themes when updating WP template_root Some WP themes may throw errors if loaded in this context, e.g., some PHP classes may not be available with `--skip-plugins`. It is not necessary to load themes when updating WP options for template_root and stylesheet root, so it is safer to `--skip-themes`. --- CHANGELOG.md | 1 + roles/deploy/hooks/finalize-after.yml | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 6aec9aec7b..8326f7839e 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,4 +1,5 @@ ### HEAD +* Deploys: `--skip-themes` when updating WP `template_root` ([#849](https://github.com/roots/trellis/pull/849)) * Option to install WP-CLI packages ([#837](https://github.com/roots/trellis/pull/837)) * Update WP-CLI to 1.2.1 ([#838](https://github.com/roots/trellis/pull/838)) * Auto-install Vagrant plugins ([#829](https://github.com/roots/trellis/pull/829)) diff --git a/roles/deploy/hooks/finalize-after.yml b/roles/deploy/hooks/finalize-after.yml index 65b510e0b9..9201ee6aa6 100644 --- a/roles/deploy/hooks/finalize-after.yml +++ b/roles/deploy/hooks/finalize-after.yml @@ -12,7 +12,7 @@ when: project.update_db_on_deploy | default(update_db_on_deploy) - name: Get WP theme template root - command: wp option get template_root --skip-plugins + command: wp option get template_root --skip-plugins --skip-themes args: chdir: "{{ deploy_helper.current_path }}" register: wp_template_root