Remove useless when conditions and notifies

We don't need restart apache/nginx when a php file gets updated,
they are completely separate from running configuration.

We can also drop the >= zabbix-5.0 condition.

In the rest of the role, we're only calling the respective handler
when we need to, so no more when conditions needed.

roles/zabbix_web/handlers/main.yml

@@ -5,17 +5,13 @@
     state: restarted
     enabled: true
   become: true
-  when:
-    - zabbix_web_http_server == 'apache'
 
 - name: test nginx config
   listen: restart nginx
   ansible.builtin.command: nginx -t
   register: zabbix_nginx_cfg_check
   notify: restart nginx tested
   become: true
-  when:
-    - zabbix_web_http_server == 'nginx'
 
 - name: restart nginx tested
   ansible.builtin.service:
@@ -24,7 +20,6 @@
     enabled: true
   become: true
   when:
-    - zabbix_web_http_server == 'nginx'
     - zabbix_nginx_cfg_check.rc == 0
 
 - name: restart php-fpm-version
@@ -33,8 +28,6 @@
     state: restarted
     enabled: true
   become: true
-  when:
-    - zabbix_web_version is version('5.0', '>=')
 
 - name: "clean repo files from proxy creds"
   ansible.builtin.shell: ls /etc/yum.repos.d/zabbix* && sed -i 's/^proxy =.*//' /etc/yum.repos.d/zabbix* || true

roles/zabbix_web/tasks/main.yml

@@ -110,8 +110,6 @@
     group: "{{ zabbix_web_group }}"
     mode: "{{ zabbix_web_conf_mode }}"
   become: true
-  notify:
-    - "restart {{ zabbix_web_http_server }}"
   tags:
     - config