Needs work
Project:
GMap Module
Version:
7.x-2.12
Component:
Code
Priority:
Major
Category:
Feature request
Assigned:
Issue tags:
Reporter:
Created:
24 Dec 2009 at 11:33 UTC
Updated:
16 Feb 2023 at 20:41 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
steven jones commentedOf course we need to update the titles function too...
Comment #2
steven jones commentedAh nuts, the marker dir is hardcoded in there, the code is not smart at all!
Comment #3
steven jones commentedAh okay, if we store everything relative to the drupal root, then we're laughing!
Comment #4
manarth commentedIt would also be useful for THEMES to be able to provide custom markers (and/or override the presentation of the markers/content defined by modules).
Comment #5
podarokall feature requests can be imported in 6.x version after committing em in major versions with backport afterwards
Can You make this ?
Comment #6
anpolimusComment #7
anpolimusComment #8
anpolimusComment #9
anpolimusComment #10
anpolimusComment #11
anpolimusComment #12
anpolimusComment #13
podarokbot up
Comment #18
valentine94Looks good to me, +1 to RTBC
Comment #19
podarokLet's make default variable value looking at system tmp path
Comment #20
paravibe commentedLooks good.
Comment #21
paravibe commentedOops, didn't see previous comment.
@anpolimus will you do this?
Comment #22
anpolimusSure, will do.
Comment #23
anpolimusI have deleted this variable at all, because new custom dir and markerfiles could be now added with hook, defined in this task.
@podarok, @paravibe please approve that this is suitable resolution in this case.
Comment #24
anpolimusComment #25
anpolimusComment #26
podarokThanks @anpolimus, #23 committed.
Comment #29
Ives commentedVariable gmap_marker_custom_dir is removed, but still available to set in settings form (gmap_settings_ui.inc).
How can we now define a custom marker folder without using the hook?
Comment #30
jsst commentedThis issue is not properly fixed and is breaking existing installs. I believe the following should be done:
- make the change backwards compatible so we don't break things between minor version changes (that's important!)
- and when we remove gmap_marker_custom_dir, clean up the settings as mentioned in #29
- document how the new hook works, users now have to reverse-engineer gmap_gmap_markerfiles_info()
Regarding the backwards compatibility, maybe gmap could not delete gmap_marker_custom_dir and implement a default markerfiles_info hook that reads the variable and uses that directory if it's set.
Comment #31
vkechagias commentedSince this is breaking functionality of the custom markers configured through the gmap settings this should be escalated to Major Priority.
On https://www.drupal.org/node/2692971 there seems to be a workaround to make custom markers work again after updating to 7.x-2.11 however this should be fixed properly.
Comment #32
markabur commentedI tried using this feature to add a new marker set using a custom module. I used a debugger to verify that the custom marker .ini is being read and that the icons are available in $icons at the end of _gmap_get_icondata(). However, the custom custom marker set is not showing up in Views. I'm looking at the "Marker / fallback marker to use" popup and don't see my marker set listed there.
Here is my .ini file. It is called "mynumbers.ini":
I've also tried leaving it as numbers.ini and naming the icons like number1.png, etc. but my custom icons don't override the default Numbers icon set (using Views to generate the map).