Closed oonqt closed 3 years ago
Hi, thanks for your pull request.
I can't find any URLs with this behaviour, but I have made a safety fix in the backend anyway. Can you let me know if you see examples of this behaviour with the park/URL that is doing it? If you do see this, please reopen this pull request and have a look at fixing the merge errors.
Thanks :)
I’m can’t remember for sure, but it was either DisneylandResortCaliforniaAdventure or DisneylandResortMagicKingdom that resulted in this error.
When using the GetOpeningTimes method, there is a chance that the calendar endpoint will give us back a value of null for certain days, so during each iteration we check that the date actually exists, if not, we skip over it.
The pattern I am seeing that any dates before the current (todays) date are marked as
null
in the calendar endpoint, only future dates are displayed.