Will this module be upgraded to 6.x at any time?
The features in geobrowser are way cooler than the gmap+location+views solution, because of the layer that constantly looks after new nodes when moving the map. I really need this module, and will try to upgrade it myself, if noone with better php/drupal knowledge than me steps forward:)

Comments

socialnicheguru’s picture

subscribing... just discovered this and looks great.

farald’s picture

I am no coder, but Ive managed to create a downsized version of geobrowser. It barely contains the functionality of the initial dev version. I leave this note here as tips if someone wants to do the job of properly porting the 1.0-version of the module.

I have ported the so-called 7.0-release (im guessing a typo) of the browser, which it rather old, and do not contain the "create geobrowser page" - stuff. Porting that version would be over my current capabilities.

When in 5.x, I removed the map selection features in admin/geobrowser, and the test map functionality.

To get it working, the first thing needed was to redraw the database calls made by geobrowser (geobrowser.module) to:

<?php
      $get_nodes = "
        SELECT
        {location}.latitude AS geobrowser_lat,
        {location}.longitude AS geobrowser_lon, 
        {location}.lid AS geobrowser_vid, 
        {node}.nid AS geobrowser_nid, 
        tid".$_REQUEST['current_tree']." as term_node_tid, 
        {node}.title AS node_revisions_title, 
        {votingapi_vote}.value IS NULL AS isnull
      
        FROM  {location}
        INNER JOIN {location_instance} ON {location_instance}.lid = {location}.lid 
		INNER JOIN {node} ON {node}.vid = {location_instance}.vid AND {node}.nid = {location_instance}.nid 
        $layers_sql
        LEFT JOIN {votingapi_vote} ON {node}.nid = {votingapi_vote}.content_id
      
        WHERE  
      ";
?>

This seems to do the trick to make the query work with location 3.x, and then in the end drupal 6.x. This because of the location 3.x's changed database structure.

The next thing I did was to do a rough deadwood conversion of the module.
With my downsized version, I encountered only two problems:

The browser.php would not load after the conversion due to that deadwood changed the geobrowser_display_map(). Changing this function back to include("browser.php"); did the trick.

The last thing to fix before it worked was the db_num_rows() call in browser.php. I removed this entire if-sentence, because this was only a checker to see in there was vocabularies assigned that did not exist. The only alternative I found was to call the db with a db_result, so I removed it instead.

enzo de simone’s picture

I experienced in these days to do some attempts to make to work the form on D6 but I have had to a series of mistakes stop that I do not know to resolve. At any rate consignment the compliments for the work you turn hoping also do not may want us long times for the passage from D5 to D6

phayes’s picture

Hi All,

Sincerest apologies to not responding to this earlier. I've been a bad neglectful contributor. I am actively working on a port to drupal6. Anyone want to help test if I were to post my work so far here?

farald’s picture

Yes go ahead! I will gladly do some betatesting! :)

phayes’s picture

Alright. geobrowser for drupal 6 is up: http://drupal.org/node/187964/release

Please beta test!

PS. You may need to wait until the nightly cron is run for the tar file.

enzo de simone’s picture

Where?

farald’s picture

Phayes, it seems that you have forgot to release the 6.x version, no release, no dev:)
..or maybe the releases haven't updated itself from CVS yet?

phayes’s picture

This is very strange.

The release is here: http://drupal.org/node/381756

But it is "permission denied" when I am not logged in -- CVS has created the file and is here:

http://ftp.drupal.org/files/projects/geobrowser-6.x-1.x-dev.tar.gz

Am I missing a step to get this to show up normal? I tried going into the project page and clicking "create new release", but when I enter all the information it says "That release already exists". Any seasoned CVSers out there?

Thanks all

P

enzo de simone’s picture

Now is OK. Thanks

farald’s picture

Still showing access denied in drupal. But the ftp link is working.

phayes’s picture

Status: Active » Fixed

Seems to have fixed itself. Must have been a drupal.org upgrade issue that got fixed.

Please report all bugs!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.