Bug 457702 - itinerary does not show full airport name when there are multiple airports
Summary: itinerary does not show full airport name when there are multiple airports
Status: RESOLVED FIXED
Alias: None
Product: KDE Itinerary
Classification: Applications
Component: general (other bugs)
Version First Reported In: unspecified
Platform: Other All
: NOR normal
Target Milestone: ---
Assignee: Volker Krause
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-08-10 03:26 UTC by johnathan
Modified: 2022-08-16 17:03 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
boarding pass (21.65 KB, image/png)
2022-08-10 03:26 UTC, johnathan
Details
android rendering of airport (36.17 KB, image/png)
2022-08-10 03:26 UTC, johnathan
Details

Note You need to log in before you can comment on or make changes to this bug.
Description johnathan 2022-08-10 03:26:14 UTC
Created attachment 151211 [details]
boarding pass

SUMMARY

This one is strange and rather dangerous.
In New Delhi, India there are actually 2 airports. New delhi T2 and T3. They are situated at a distance from one another and flights operate at either airport depending upon if the airline is domestic or international (t2 for domestic and t3 for international)

as attached screenshots, the boarding pass does show clearly T3 but the same is not rendered in the android app at least and i suspect the same is case in all versions?

maybe the airport name is being parsed from list of airports? i am not sure but in this particular case, the critical information is NOT present
Comment 1 johnathan 2022-08-10 03:26:36 UTC
Created attachment 151212 [details]
android rendering of airport
Comment 2 Volker Krause 2022-08-11 15:11:20 UTC
Similar to bug 457684 that is most likely a data extraction issue, it doesn't know the departure terminal. If that is known it's displayed right below the departure airport name.
Comment 3 Volker Krause 2022-08-16 17:03:50 UTC
Git commit 295b576dac8fa1588636012766190500d98607af by Volker Krause.
Committed on 16/08/2022 at 17:03.
Pushed by vkrause into branch 'release/22.08'.

Perform airport terminal detection in the generic boarding pass extractor

This was only done in post-processing so far, in case the terminal
information had been wrongly put into the airport name field. But it
actually works just as well here if we use a vastly less aggressive
end anchor.

M  +1    -0    autotests/extractordata/synthetic/iata-bcbp-demo.pdf.json
M  +15   -0    src/lib/extractors/genericboardingpassextractor.cpp

https://invent.kde.org/pim/kitinerary/commit/295b576dac8fa1588636012766190500d98607af