https://bugs.kde.org/show_bug.cgi?id=516999

--- Comment #1 from Kjetil Kilhavn <[email protected]> ---
I tested a little more after trying to find out where in the source code this
goes wrong, and I see that if I enter the flight number as "SK 4016" instead of
"SK4016", it is also displayed as "SK 4016".

IATA airline codes are always two characters, but the regular expression for
flight number takes the two or three first characters. I think that should be
changed to two letters only. 
https://github.com/KDE/itinerary/blob/4bf4c4f8d4a623918d7419f5412d70893ff07e09/src/app/FlightEditor.qml#L52

I don't understand the code well enough to understand why 3-letter codes are
also extracted as the IATA code. 
Perhaps it is to identify airports, but then it should be refined. 
If the code is three letters long and only contains letters it identifies an
airport. 
If it is three letters long and ends in a digit, it is a flight number, e.g.
BA1. 
If the code is more then three characters long it is a flight number and the
first two characters identify the airline.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to