[IMP] vcp_management: Pass some parameters to speed things a little

Author: etobella

vcp_management/models/vcp_repository_branch.py

@@ -65,9 +65,12 @@ def _get_local_path(self):
     def process_rules(self):
         for record in self:
             rules = record._get_rules()
+            # This parameters dict can be used to store parameters that will
+            # be used by other rules.
+            parameters = {}
             for rule in rules:
                 if re.match(rule.branch_pattern, record.branch_id.name):
-                    rule._process_rule(record)
+                    rule._process_rule(record, parameters)
 
     def _download_code(self):
         result = super()._download_code()

vcp_management/models/vcp_rule.py

@@ -6,7 +6,7 @@
 
 import pathspec
 
-from odoo import fields, models
+from odoo import fields, models, tools
 from odoo.fields import Command
 
 _logger = logging.getLogger(__name__)
@@ -32,19 +32,27 @@ class VcpRule(models.Model):
         default="cloc",
     )
 
-    def _process_rule(self, record):
+    def _process_rule(self, record, parameters=None):
         """
         Process the rule on the given repository and branch.
         It will call the corresponding method based on the rule type.
         """
-        return getattr(self, f"_process_rule_{self.rule_type}")(record)
+        return getattr(self, f"_process_rule_{self.rule_type}")(record, parameters)
 
-    def _process_rule_cloc(self, record):
+    def _process_rule_cloc(self, record, parameters=None):
         """
         Process the rule as a cloc analysis.
         """
         record._download_code()
-        cloc_response = self._call_cloc_command(record.local_path)
+        if parameters is None:
+            parameters = {}
+        if "cloc" in parameters:
+            cloc_response = parameters["cloc"]
+        else:
+            cloc_response = self.browse()._call_cloc_command(
+                record.id, record.local_path
+            )
+            parameters["cloc"] = cloc_response
         matches = self._cloc_get_matches(record.local_path)
         cloc_data = self._action_analysis_process_cloc(
             record.local_path, matches, cloc_response
@@ -94,7 +102,8 @@ def _action_analysis_process_cloc(self, path, matchs, cloc_response):
                 res["empty"] += res_file["blank"]
         return res
 
-    def _call_cloc_command(self, local_path):
+    @tools.ormcache("record_id", "local_path")
+    def _call_cloc_command(self, record_id, local_path):
         res = check_output(["cloc", "--by-file", "--json", local_path])
         return json.loads(res)