[patch] Validator - unusual value of interval

[nlehuby]

The validator raises warning of values of interval tags that seems legit:
for instance interval=120

(As interval=90 is ok, I can't see why 120 won't be, it is also a number of minutes)

And BTW, it does not raise on 01:30:90 which is not ok

[nlehuby]

Here is attached a patch with a new regexp to fix both issues

[skyper]

How to tag an interval of several days? For a weekly interval I used 168:00:00 but that will still raise a warning. I could use only minutes, though, I do not like interval=10080.

It might be better to split the regex into two parts. One for only numbers (minutes) and one for the HH:MM:SS with :.

[nlehuby]

It might be better to split the regex into two parts. One for only numbers (minutes) and one for the HH:MM:SS with :.

I don't understand, it is already the case, the regexp is in two parts with a | between the two expressions

[skyper]

Replying to nlehuby:

It might be better to split the regex into two parts. One for only numbers (minutes) and one for the HH:MM:SS with :.

I don't understand, it is already the case, the regexp is in two parts with a | between the two expressions

I am sorry, I misread the patch.

[nlehuby]

Replying to skyper:

How to tag an interval of several days? For a weekly interval I used 168:00:00 but that will still raise a warning. I could use only minutes, though, I do not like interval=10080.

For public transport interval at least, interval tag should be used in addition with opening_hours tag, and it could be better suited for weekly interval
For instance, for a municipal shuttle bus to the city market on Wednesday morning, I would use instead opening_hours = We 10:00-12:00 and no interval tag

[skyper]

Sorry, my example was bad.
Especially, with ferries I knew some that sailed only every three, four or five days. This cannot be expressed in opening hours. Sorry, I do not find the timetables atm.

[nlehuby]

new patch

[nlehuby]

Hello,

Does this new patch sound good to you ?

[skyper]

Yes, thanks.
No translation updated needed.

[stoecker]

In 18384/josm:

fix #21853 - patch by nlehuby - improve interval validator test