Hi,
First of all, thanks for really great module!
I recently noticed that GMap module generate some error notice in one particular case.
I've got Better Exposed Filters that filter my GMap by province. After filtering everything shows up, but I've got this error:
Notice: Undefined index: style_plugin w gmap_views_ajax_data_alter() (linia 1378 z /sites/all/modules/gmap/gmap.module).
I don't have a clue what's the problem. Could you help?
| Comment | File | Size | Author |
|---|---|---|---|
| #12 | gmap.code_.1890460-12.patch | 607 bytes | helmo |
| #10 | gmap_1890460_10_plugin_style_error.patch | 866 bytes | johnv |
| #2 | gmap-undefined-style-plugin-1890460-1782709.patch | 721 bytes | pixelula |
Comments
Comment #1
grozozo commentedThx for this great module!
I get the same message since I added a contextual filters for current node ID in my Gmap view.
Everything is working but this message keeps showing up, 2 times per node from the only content type that contain locations and use gmap.
Comment #2
pixelula commentedHi, this patch work for me. I just added a index validation.
Comment #4
mist commentedWorks for me thx :-)
Btw.
in my opinion should look like
if (! (in_array ('gmap', $plugin_styles))) retrun ;
Comment #5
podaroklooks like test bot unhappy (
this needs updating tests
Comment #6
johnv@podarok, seems there is a general error in the tests. Below error also appears in #1926360:
PHP Fatal error: Call to undefined function gmap_todim() in /var/lib/drupaltestbot/sites/default/files/checkout/sites/default/modules/gmap/tests/gmap.test on line 22
Comment #7
podarok#6
yes - tests for gmap module a old and with deprecated functions in it
we need update em for successfull "need review" process
possibly in follow-up
Comment #8
podarok#2: gmap-undefined-style-plugin-1890460-1782709.patch queued for re-testing.
Comment #9
podaroktestbot happy
looks like #4 reasonable for me
can You update a patch with it?
Comment #10
johnvFresh patch including #3 and #4 and code styling.
These kind of patches seem more like symptom-busting then root cause analysis, but "a'-la"
Comment #11
podarokgood to see tests here for page generating Notice( /tests/gmap.test )
Comment #12
helmo commentedA quick re-roll of the patch in #10
@podarok: What would you like to test for? This issue seems clear enough.
Comment #13
podarok2 #12
Good to see here a test reproducing notice
Comment #14
podarok#10: gmap_1890460_10_plugin_style_error.patch queued for re-testing.
Comment #15
podarok#12 commited pushed to 7.x-2.x
thanks
Comment #16
podarokstatus for tests