[IMP] count in response & add id in domain if filled

controllers/endpoint.py

@@ -51,6 +51,7 @@ class RouteApi(http.Controller):
             ("model_id.model", "=", model_name),
             ("route_name", "=", exposition_name),
         ]
+
         exposition_record = request.env["route.exposition"].sudo().search(domain)
 
         if not exposition_record:
@@ -76,6 +77,9 @@ class RouteApi(http.Controller):
 
         try:
             exposition_domain = exposition_record.get_domain()
+            if id:
+                exposition_domain.append(("id", "=", id))
+
             field_exposed = (
                 request.env["route.exposition.field"]
                 .sudo()
@@ -107,7 +111,7 @@ class RouteApi(http.Controller):
             )
 
         response = http.Response(
-            response=json.dumps(result),
+            response=json.dumps({"count": len(result), "data": result}),
             headers=[("Content-Type", "application/json")],
         )
         return response

models/route_exposition.py

@@ -19,6 +19,9 @@ class RouteExposition(models.Model):
         "for a given odoo model"
     )
 
+    # ------------------------------------------------------
+    # Fields declaration
+    # ------------------------------------------------------
     name = fields.Char(string="Nom", required=True)
     description = fields.Text()
     model_id = fields.Many2one(comodel_name="ir.model", string="Modèle")
@@ -46,6 +49,9 @@ class RouteExposition(models.Model):
         )
     ]
 
+    # ------------------------------------------------------
+    # Onchange
+    # ------------------------------------------------------
     @api.onchange("model_id")
     def _onchange_model_id(self):
         self.field_ids = None
@@ -61,6 +67,9 @@ class RouteExposition(models.Model):
         except SyntaxError as e:
             raise ValidationError(f"Uhoh, the domain seems wrong: {e}") from SyntaxError
 
+    # ------------------------------------------------------
+    # Compute
+    # ------------------------------------------------------
     @api.depends("name")
     def _compute_route_name(self):
         for exposition in self:
@@ -78,6 +87,18 @@ class RouteExposition(models.Model):
             else:
                 exposition.route_name = ""
 
+    # ------------------------------------------------------
+    # Actions
+    # ------------------------------------------------------
+    def refresh_api_key(self):
+        self.api_key = self.generate_key()
+
+    def delete_api_key(self):
+        self.api_key = False
+
+    # ------------------------------------------------------
+    # Business Method
+    # ------------------------------------------------------
     def get_domain(self):
         """
         Evaluate the domain of an exposition, meaning it transforms the string domain to
@@ -89,9 +110,6 @@ class RouteExposition(models.Model):
         self.ensure_one()
         return safe_eval(self.domain) if self.domain else []
 
-    def refresh_api_key(self):
-        self.api_key = self.generate_key()
-
     def generate_key(self) -> str:
         """
         Generate an API key based on
@@ -107,6 +125,3 @@ class RouteExposition(models.Model):
         h.update(encoded_data)
         hashed = h.hexdigest()
         return hashed
-
-    def delete_api_key(self):
-        self.api_key = ""