Add domain to cron time fields

issue10193 review332661010
2021-04-24 19:28:39 +02:00
parent 42cb4a0f53
commit 1ee46331ef
1 changed files with 12 additions and 0 deletions
@@ -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']),