There needs to be a limitation to the hours field as of now it's possible to set the time to -5 or 27 it should only be possible to set it from 0 to 24.

CommentFileSizeAuthor
#4 limitation-missing-2812993-3.patch2.2 KBssimdev
#2 limitation-missing-2812993-2.patch2.31 KBssimdev

Comments

STodorov created an issue. See original summary.

ssimdev’s picture

ssimdev commented
Version: 7.x-1.2 » 7.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new limitation-missing-2812993-2.patch2.31 KB

I don't thing that we need limitation by default. But we can add extra option.
Patch to add settings option to limit max value in field settings and validator for HMS field.

Status: Needs review » Needs work

The last submitted patch, 2: limitation-missing-2812993-2.patch, failed testing. View results

ssimdev’s picture

ssimdev commented
Status: Needs work » Needs review
StatusFileSize
new limitation-missing-2812993-3.patch2.2 KB

Quick patch root fix.

  • msajko committed 4678a39 on 7.x-1.x authored by ssimdev 
    Issue #2812993 by ssimdev, STodorov: Limitation missing
msajko’s picture

msajko commented
Status: Needs review » Fixed

Now that we have optional [max] limitation, can we add optional [min] limitation too? If yes, please open new issue

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.