Bug 505891 - Specific .pkpass files can't be imported, empty menu item
Summary: Specific .pkpass files can't be imported, empty menu item
Status: RESOLVED FIXED
Alias: None
Product: KDE Itinerary
Classification: Applications
Component: general (other bugs)
Version First Reported In: unspecified
Platform: unspecified All
: NOR normal
Target Milestone: ---
Assignee: Volker Krause
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-06-21 06:29 UTC by Illia
Modified: 2025-06-22 07:24 UTC (History)
0 users

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


Attachments
Emtpy ticket name (49.83 KB, image/png)
2025-06-21 06:29 UTC, Illia
Details
empty menu item (93.60 KB, image/png)
2025-06-21 06:30 UTC, Illia
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Illia 2025-06-21 06:29:43 UTC
Created attachment 182465 [details]
Emtpy ticket name

I just tested it on multiple platforms, .pkpass files with Cyrillic characters cannot be imported correctly at all and remain empty in menu.
As an example, I can send a .pkpass file that cannot be imported, but it contains sensetive information, so it can only be sent via email. Such files are created by tickets.ua and for other applications that can read .pkpass files, they are valid. Screenshots of how it looks on Android, version 25.04.2 (be5fd3f), are attached.
Comment 1 Illia 2025-06-21 06:30:33 UTC
Created attachment 182466 [details]
empty menu item
Comment 2 Volker Krause 2025-06-21 07:06:33 UTC
That indeed would need the specific pkpass to investigate this, sharing by email (vkrause@kde.org) is perfectly fine. Thank you!
Comment 3 Illia 2025-06-21 07:13:23 UTC
Just sent it!
Comment 4 Volker Krause 2025-06-21 08:08:04 UTC
Git commit 5062b49412c3d09663c63a95bb1c234d8162dcbd by Volker Krause.
Committed on 21/06/2025 at 07:56.
Pushed by vkrause into branch 'master'.

Add fallback for empty pkpass description in that pass list view

M  +1    -1    src/app/PassDelegate.qml

https://invent.kde.org/pim/itinerary/-/commit/5062b49412c3d09663c63a95bb1c234d8162dcbd
Comment 5 Volker Krause 2025-06-21 08:08:06 UTC
Git commit 2010fd98692173da074a28741b88ed41a8a7fabb by Volker Krause.
Committed on 21/06/2025 at 08:06.
Pushed by vkrause into branch 'master'.

Work around pkpass type identifier case-sensitivity getting lost in URLs

We have paths that pass this through a QUrl::host, which converts to
lower-case, and are then unable to find the correct pkpass file on disk
again.

This shouldn't happen in the first place, but since it's part of the stored
schema.org data we have to have a fallback either way to recover already
existing data affected by this.

M  +32   -13   src/app/pkpassmanager.cpp

https://invent.kde.org/pim/itinerary/-/commit/2010fd98692173da074a28741b88ed41a8a7fabb
Comment 6 Volker Krause 2025-06-21 08:08:12 UTC
Git commit 4fc26cc3180e2b521c1c63b6327e230d128f2334 by Volker Krause.
Committed on 21/06/2025 at 07:55.
Pushed by vkrause into branch 'master'.

Add a few fallbacks for empty pkpass descriptions when importing

M  +6    -0    src/app/importcontroller.cpp

https://invent.kde.org/pim/itinerary/-/commit/4fc26cc3180e2b521c1c63b6327e230d128f2334
Comment 7 Volker Krause 2025-06-21 09:00:03 UTC
Git commit b00600b6ffb4b5ba05cda0dd9575803ad4a943c3 by Volker Krause.
Committed on 21/06/2025 at 08:46.
Pushed by vkrause into branch 'master'.

Add extractor script for ticket-ua railway passes

M  +2    -0    src/lib/scripts/extractors.qrc
A  +36   -0    src/lib/scripts/tickets-ua.js
A  +13   -0    src/lib/scripts/tickets-ua.json

https://invent.kde.org/pim/kitinerary/-/commit/b00600b6ffb4b5ba05cda0dd9575803ad4a943c3
Comment 8 Volker Krause 2025-06-21 09:08:12 UTC
Git commit acae9b73667d87072196a3ae6ebb90b34fa3ae57 by Volker Krause.
Committed on 21/06/2025 at 09:07.
Pushed by vkrause into branch 'release/25.04'.

Add extractor script for ticket-ua railway passes
(cherry picked from commit b00600b6ffb4b5ba05cda0dd9575803ad4a943c3)

M  +2    -0    src/lib/scripts/extractors.qrc
A  +36   -0    src/lib/scripts/tickets-ua.js
A  +13   -0    src/lib/scripts/tickets-ua.json

https://invent.kde.org/pim/kitinerary/-/commit/acae9b73667d87072196a3ae6ebb90b34fa3ae57
Comment 9 Volker Krause 2025-06-21 09:17:23 UTC
Git commit dae94ee1fb0fffe8b88c5d99aa05bbb1ad68b41c by Volker Krause.
Committed on 21/06/2025 at 09:17.
Pushed by vkrause into branch 'release/25.04'.

Add a few fallbacks for empty pkpass descriptions when importing
(cherry picked from commit 4fc26cc3180e2b521c1c63b6327e230d128f2334)

M  +6    -0    src/app/importcontroller.cpp

https://invent.kde.org/pim/itinerary/-/commit/dae94ee1fb0fffe8b88c5d99aa05bbb1ad68b41c
Comment 10 Volker Krause 2025-06-21 09:17:25 UTC
Git commit 314c2550b90d0c5d0a7a155eea02cd2517913be3 by Volker Krause.
Committed on 21/06/2025 at 09:17.
Pushed by vkrause into branch 'release/25.04'.

Work around pkpass type identifier case-sensitivity getting lost in URLs

We have paths that pass this through a QUrl::host, which converts to
lower-case, and are then unable to find the correct pkpass file on disk
again.

This shouldn't happen in the first place, but since it's part of the stored
schema.org data we have to have a fallback either way to recover already
existing data affected by this.
(cherry picked from commit 2010fd98692173da074a28741b88ed41a8a7fabb)

M  +32   -13   src/app/pkpassmanager.cpp

https://invent.kde.org/pim/itinerary/-/commit/314c2550b90d0c5d0a7a155eea02cd2517913be3
Comment 11 Volker Krause 2025-06-21 09:17:30 UTC
Git commit 9b5f4b5e031808cb2b405ac7fbd16c10005d54b7 by Volker Krause.
Committed on 21/06/2025 at 09:17.
Pushed by vkrause into branch 'release/25.04'.

Add fallback for empty pkpass description in that pass list view
(cherry picked from commit 5062b49412c3d09663c63a95bb1c234d8162dcbd)

M  +1    -1    src/app/PassDelegate.qml

https://invent.kde.org/pim/itinerary/-/commit/9b5f4b5e031808cb2b405ac7fbd16c10005d54b7
Comment 12 Illia 2025-06-22 07:24:52 UTC
I can confirm, that with the latest commit everything works as intended. Thanks!