[Panel] Date field min/max settings not validated

[texnixe]

Describe the bug
When setting up a date field with min and max values the settings are not validated.

published:
  type: date
  required: true
  label: Published On
  min: 2019-01-01
  max: 2019-05-01

While the dropdown limits the years to choose from (only if time: true is not set), this is not true for the months and days. Also, you can still pick any date using the date picker. What is more, you cannot only pick a date outside this range, you can also save this date without validation kicking in.

To Reproduce
Steps to reproduce the behavior:

1. Put the above field into a blueprint
2. Open/create a page with this blueprint in the Panel
3. Use the date picker to choose a date outside the given range.
4. See the page saving without a complaint.

Expected behavior
Dates outside the range should not be pickable nor should they validate.

Screenshots

Kirby Version
3.1.4 and feature branch

Desktop (please complete the following information):

• macOS
• Safari

[GiantCrocodile]

This is the original thread about the issue I face: https://forum.getkirby.com/t/how-to-use-min-max-for-date-fields/14376

Thanks for reporting this @texnixe!

[texnixe]

If I set time: true, the dropdown doesn't even limit the years.

[bastianallgeier]

✅

[GiantCrocodile]

I downloaded v3.2.1-rc1 and used the above example and my origonal example. I'm still able to select invalid dates and to save them. Am I doing something wrong @bastianallgeier?

  published:
    type: date
    time: true
        - step: 1
    required: true
    default: now
    label: Published On
    min: 2019-01-01
    max: 2019-05-01

I tried US and German date format for min/max property.

[texnixe]

Hm, I can still enter wrong dates, but at least validation does now kick in when I select a wrong date.

[GiantCrocodile]

I get no validation at all @texnixe. Maybe it is related to the default option. Try this field:

  published:
    type: date
    time: true
        - step: 1
    required: true
    default: now
    label: Published On
    min: 2019-01-01
    max: 2019-05-01

[texnixe]

Hm, even with that field, it works for me.

Is your page a draft? Drafts are not validated. What happens, if you want to publish it?

The thing is that it probably wouldn't make sense or wouldn't even be possible to limit the days and months shown in a way that would completely make it impossible to select a wrong date - depending on what date ranges you have selected. Let's say you have a date range from 01.01.2015 to 25.06.2017, in that case, you would have to show all days and months. Still, selecting a wrong date like 30.08.2017 would be possible, so validating the date is the only way.

To help the user choosing a correct data, some help text under the field would be useful though.

[GiantCrocodile]

True, I forgot that validation happens on publishing level, not on draft level, although it happens with required fields earlier (red border around non-filled field).

You are right about showing all days and months in specific situations. But in some cases we could limit it or at least limit the javascript datepicker. I think that would be a cheap but good improvement.