[FIX] util.trigger_auto_install

Correctly set the auto_install modules as installed.
Just changing the state is not enough. All its dependencies should be
installed.

As example, in version 16.0, the new `spreadsheet_dashboard_account` module
is defined as
```
{
    'depends': ['spreadsheet_dashboard', 'account'],
    'auto_install': ['account'],
}
```

So, when `account` is installed, this new module should be auto
installed, but also its uninstalled dependencies. In this case, the
module `spreadsheet_dashboard` should also be marked as installed, as
well as its dependencies, recursively.

Part of odoo/upgrade#5343

Signed-off-by: Christophe Simonis (chs) <[email protected]>

Author: KangOl

src/util/modules.py

@@ -592,7 +592,6 @@ def trigger_auto_install(cr, module):
         cat_match = cr.mogrify("m.category_id = %s", [hidden]).decode()
 
     query = """
-        WITH to_install AS (
             SELECT m.id
               FROM ir_module_module_dependency d
               JOIN ir_module_module m ON m.id = d.module_id
@@ -604,17 +603,15 @@ def trigger_auto_install(cr, module):
                AND {}
           GROUP BY m.id
             HAVING bool_and(md.state IN %s)
-        )
-        UPDATE ir_module_module m
-           SET state = 'to install'
-          FROM to_install t
-         WHERE t.id = m.id
     """.format(
         dep_match, cat_match
     )
 
     cr.execute(query, [module, INSTALLED_MODULE_STATES])
-    return bool(cr.rowcount)
+    if cr.rowcount:
+        force_install_module(cr, module)
+        return True
+    return False
 
 
 def _set_module_category(cr, module, category):