Attempt to fix ansible_facts deprecation warnings.

Author: geerlingguy

README.md

@@ -35,7 +35,7 @@ The PHP version to be installed. Any [currently-supported PHP major version](htt
 
       roles:
         - name: geerlingguy.repo-remi
-          when: ansible_os_family == 'RedHat'
+          when: ansible_facts.os_family == 'RedHat'
         - geerlingguy.php-versions
         - geerlingguy.php
 

molecule/default/8.2.yml

@@ -10,11 +10,11 @@
   pre_tasks:
     - name: Update apt cache.
       apt: update_cache=true cache_valid_time=600
-      when: ansible_os_family == 'Debian'
+      when: ansible_facts.os_family == 'Debian'
 
   roles:
     - role: geerlingguy.repo-remi
-      when: ansible_os_family == 'RedHat'
+      when: ansible_facts.os_family == 'RedHat'
     - role: geerlingguy.php-versions
     - role: geerlingguy.php
 

molecule/default/converge.yml

@@ -9,11 +9,11 @@
   pre_tasks:
     - name: Update apt cache.
       apt: update_cache=true cache_valid_time=600
-      when: ansible_os_family == 'Debian'
+      when: ansible_facts.os_family == 'Debian'
 
   roles:
     - role: geerlingguy.repo-remi
-      when: ansible_os_family == 'RedHat'
+      when: ansible_facts.os_family == 'RedHat'
     - role: geerlingguy.php-versions
     - role: geerlingguy.php
 

tasks/main.yml

@@ -2,8 +2,8 @@
 - name: Include OS-specific variables.
   include_vars: "{{ item }}"
   with_fileglob:
-    - "{{ role_path }}/vars/{{ ansible_os_family }}.yml"
-    - "{{ role_path }}/vars/{{ ansible_os_family }}-php{{ php_version }}.yml"
+    - "{{ role_path }}/vars/{{ ansible_facts.os_family }}.yml"
+    - "{{ role_path }}/vars/{{ ansible_facts.os_family }}-php{{ php_version }}.yml"
 
 - name: Remove missing JSON extension for PHP 8.0 (included by default)
   set_fact:
@@ -38,4 +38,4 @@
     php_install_recommends: "{{ php_versions_install_recommends }}"
 
 # Setup tasks.
-- include_tasks: "setup-{{ ansible_os_family }}.yml"
+- include_tasks: "setup-{{ ansible_facts.os_family }}.yml"

tasks/setup-Debian.yml

@@ -40,15 +40,15 @@
     # Remove 'Comment' because response can be out of order.
     # See: https://github.com/geerlingguy/ansible-role-php-versions/issues/94
     ondrej_repo_signed_by: "{{ keyserver_response.content | regex_replace('\nComment.+\n','\n') }}"
-  when: ansible_distribution == "Ubuntu"
+  when: ansible_facts.distribution == "Ubuntu"
 
 - debug: var=ondrej_repo_signed_by
 
 - name: Set data for Ondrej's repo (Debian).
   ansible.builtin.set_fact:
     ondrej_repo_uri: "https://packages.sury.org/php"
     ondrej_repo_signed_by: "https://packages.sury.org/php/apt.gpg"
-  when: ansible_distribution == "Debian"
+  when: ansible_facts.distribution == "Debian"
 
 - name: Add Ondrej's PHP repo.
   become: true
@@ -66,7 +66,7 @@
   apt: update_cache=true
   when:
     - php_ondrej_debian_repo.changed
-    - ansible_os_family == "Debian"
+    - ansible_facts.os_family == "Debian"
   tags: ['skip_ansible_lint']
 
 - name: Purge PHP version packages (besides the currently chosen php_version).

tasks/setup-RedHat.yml

@@ -31,5 +31,5 @@
       changed_when: "'Nothing to do' not in dnf_module_enable.stdout"
 
   when:
-    - ansible_os_family == 'RedHat'
-    - ansible_distribution_major_version | int >= 8
+    - ansible_facts.os_family == 'RedHat'
+    - ansible_facts.distribution_major_version | int >= 8