Clean wordpress/tasks/install.yml

roles/wordpress/tasks/install.yml

@@ -21,19 +21,19 @@
   register: wp_download_output
   when: internet_available | bool
 
-- name: Symlink /opt/iiab/downloads/wordpress.tar.gz -> {{ wp_download_output.dest }}
+- name: Symlink {{ downloads_dir }}/wordpress.tar.gz -> {{ wp_download_output.dest }}
   file:
     src: "{{ wp_download_output.dest }}"
-    path: "{{ downloads_dir }}/wordpress.tar.gz"
+    path: "{{ downloads_dir }}/wordpress.tar.gz"    # /opt/iiab/downloads
     state: link
   when: wp_download_output.dest is defined
 
-- name: Does /opt/iiab/downloads/wordpress.tar.gz link exist?
+- name: Does {{ downloads_dir }}/wordpress.tar.gz link exist?
   stat:
-    path: "{{ downloads_dir }}/wordpress.tar.gz"
+    path: "{{ downloads_dir }}/wordpress.tar.gz"    # /opt/iiab/downloads
   register: wp_link
 
-- name: FAIL (force Ansible to exit) IF /opt/iiab/downloads/wordpress.tar.gz doesn't exist
+- name: FAIL (force Ansible to exit) IF {{ downloads_dir }}/wordpress.tar.gz doesn't exist
   fail:
     msg: "{{ downloads_dir }}/wordpress.tar.gz is REQUIRED in order to install WordPress."
   when: not wp_link.stat.exists
@@ -42,7 +42,7 @@
   unarchive:
     src: "{{ downloads_dir }}/wordpress.tar.gz"
     dest: "{{ wp_install_path }}"
-    owner: root
+    # owner: root
     group: "{{ apache_user }}"    # DO WE REALLY STILL WANT THIS FOR NGINX?
     mode: '0664'
     keep_newer: yes
@@ -54,7 +54,7 @@
   copy:
     src: wp-keys.php.BAK
     dest: "{{ wp_abs_path }}/wp-keys.php.BAK"
-    owner: root    # Not nec? Remove?
+    # owner: root
     group: "{{ apache_user }}"    # DO WE REALLY STILL WANT THIS FOR NGINX?
     mode: '0640'
 
@@ -64,8 +64,8 @@
   template:
     src: get-iiab-wp-salts.j2
     dest: /tmp/get-iiab-wp-salts
-    owner: root    # Not nec? Remove?
-    group: root    # Not nec? Remove?
+    # owner: root
+    # group: root
     mode: '0700'
 
 - name: Run /tmp/get-iiab-wp-salts to create /library/wordpress/wp-keys.php
@@ -80,7 +80,7 @@
   template:
     src: wp-config.php.j2
     dest: "{{ wp_abs_path }}/wp-config.php"
-    owner: root    # Not nec? Remove?
+    # owner: root
     group: "{{ apache_user }}"    # DO WE REALLY STILL WANT THIS FOR NGINX?
     mode: '0660'
 
@@ -90,7 +90,7 @@
     dest: "/etc/{{ apache_config_dir }}/wordpress.conf"
   when: apache_enabled | bool
 
-- name: Add 'wordpress_installed' variable values to {{ iiab_state_file }}
+- name: "Add 'wordpress_installed: True' to {{ iiab_state_file }}"
   lineinfile:
     dest: "{{ iiab_state_file }}"
     regexp: '^wordpress_installed'