Closed (fixed)
Project:
GMap Module
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
18 Sep 2007 at 23:31 UTC
Updated:
19 Jan 2008 at 02:53 UTC
Jump to comment: Most recent file
Today, I finally got around to bringing the marker managers up to date. I also refactored the common behavior into its own file. Please test the next snapshot (Should appear in ~30 minutes from now) and report any issues. I want to make sure I didn't break anything else while doing this.
| Comment | File | Size | Author |
|---|---|---|---|
| #23 | gmap_bug_0.jpg | 82.5 KB | mikey081 |
| #4 | js-clusterer_marker.js_.patch | 707 bytes | pillarsdotnet |
Comments
Comment #1
bdragon commentedRegarding Clusterer:
The README is currently out of date in regards to installing Clusterer. You want Clusterer2.js, and it goes in thirdparty/, NOT the root gmap folder.
I have a rewritten README in my working copy, but I'm not quite done with it yet.
Comment #2
bdragon commentedWhoops, forgot to post the dupe list.
http://drupal.org/node/127449
http://drupal.org/node/157330
http://drupal.org/node/154794
http://drupal.org/node/149275
http://drupal.org/node/134110
http://drupal.org/node/126845
Additionally, I fixed a bug earlier today regarding click not working after my changes. This is now fixed.
Comment #3
pillarsdotnet commentedWhen enabling Clusterer2.js, clicking on a non-clustered pin has no effect. Still trying to figure out why...
Comment #4
pillarsdotnet commentedFixed. Don't know why you create a copy of the marker object; you lost some of the properties in the copy. Commenting out that part yields the proper behavior (at least for me).
Here's the patch, both inline and attached.
Comment #5
mr.andrey commentedThe patched fixed the single-node info window pop-up problem.
For some reason, double entries of nodes appear in the cluster popup. For example, if I click on a cluster that has two users, it gives me:
user1
user2
user3
user2
user2
And when there is a cluster present, but there's only one user, it somehow becomes a cluster and pops up:
user4
user4
Any ideas what might be causing this?
Thanks,
Andrey.
Comment #6
bdragon commentedBlargh, I missed clusterer when moving the marker creation calls to a common file. Sorry about that.
@pillarsdotnet: Your patch is correct. Instead of commenting out the lines, I removed them. Thanks.
Comment #7
pillarsdotnet commentedSo... is the patch going to be committed to dev or to HEAD ?
I'm already using it on http://www.kbsystem.com/dealers
Comment #8
bdragon commentedSorry, thought it was obvious that I had committed it.
http://drupal.org/cvs?commit=82829
Comment #9
mr.andrey commentedIs anyone else getting the problem where a single user is repeated twice, thus forming a cluster where it should be just a single marker? Am I missing something here?
Thanks,
A.
Comment #10
bdragon commentedGot a link to a test site?
Comment #11
mr.andrey commentedI just sent you the login info.
Thanks,
Andrey.
Comment #12
bdragon commentedHmm, you do have multiple markers in the same spot.
Here's the markers that are being generated, in order.
andrey, fredburks, admin, admin, admin, bethreuter, admin, andrey, andrey, fredburks, admin, andrey, andrey, andrey, admin.
Could I see what your "location" table in the database contains at the moment?
Comment #13
mr.andrey commentedSure.
Looks like a load of extra stuff.
A.
Comment #14
bdragon commentedYeah, it looks like the actual data contains duplicates. Not sure why though. Are you using the modified location and gmap_location modules from http://drupal.org/node/125001 and http://drupal.org/node/172841 ?
Comment #15
mikey081 commentedHad a similar problem. I applied the patches you provided in the links. This solved the problem of my single user getting clustered. However, even though I have "Marker Action: Open Info Window" checked, no info window pops up with the Cluster Manager activated. Only with it turned off do the info windows pop up. Futhermore, I have numerous nodes located at the same address, yet the node location map shows no clusters.
Comment #16
mr.andrey commentedI just downloaded the Oct-2 release and applied the patches you mentioned. The patch to the location module worked fine, and the second, very long patched mostly worked except for gmap_marker.js:
update.php ran fine, and now looking at the map, the doubles disappeared.
Everything seems to be working fine, thanks!
One minor bug I'm noticing is that at the default zoom level of 8, there is one cluster and one user. When I zoom in one level, the single marker becomes a part of the cluster. Any ideas why this would happen? You can see it on my site.
Best,
Andrey.
Comment #17
mr.andrey commentedOh, just noticed, the Location Map tab on the user profile has disappeared, so users can't set their location. Any ideas on what might have happened?
Thanks,
Andrey.
Comment #18
mr.andrey commentedOops, forgot to copy the location directory to the live site from the dev server. Ignore my last comment. The Location Map now shows up under the address. -A
Comment #19
mr.andrey commentedHi there,
There's a strange behavior that I'm seeing.
Starting at zoom level 0 (farthest out), I have a few people in the SF Bay Area. They are all in a cluster, which is good. Zoom level 1 is the same. On zoom level 2, one of them separated from the cluster and is showing up by itself, thus shadowing the cluster. On zoom level 3, it's back into the cluster. Zoom level 4, 5 and 6, there is only the cluster. On zoom level 7, a marker separated from the cluster, and it is not the same one as the one on level 2 (?). On zoom level 8, it's back in the cluster and the marker that separated on zoom level 2 is now broken away from the cluster. Zoom level 9 & 10 - everyone's back in the cluster. 11 - both of the markers that got broken away are now separate from the cluster. From this point on, the behavior is normal.
Any ideas? This seems totally random, though very consistent.
bdragon, if you want you can see this on my dev server.
Thanks,
Andrey.
Comment #20
bdragon commentedI'm seeing this too. My instinct is that it is actually a bug in Clusterer.
Comment #21
mr.andrey commentedI just sent an email to Acme Labs with login info so they can see it for themselves. I included this thread too.
Any other ideas?
Andrey.
Comment #22
bdragon commentedMore appropriate title
Comment #23
mikey081 commentedNoticed you made a lot of awesome changes and updates to the GMap module this past day. Excellent work. The clusterer now works for me for the first time now! It works well for my node location map- I do notice the cluster and un-cluster issue at various zooms that mr.andrey noted. The bigger problem I'm having is that it still doesn't work well with my user map. I have a single user, yet a cluster is created for it. Also, when I click on the cluster, I get 10 replications in the info box, and the last line says there are 277 more! Pic attached to show you exactly what I mean...
Seems like a simple bug that is reading things weirdly from the database, or might be something else entirely. I'm not a programmer, so I can only tell you what I see in plain view...
Thanks for all the hard work!
Mikey
Comment #24
mr.andrey commentedDid you try this patch?
http://drupal.org/files/issues/gmap_location_rewrite.patch
And this one on Location?
http://drupal.org/files/issues/location-user-latlon.patch
I had the same issue with many usernames being generated for one person, thus creating a cluster, and the above two patches with the Oct-4 release fixed that.
And yes, thanks bdragon for all the wonderful work on this, Category, and other modules - it's really making a lot of things possible for many people, and it is very much appreciated! I'm looking forward to your new baby Geo.
Best,
Andrey.
Comment #25
mikey081 commentedThat did it- although my eyes are killing me after going in to the gmap location module modifications... Thank you for your help!
Comment #26
mr.andrey commentedYou did it manually? ack! Just do this next time you upgrade gmap:
In shell:
I tried to do it manually at first too, but it drove me nuts, and didn't work, as I definitely missed something, then I remembered patching linux kernel back in my sysadmin days, and dug up how to patch things automatically. You can do this with most patches, however, if you apply multiple short ones, sometimes it's still better to do it manually, as the changes might be conflicting.
Best,
Andrey.
P.S. I would really like to get the Clusterer fixed, as it's the only thing that doesn't work flawlessly. I didn't hear back from Acme Labs yet about their script.
Comment #27
mr.andrey commentedAnother clustering issue, this one's very strange.
On Firefox everything works, but on IE, I'm getting a javascript error:
And though the cluster appears, I cannot click on it, whereas in Firefox, I can and it works fine.
Any ideas on this one?
Best,
Andrey.
Comment #28
mr.andrey commentedWhile using a debugger, this is the line that breaks the script:
And here it is in context:
Comment #29
bdragon commentedLooks like google maps compressed code. Reading that compressed code makes me feel that this is the code that sets up popup dimensions. Perhaps it's balking on displaying too much data?
Comment #30
mikey081 commentedSecond that- except I don't even get markers showing in the latest version of IE. For some reason, an older version of IE showed markers, but they were unclickable.
Comment #31
mr.andrey commentedI noticed that the old version of gmap that is on my dev server works fine, so I did a diff -qr and replaced the files one by one until the error was gone. The file that seemed to be making the difference was js/icon.js, and here is the difference between the old (working) and the new (broken) Oct-10 version:
Changing the line back to the old version fixes the problem.
Any ideas on this?
Andrey
Comment #32
bdragon commentedYeah, do you have any references to nonexistent files in your marker .ini files?
Comment #33
mr.andrey commentedNot any that I put in there - it all came with standard markers as unpacked from gmap module, and I didn't alter anything in the .ini files prior to this error.
Best,
Andrey.
Comment #34
bdragon commentedYeah, I had accidentally put shadow = '' in several .ini files. This was causing major weirdness in some browsers.
Should be working now.
Comment #35
(not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.