Bug 184217

Summary: No dialog is shown to report error loading KML file
Product: [Applications] marble Reporter: Miguel Frutos <m.frutos>
Component: generalAssignee: marble-bugs
Status: RESOLVED FIXED    
Severity: normal CC: acb, marble-bugs, nienhueser, ps_ml
Priority: NOR    
Version: unspecified   
Target Milestone: 1.0 (KDE 4.6)   
Platform: Ubuntu   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Miguel Frutos 2009-02-13 12:29:08 UTC 
Version:            (using KDE 4.1.4)
OS:                Linux
Installed from:    Ubuntu Packages

No dialog is shown to report an error loading a KML file (this may affect other file types, but I have not tested it).

The application tries to open the file, but if it fails, nothing happens. The only way is to run from the shell and see the logs.

To reproduce: create an empty file.kml and try to open it. You will get the following messages on the shell but nothing on the gui:
[GeoParser::read] -> Error occurred: "Start tag expected." 
Could not parse file!
Comment 1 Patrick Spendrin 2009-07-10 02:53:53 UTC 
valid bug report, I will try to make something up for that.
Comment 2 Andrew Manson 2009-07-21 11:10:13 UTC 
This is a confirmed bug and it does indeed affect all file types, including any file type that is parsed by handlers shipped with a plugin ( eg. OSM )
Comment 3 Thibaut Gridel 2011-09-22 21:16:33 UTC 
Git commit 5930b2c3d7a608bb909a4fe9bbcba64230890dfe by Thibaut Gridel.
Committed on 22/09/2011 at 23:14.
Pushed by tgridel into branch 'master'.

FileManager: report file parsing error

BUG: 184217

M  +20   -13   src/lib/FileLoader.cpp
M  +2    -1    src/lib/FileLoader.h
M  +9    -0    src/lib/FileManager.cpp
M  +1    -1    src/lib/MarbleAbstractRunner.cpp
M  +2    -2    src/lib/MarbleAbstractRunner.h
M  +5    -5    src/lib/MarbleRunnerManager.cpp
M  +3    -3    src/lib/MarbleRunnerManager.h
M  +0    -13   src/lib/geodata/parser/GeoDataParser.cpp
M  +0    -1    src/lib/geodata/parser/GeoDataParser.h
M  +6    -10   src/lib/geodata/parser/GeoParser.cpp
M  +0    -2    src/lib/geodata/parser/GeoParser.h
M  +0    -11   src/lib/geodata/parser/GeoSceneParser.cpp
M  +0    -1    src/lib/geodata/parser/GeoSceneParser.h
M  +1    -2    src/plugins/runner/cache/CacheRunner.cpp
M  +0    -5    src/plugins/runner/gpx/GpxParser.cpp
M  +0    -1    src/plugins/runner/gpx/GpxParser.h
M  +1    -1    src/plugins/runner/gpx/GpxRunner.cpp
M  +0    -5    src/plugins/runner/osm/OsmParser.cpp
M  +0    -1    src/plugins/runner/osm/OsmParser.h
M  +1    -1    src/plugins/runner/osm/OsmRunner.cpp

http://commits.kde.org/marble/5930b2c3d7a608bb909a4fe9bbcba64230890dfe