Bug report #17674

Spatial Bookmarks -- Export and Import problems after recent fixes

Added by Steven Mizuno over 6 years ago. Updated about 6 years ago.

Assignee:Alessandro Pasotti
Affected QGIS version:master Regression?:Yes
Operating System: Easy fix?:No
Pull Request or Patch supplied:No Resolution:fixed/implemented
Crashes QGIS or corrupts data:No Copied to github as #:25571


1. Exporting bookmarks writes the srid as "EPSG:4326" or similar rather than the internal integer value. This causes errors on Importing bookmarks -- one per bookmark in the file.

Note that using the internal srid is a likely problem for user defined CRS as the srid value may not exist on another system (or have a different definition) or even with a different profile as QGIS3 has. This could be very messy.

2. On import of an otherwise good file (from older version) if there is an apostrophe in the bookmark name an error occurs inserting the record into tbl_bookmarks. I could cheat by doubling the apostrophe in the XML file -- this would work.
This problem likely has existed for some time, but I just noticed it while testing the recent bookmarks fixes.

3. The list should be refreshed after importing bookmarks. I have had to restart QGIS to see the bookmarks on master, but on 2.18.15 the list was refreshed.

These do affect 2.18 as well.

Associated revisions

Revision 28a61e1e
Added by Alessandro Pasotti over 6 years ago

[bugfix] Bookmarks fix export SRID and refresh on import

Fixes #17674 - Export and Import problems after recent fixes


#1 Updated by Steven Mizuno over 6 years ago

  • Description updated (diff)

#2 Updated by Alessandro Pasotti over 6 years ago

  • Assignee set to Alessandro Pasotti

1. I'll fix that
2. I'm aware of that, but I don't see any other option: saving the proj definitions would solve the issue but it's not suitable for display , I'm not going to fix that for now, feel free to file a feature request
3. I'll have a look

#3 Updated by Anonymous over 6 years ago

  • % Done changed from 0 to 100
  • Status changed from Open to Closed

#4 Updated by Giovanni Manghi about 6 years ago

  • Resolution set to fixed/implemented

Also available in: Atom PDF