Bug 420089

Summary: .kml files cause thumbnail.so to pop up error message windows
Product: [Applications] marble Reporter: Laura David Hurka <laura.stern>
Component: generalAssignee: marble-bugs
Status: REPORTED ---    
Severity: normal CC: kde
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Neon   
OS: Linux   
See Also: https://bugs.kde.org/show_bug.cgi?id=428437
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: Example file with some KML code, which causes thumbnail.so to pop up error message windows.

Description Laura David Hurka 2020-04-14 17:55:36 UTC
Created attachment 127547 [details]
Example file with some KML code, which causes thumbnail.so to pop up error message windows.

SUMMARY
After streamplot-config+xml .spx, I created the next killer file type. :D Keymap logic (.kml) files cause thumbnail.so to pop up error message windows which say "Invalid Document". This can be problematic, because for every file one such window is created, which potentially steals focus.

.kml seams to be associated to “KML geographic data”

STEPS TO REPRODUCE
1. Navigate Dolphin or another KIO application to the attached test.kml

OBSERVED RESULT
Error message window pops up: "Invalid Document"

EXPECTED RESULT
thumbnail.so realizes that it doesn’t understand Keymap Logic Language, and pops up no window.

SOFTWARE/OS VERSIONS
Operating System: KDE neon 5.18
KDE Plasma Version: 5.18.4
KDE Frameworks Version: 5.69.0
Qt Version: 5.14.1
Kernel Version: 5.3.0-46-generic
Comment 1 Kai Uwe Broulik 2020-04-14 18:00:56 UTC
Re-assigning to Marble where the KML thumbnailer comes from
Comment 2 Stefan Brüns 2020-11-03 11:08:16 UTC
Git commit 5e2c369a754a7457f2a746edd057ccc475c962af by Stefan Brüns.
Committed on 02/11/2020 at 17:19.
Pushed by rahn into branch 'master'.

Dont open a MessageBox when a file fails to load from the library

As the filemanager is also used from non-interactive contexts, a message
box is a no-go.

Interactive applications can connect to the fileError signal. This is
done for both marble-qt and marble-kde.
Related: bug 416925

M  +8    -0    src/apps/marble-kde/marble_part.cpp
M  +8    -0    src/apps/marble-qt/QtMainWindow.cpp
M  +0    -6    src/lib/marble/FileManager.cpp

https://invent.kde.org/education/marble/commit/5e2c369a754a7457f2a746edd057ccc475c962af