Merge pull request #827 from Som-Energia/FIX_validation_3043

Arreglar la funció search quan l'operador no és ilike

Author: oriolpiera

giscedata_facturacio_som/giscedata_facturacio.py

@@ -34,13 +34,13 @@ def search(self, cr, user, args, offset=0, limit=None, order=None, context=None,
             for idx, arg in enumerate(args):
                 if len(arg) == 3:
                     field, operator, match = arg
-                    if exact_number and field == "number" and isinstance(match, (unicode, str)):  # noqa: E501, F821
-                        if "%" not in match:
-                            operator = "="
+                    if (exact_number and field == "number" and operator == "ilike"
+                            and isinstance(match, (unicode, str)) and "%" not in match):
+                        operator = "="
                         args[idx] = (field, operator, match)
-                    if exact_origin and field == "origin" and isinstance(match, (unicode, str)):  # noqa: E501, F821
-                        if "%" not in match:
-                            operator = "="
+                    if (exact_origin and field == "origin" and operator == "ilike"
+                            and isinstance(match, (unicode, str)) and "%" not in match):
+                        operator = "="
                         args[idx] = (field, operator, match)
         return super(GiscedataFacturacio, self).search(
             cr, user, args, offset, limit, order, context, count

giscedata_facturacio_som/tests/tests_giscedata_facturacio_som.py

@@ -112,3 +112,21 @@ def test_search_exactNotExist_disabled(self):  # noqa: F811
         gff_ids = self.gff_obj.search(self.cursor, self.uid, [("origin", "ilike", "sample_origin")])
 
         self.assertGreater(len(gff_ids), 1)
+
+    def test_search_nonequal_operator_active(self):
+        self.set_invoice_origin_cerca_exacte(self.cursor, self.uid, "1")
+
+        gff_ids = self.gff_obj.search(
+            self.cursor, self.uid, [("origin", "!=", "sample_origin1")]
+        )
+
+        self.assertEqual(len(gff_ids), 3)
+
+    def test_search_equal_operator_active(self):
+        self.set_invoice_origin_cerca_exacte(self.cursor, self.uid, "1")
+
+        gff_ids = self.gff_obj.search(
+            self.cursor, self.uid, [("origin", "=", "sample_origin1")]
+        )
+
+        self.assertEqual(len(gff_ids), 1)