poetaster / fahrplan

QT Application for Public transportation
GNU General Public License v2.0
1 stars 7 forks source link

Calendar event wrong in some cases (9292.nl related?) #31

Closed dyraig closed 1 year ago

dyraig commented 1 year ago

While using Fahrplan with 9292,nl, I noticed another issue, this time with importing the journey into the agenda. Here's what I've seen:

Case 1, no walking:

Case 2, walking at the end:

Case 3, walking at the start and end:

Case 4 - same as case 3, but different time:

Somehow, when there is a walk at the start of the journey, the event is not created correctly.

poetaster commented 1 year ago

Confirmed. It's related to the way that legs of a journey are represented. The walk parts are ONLY duration. Which requires calculating times (was a pita for a journey) and it seems I've overlooked scanning back to the first leg with date time and subtracting walking AND saving that. hmmm. The time show correctly from the street adress, so it should be fixable.

poetaster commented 1 year ago

ah, I had a ticket related to this: #28

poetaster commented 1 year ago

https://github.com/poetaster/fahrplan/releases/tag/2.0.46

This should be fixed now. Please test as you have before and close this ticket if it's fixed.

Thanks a lot!

dyraig commented 1 year ago

Just tested some familiar scenarios and specifically Case 3/4 from above - seems like you've nailed it! Thank you very much indeed!