From 924e6bbf0391459843bfbbdd638d85baa07fedab Mon Sep 17 00:00:00 2001
From: Christoph <ckubu@oopen.de>
Date: Tue, 21 Jul 2026 00:24:25 +0200
Subject: [PATCH] Refactor variable names in copy_files.yml for consistency and
 clarity

---
 roles/common/tasks/copy_files.yml | 21 +++++++++++----------
 1 file changed, 11 insertions(+), 10 deletions(-)

diff --git a/roles/common/tasks/copy_files.yml b/roles/common/tasks/copy_files.yml
index 2ab1127..f85f5d4 100644
--- a/roles/common/tasks/copy_files.yml
+++ b/roles/common/tasks/copy_files.yml
@@ -6,16 +6,16 @@
 - name: Check if file '/etc/postfix/relay_domains' exists
   ansible.builtin.stat:
     path: /etc/postfix/relay_domains
-  register: relay_domains_actual
+  register: common_relay_domains_actual
 
 - name: (copy_files.yml) Get checksum of '/etc/postfix/relay_domains'
   ansible.builtin.set_fact:
-    relay_domains_sha1: "{{ relay_domains_actual.stat.checksum }}"
+    common_relay_domains_sha1: "{{ common_relay_domains_actual.stat.checksum }}"
   when:
-    - relay_domains_actual.stat.exists
+    - common_relay_domains_actual.stat.exists
 
 # ---
-# Copy files - main 
+# Copy files - main
 # ---
 
 - name: (copy_files.yml) Copy plain files
@@ -135,18 +135,19 @@
 - name: Get checksum oif (possible upodated) file '/etc/postfix/relay_domains' exists
   ansible.builtin.stat:
     path: /etc/postfix/relay_domains
-  register: relay_domains_new
+  register: common_relay_domains_new
 
 - name: (copy_files.yml) Get checksum of '/etc/postfix/relay_domains'
   ansible.builtin.set_fact:
-    relay_domains_sha1_new: "{{ relay_domains_new.stat.checksum }}"
+    common_relay_domains_sha1_new: "{{ common_relay_domains_new.stat.checksum }}"
   when:
-    - relay_domains_new.stat.exists
+    - common_relay_domains_new.stat.exists
 
 - name: (copy_files.yml) Renew database /etc/postfix/relay_domains.db
   ansible.builtin.command: "/usr/sbin/postmap btree:/etc/postfix/relay_domains"
+  changed_when: true
   when:
-    - relay_domains_actual.stat.exists
-    - relay_domains_new.stat.exists
-    - relay_domains_actual.stat.checksum != relay_domains_new.stat.checksum
+    - common_relay_domains_actual.stat.exists
+    - common_relay_domains_new.stat.exists
+    - common_relay_domains_actual.stat.checksum != common_relay_domains_new.stat.checksum
   notify: "Reload postfwd"