Modify ↓

Opened 13 months ago

Closed 13 months ago

Last modified 5 months ago

#22074 closed enhancement (fixed)

[Patch] Make opening hours parsing more lenient in strict mode

Reported by: SimonPoole Owned by: team
Priority: normal Milestone: 22.05
Component: Core Version:
Keywords: opening_hours Cc:

Description

See the attached patch.

Attachments (2)

0001-Disable-time-strict-mode.patch​ (1.8 KB) - added by SimonPoole 13 months ago.
22074.patch​ (1.8 KB) - added by taylor.smock 13 months ago.
Add advanced preference setting to enable/disable lenient mode

Download all attachments as: .zip

Change History (8)

Changed 13 months ago by SimonPoole

Attachment: 0001-Disable-time-strict-mode.patch​ added

comment:1 Changed 13 months ago by taylor.smock

Stupid question:
Can you think of any reason to pass true in to OpeningHoursParser#rules(boolean, boolean)?

If so, I'd prefer to have a preference setting that users can override.

Changed 13 months ago by taylor.smock

Attachment: 22074.patch​ added

Add advanced preference setting to enable/disable lenient mode

comment:2 Changed 13 months ago by taylor.smock

Keywords: opening_hours added
Summary: Make opening hours parsing more lienent in strict mode → Make opening hours parsing more lenient in strict mode

comment:3 Changed 13 months ago by anonymous

Summary: Make opening hours parsing more lenient in strict mode → [Patch] Make opening hours parsing more lenient in strict mode

comment:4 Changed 13 months ago by SimonPoole

There is already a preference for strict / non-strict mode overall, I don't see much value in adding a further pref for a sub-variant of the strict mode. In particular, depending on feedback, the option might go away in a future version in any case.

comment:5 Changed 13 months ago by taylor.smock

Resolution: → fixed
Status: new → closed

In 18453/josm:

Fix #22074, #22075: OpeningHoursParser updates (patch by SimonPoole)

Disable time strict mode

This change makes parsing in strict mode slightly more lenient with
respect to time ranges that extend over midnight, with other words this
will no longer complain if non-conflicting long time ranges are used
instead of using the extended time syntax.


Parsing in non-strict mode remains unchanged.

Update OpeningHoursParser to 0.27.0

The update fixes an issue with parsing open ended date ranges followed
by a weekday.

comment:6 Changed 5 months ago by taylor.smock

Milestone: → 22.05

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted.

Add Comment

E-mail address and name can be saved in the Preferences.

Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.