Replace _constraints with validate in ModelStorage and improve user messages.

issue2616
review371002

work.py

@@ -73,13 +73,11 @@ class Work:
     @classmethod
     def __setup__(cls):
         super(Work, cls).__setup__()
-        cls._constraints += [
-            ('check_recursion', 'recursive_dependency'),
-            ]
-        cls._error_messages.update({
-                'recursive_dependency': 'You can not create recursive '
-                        'dependencies!',
-                })
+
+    @classmethod
+    def validate(cls, works):
+        super(Work, cls).validate(works)
+        cls.check_recursion(works)
 
     @classmethod
     def check_recursion(cls, records, parent='successors'):

work.xml

@@ -64,5 +64,14 @@ this repository contains the full copyright notices and license terms. -->
             <field name="model">project.work</field>
         </record>
 
+        <record model="ir.translation" id="project_work_recursion_error">
+            <field name="name">project.work</field>
+            <field name="lang">en_US</field>
+            <field name="type">error</field>
+            <field name="src">recursion_error</field>
+            <field name="value">Recursion error: Project Work "%(rec_name)s" with parent "%(parent_rec_name)s" was configured as ancestor of itself.</field>
+            <field name="module">project_plan</field>
+            <field name="fuzzy" eval="False"/>
+        </record>
     </data>
 </tryton>