Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.It seems that the GMap module creates a file at sites/default/files/js/gmap_markers.js, but then it calls for a file called sites/default/files/js/gmap_marker.js (note the difference in pluralism in the file name). I had this same problem as these guys, but none of us could find the solution:
http://drupal.org/node/1932288
GMap spits out errors all day long that read:
"Page not found: sites/default/files/js/gmap_marker.js"
If I flush the js directory and regenerate the GMap marker cache then I get a message that indicates everything will be fine, but then the file is still not named correctly. I have also checked that the directory and file permissions are correct, and they are.
| Comment | File | Size | Author |
|---|---|---|---|
| #22 | gmap_marker_js_fix-1946494-21.diff | 476 bytes | reeller_sk |
| #6 | gmap-1946494-marker_js_fix.diff | 438 bytes | reeller_sk |
Comments
Comment #1
jlemmon CreditAttribution: jlemmon commentedIt seems that this has been fixed in at least the most recent development version. However, in Version 7.x-2.4 you might take a look at line 352 of the gmap.module file, where it reads:
$ret[variable_get('file_public_path', conf_path() . '/files') . '/js/gmap_marker.js'] = array(Try changing this to:
$ret[variable_get('file_public_path', conf_path() . '/files') . '/js/gmap_markers.js'] = array(Comment #2
ggarry CreditAttribution: ggarry commentedComment #3
ggarry CreditAttribution: ggarry commentedPretty sure its no leading slash that is causing it.
"Page not found: sites/default/files/js/gmap_marker.js" should be "Page not found: /sites/default/files/js/gmap_marker.js"
http://drupal.org/node/1944692
Comment #4
johnvPlease try -dev version. Most probably it will fix your problem.
Comment #5
podaroktag beta release
Comment #6
reeller_sk CreditAttribution: reeller_sk commentedActually in dev version is still bug. Here is a patch. Problem is missing base_path at url start.
Comment #7
podarokbot?
Comment #9
podarokpatch introduces a Undefined index bug
Comment #10
reeller_sk CreditAttribution: reeller_sk commentedIf you see the file bugs are there already, in patch is only change the beginning slash changed to base_path function.
Comment #11
johnv@podarok, the test errors in "Location Gmap Find Address button checks" are a general error. They exist in every patch the last few weeks.
Comment #12
podarok#11 any patch can fix the bug with ternary operator for undefined index in
Undefined index: user_latitude Notice location.module line 692 location_locationapi()We shouldn`t commit any changes and bug fixes without fixing that
Tests developed for quality and fixing bugs , not for skipping errors introduced by any previous developers
Better do not commit bug fixes, than ignore tests
Comment #13
johnv@#12, hmm, IMO a patch should only address the issue at hand. We should open separate issue for this other test error.
Comment #14
podarok#13 right
Comment #15
seaneffel CreditAttribution: seaneffel commented#3 The leading slash was copy/pasted poorly from the original error log, so the sample error message shouldn't be taken literally. I believe the problem is the call to the name of the file, from plural "markers" to singular "marker".
I've tried the dev version and it no longer throws the error for incorrect file path, but it also no longer displays the markers on the map.
Comment #16
johnv#6: gmap-marker_js_fix.diff queued for re-testing.
Comment #18
podarok#6 needs reroll
Comment #19
reeller_sk CreditAttribution: reeller_sk commented#6: gmap-marker_js_fix.diff queued for re-testing.
Comment #21
johnvIMO we should close this issue. The original error (marker.js vs. markers.js) is resolved.
The remaining error is equal to #1218870: Marker managers give wrong path to marker icon when Drupal is not installed at root
That should handle the subdirectory-issue in 2 or 3 locations.
Then all these duplicate issues should be closed.
Let's continue at #1218870: Marker managers give wrong path to marker icon when Drupal is not installed at root
Comment #22
reeller_sk CreditAttribution: reeller_sk commentedHere is the patch from #6 on new codebase.
Comment #23
reeller_sk CreditAttribution: reeller_sk commentedOk, so I will post the new patch in that thread also.
Comment #24
podarokYou should set needs review for bot`s tests
Comment #25
podarokdue to #21 this is duplicate of #1218870: Marker managers give wrong path to marker icon when Drupal is not installed at root