After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 789925 - Maps is not listed in the Open With menu for KML files
Maps is not listed in the Open With menu for KML files
Status: RESOLVED FIXED
Product: gnome-maps
Classification: Applications
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: gnome-maps-maint
gnome-maps-maint
Depends on:
Blocks:
 
 
Reported: 2017-11-05 05:57 UTC by Dylan McCall
Modified: 2017-11-08 21:30 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch that adds "%U" to Exec (794 bytes, patch)
2017-11-05 06:01 UTC, Dylan McCall
committed Details | Review
socialPlaceMarker: Compensate for JS string indecies starting at 0 (1.05 KB, patch)
2017-11-08 21:29 UTC, Marcus Lundblad
none Details | Review
socialPlaceMatcher: Compensate for JS string indecies starting at 0 (1.05 KB, patch)
2017-11-08 21:30 UTC, Marcus Lundblad
none Details | Review

Description Dylan McCall 2017-11-05 05:57:49 UTC
When I try to open a KML file, Maps is not listed as an option for this file type.

It looks like this happens because there is no file field for the Exec property in org.gnome.Maps.desktop.
Comment 1 Dylan McCall 2017-11-05 06:01:59 UTC
Created attachment 363003 [details] [review]
Patch that adds "%U" to Exec

Here is a simple patch that resolves this.
Comment 2 Marcus Lundblad 2017-11-05 20:13:03 UTC
Review of attachment 363003 [details] [review]:

LGTM
Comment 3 Marcus Lundblad 2017-11-05 20:13:27 UTC
Thanks for the patch!
Comment 4 Marcus Lundblad 2017-11-05 20:18:41 UTC
Interestingly enough, double-clicking on supported files worked, but Maps didn't appear in the "Open with other" dialog. It looks like other apps does have the %U, so it seems to be fit.
Comment 5 Dylan McCall 2017-11-06 20:32:10 UTC
Glad that solves it! Thanks for the excellent app :) Yeah, I was remembering it seemed to work sometimes as well, but couldn't quite put my finger on it. Maybe an issue with Nautilus or GAppInfo…?
Comment 6 Marcus Lundblad 2017-11-08 21:29:55 UTC
Created attachment 363257 [details] [review]
socialPlaceMarker: Compensate for JS string indecies starting at 0

The pseudo code for the iterative Levenstein implementation on
Wikipedia assumes 1-indexed input strings. To minimize the impact
just offset the indecies in the comparison statement using
values from the strings.
Comment 7 Marcus Lundblad 2017-11-08 21:30:44 UTC
Created attachment 363258 [details] [review]
socialPlaceMatcher: Compensate for JS string indecies starting at 0

The pseudo code for the iterative Levenstein implementation on
Wikipedia assumes 1-indexed input strings. To minimize the impact
just offset the indecies in the comparison statement using
values from the strings.