Opening hours: Possibly false positive "can be prettified" with complex value

[skyper]

What steps will reproduce the problem?

1. Have strict mode for opening hours validations plus informal warnings enabled
2. Have an object with opening_hours=Mo 00:00-00:14, Mo-Sa 23:06-23:30; Su,PH 22:45-24:00; PH +1 day Mo-Sa 00:00-00:14,23:06-23:30; PH +1 day Su,PH 00:00-00:14,22:45-24:00
3. Run validator

What is the expected result?

No informal warning

What happens instead?

Informal warning:
Opening hours syntax - opening_hours value can be prettified

Please provide any additional information below. Attach a screenshot if possible.

I can not find anything to prettify and guess this is a false positive but maybe someone else can tell me what validator does not like.

Relative:URL: ^/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2022-02-10 22:03:15 +0100 (Thu, 10 Feb 2022)
Revision:18379
Build-Date:2022-02-11 02:31:13
URL:https://josm.openstreetmap.de/svn/trunk

Identification: JOSM/1.5 (18379 en) Linux Debian GNU/Linux 11 (bullseye)
Java version: 17.0.2+8-Debian-1deb11u1, Debian, OpenJDK 64-Bit Server VM

[gaben]

(The OH validator provided by an external component ​https://github.com/simonpoole/OpeningHoursParser)

[gaben]

It gave me this result, only a single space difference before PH.

-Mo 00:00-00:14, Mo-Sa 23:06-23:30; Su,PH 22:45-24:00; PH +1 day Mo-Sa 00:00-00:14,23:06-23:30; PH +1 day Su,PH 00:00-00:14,22:45-24:00
+Mo 00:00-00:14, Mo-Sa 23:06-23:30; Su,PH 22:45-24:00; PH +1 day Mo-Sa 00:00-00:14,23:06-23:30; PH +1 day Su, PH 00:00-00:14,22:45-24:00

[skyper]

Thanks. The extra space does not make any sense to me and ​https://openingh.openstreetmap.de/evaluation_tool/ tells exactly that.
I guess this is related to #20316.

[gaben]

Adding the OpeningHoursParser maintainer

[skyper]

Well, looking at the displayed times on Easter Monday plus the following Tuesday even ​​https://openingh.openstreetmap.de/evaluation_tool/ does not get the correct times.

[gaben]

Not sure if usernames are case sensitive, this is the "correct" one.