From 1ee46331ef8bc8cb43a5737c844bc2eb8b30f859 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Sat, 24 Apr 2021 19:28:39 +0200
Subject: [PATCH] Add domain to cron time fields

issue10193
review332661010
---
 trytond/ir/cron.py | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/trytond/ir/cron.py b/trytond/ir/cron.py
index 4261dda450..60aa64adc6 100644
--- a/trytond/ir/cron.py
+++ b/trytond/ir/cron.py
@@ -30,11 +30,19 @@ class Cron(DeactivableMixin, ModelSQL, ModelView):
             ('months', 'Months'),
             ], "Interval Type", sort=False, required=True)
     minute = fields.Integer("Minute",
+        domain=['OR',
+            ('minute', '=', None),
+            [('minute', '>=', 0), ('minute', '<=', 59)],
+            ],
         states={
             'invisible': Eval('interval_type').in_(['minutes']),
             },
         depends=['interval_type'])
     hour = fields.Integer("Hour",
+        domain=['OR',
+            ('hour', '=', None),
+            [('hour', '>=', 0), ('hour', '<=', 23)],
+            ],
         states={
             'invisible': Eval('interval_type').in_(['minutes', 'hours']),
             },
@@ -47,6 +55,10 @@ class Cron(DeactivableMixin, ModelSQL, ModelView):
             },
         depends=['interval_type'])
     day = fields.Integer("Day",
+        domain=['OR',
+            ('day', '=', None),
+            ('day', '>=', 0),
+            ],
         states={
             'invisible': Eval('interval_type').in_(
                 ['minutes', 'hours', 'days', 'weeks']),