Clear state after time domain push

[molind]

Issue

For string with two or more time intervals e.g. Mo-Sa 09:00-21:00; Su 10:00-20:00 current version returns second interval for all days of week.

After push we need to reset timedomain.

What do you think?

Tasklist

• Add tests
• Add #fixes with the issue number that this PR addresses
• Update the docs with any new request parameters or changes to behavior described
• Update the changelog
• If you made changes to the lua files, update the taginfo too.

Requirements / Relations

Link any requirements here. Other pull requests this PR is based on?

[kevinkreiser]

@gknisely this should be testable with some gurka right?

[molind]

Also, I've noticed that 24/7 isn't parsed at all. Maybe that makes sense, because in Valhalla, this function is used to parse data when the road is closed.

[molind]

DOW for 09:00-21:00 is empty. Is that expected behaviour?

[gknisely]

DOW for 09:00-21:00 is empty. Is that expected behaviour?

Yes. Means the restriction is in place everyday for the time noted above

[gknisely]

Also, I've noticed that 24/7 isn't parsed at all. Maybe that makes sense, because in Valhalla, this function is used to parse data when the road is closed.

Yes. People usually just add access=no

[gknisely]

The old tests are here https://github.com/valhalla/valhalla/blob/master/test/timeparsing.cc but should be able to add a gurka test too