From a86daffba0bd6b41ab404d0e3eb94830299f930b Mon Sep 17 00:00:00 2001
From: Silejonu <48565914+Silejonu@users.noreply.github.com>
Date: Wed, 7 May 2025 16:03:42 +0200
Subject: [PATCH] fix(acl): correctly assert needed changes when recursive is
 true

---
 plugins/modules/acl.py | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/plugins/modules/acl.py b/plugins/modules/acl.py
index 4cc94e7..3e9549c 100644
--- a/plugins/modules/acl.py
+++ b/plugins/modules/acl.py
@@ -243,13 +243,19 @@ def acl_changed(module, cmd, entry, use_nfsv4_acls=False):
     cmd.insert(1, '--test')
     lines = run_acl(module, cmd)
     counter = 0
+    lines_checked = 0
+    lines_unchanged = 0
     for line in lines:
+        lines_checked += 1
         if line.endswith('*,*') and not use_nfsv4_acls:
-            return False
+            lines_unchanged += 1
         # if use_nfsv4_acls and entry is listed
         if use_nfsv4_acls and entry == line:
             counter += 1
 
+    if lines_unchanged == lines_checked:
+        return False
+
     # The current 'nfs4_setfacl --test' lists a new entry,
     #  which will be added at the top of list, followed by the existing entries.
     # So if the entry has already been registered, the entry should be find twice.