[16.0][FIX] project_key: handle project search safely by id, key, or name

project_key/models/project_project.py

@@ -7,7 +7,26 @@
 
 class Project(models.Model):
     _inherit = "project.project"
-    _rec_names_search = ["key", "name", "id"]
+    _rec_names_search = ["key", "name"]
+
+    @api.model
+    def _name_search(
+        self, name="", args=None, operator="ilike", limit=100, name_get_uid=None
+    ):
+        if name:
+            name = name.strip()
+        res = super()._name_search(name, args, operator, limit, name_get_uid)
+
+        if name and name.isdigit():
+            args = list(args or [])
+            args.insert(0, ("id", "=", int(name)))
+            res_id = super()._name_search("", args, "ilike", limit, name_get_uid)
+            res = list(res)
+            res_ids = set(res)
+            res_id = [x for x in res_id if x not in res_ids]
+            res = res_id + res
+
+        return res
 
     task_key_sequence_id = fields.Many2one(
         comodel_name="ir.sequence", string="Key Sequence", ondelete="restrict"

project_key/tests/test_project.py

@@ -81,3 +81,20 @@ def test_12_onchange_key(self):
 
         project.key = "TEST-1"
         self.assertTrue(project.show_key_warning)
+
+    def test_13_search_by_string_and_id(self):
+        project = self.env["project.project"].create({"name": "Alpha", "key": "ALP"})
+        pid = project.id
+
+        res = self.env["project.project"].name_search("Alpha")
+        ids = [x[0] for x in res]
+        self.assertIn(pid, ids, "Should find project by name 'Alpha'")
+
+        res = self.env["project.project"].name_search(str(pid))
+        ids = [x[0] for x in res]
+        self.assertIn(pid, ids, "Should find project by ID '{}'".format(pid))
+
+        try:
+            self.env["project.project"].name_search("inter")
+        except Exception as e:
+            self.fail("Searching for 'inter' caused an exception: %s" % e)