[Fix] Really fix chained SA dependencies

diff --git a/src/plugins/lua/spamassassin.lua b/src/plugins/lua/spamassassin.lua
index c71fe12eb73a2349e6766cad16364778d9f3a0e9..e6d7639cf94a1ba332af1dc4dbf4c46ba5d41009 100644 (file)
--- a/src/plugins/lua/spamassassin.lua
+++ b/src/plugins/lua/spamassassin.lua
@@ -1524,7 +1524,7 @@ local function post_process()
         local expr_atoms = r['expression']:atoms()
         for _,a in ipairs(expr_atoms) do
           if type(external_deps[a]) == 'table' then
-            for _,dep in ipairs(external_deps[a]) do
+            for dep in pairs(external_deps[a]) do
               if not external_deps[k] then
                 external_deps[k] = {}
               end
@@ -1532,11 +1532,11 @@ local function post_process()
                 rspamd_config:register_dependency(k, dep)
                 external_deps[k][dep] = true
                 rspamd_logger.debugm(N, rspamd_config,
-                  'atom %1 is a direct foreign dependency, ' ..
+                  'atom %1 is an indirect foreign dependency, ' ..
                   'register dependency for %2 on %3',
                   a, k, dep)
+                  nchanges = nchanges + 1
               end
-              nchanges = nchanges + 1
             end
           else
             local rspamd_symbol, replaced_symbol = replace_symbol(a)