Escape wildcard in tree separator for name domain

issue11321
review356281002

trytond/model/tree.py

@@ -3,6 +3,7 @@
 from itertools import chain
 
 from trytond.i18n import gettext
+from trytond.tools import escape_wildcard
 
 from .modelstorage import ValidationError
 
@@ -22,7 +23,8 @@ def tree(parent='parent', name='name', separator=None):
             def __setup__(cls):
                 super(TreeMixin, cls).__setup__()
                 field = getattr(cls, name)
-                clause = (name, 'not like', '%' + separator + '%')
+                clause = (
+                    name, 'not like', '%' + escape_wildcard(separator) + '%')
                 # If TreeMixin is after the class where name is defined in
                 # __mro__, it modifies the base field copied so it must ensure
                 # to add only once the domain

trytond/tests/test_tree.py

@@ -188,6 +188,15 @@ class TreeMixinTestCase(unittest.TestCase):
         with self.assertRaises(DomainValidationError):
             record.save()
 
+    @with_transaction()
+    def test_name_domain_wildcard(self):
+        "Test name domain on tree with wildcard"
+        pool = Pool()
+        Tree = pool.get('test.tree_wildcard')
+
+        record = Tree(name="test 10%")
+        record.save()
+
     @with_transaction()
     def test_rec_name(self):
         "Test rec_name"

trytond/tests/tree.py

@@ -12,6 +12,13 @@ class Tree(tree(separator=' / '), ModelSQL):
     parent = fields.Many2One('test.tree', "Parent")
 
 
+class TreeWildcard(tree(separator='\\'), ModelSQL):
+    "Tree separator wildcard"
+    __name__ = 'test.tree_wildcard'
+    name = fields.Char("Name")
+    parent = fields.Many2One('test.tree_wildcard', "Parent")
+
+
 class Polytree(tree(parent='parents'), ModelSQL):
     "PolyTree"
     __name__ = 'test.polytree'
@@ -30,6 +37,7 @@ class PolytreeEdge(ModelSQL):
 def register(module):
     Pool.register(
         Tree,
+        TreeWildcard,
         Polytree,
         PolytreeEdge,
         module=module, type_='model')