Bug 184217 - No dialog is shown to report error loading KML file
Summary: No dialog is shown to report error loading KML file
Status: RESOLVED FIXED
Alias: None
Product: marble
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Ubuntu Linux
: NOR normal
Target Milestone: 1.0 (KDE 4.6)
Assignee: marble-bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-02-13 12:29 UTC by Miguel Frutos
Modified: 2011-09-22 21:16 UTC (History)
4 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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