When I click the full screen button, I'm getting a javascript error: "ReferenceError: map is not defined". If I look closer in openlayers_behavior_fullscreen.js (line 37) the 'map' variable indeed is undefined. (maybe a leftover from a previous code)

CommentFileSizeAuthor
#3 openlayers-1962240-fullscreen_map_extent_undefined.diff824 bytesfilsterjisah
#1 openlayers-1962240-fullscreen_map_extent_undefined.diff928 bytesfilsterjisah
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

filsterjisah’s picture

filsterjisah CreditAttribution: filsterjisah commented
Status: Active » Needs review
FileSize
openlayers-1962240-fullscreen_map_extent_undefined.diff928 bytes

Patch below should fix it, needs a review.

Status: Needs review » Needs work

The last submitted patch, openlayers-1962240-fullscreen_map_extent_undefined.diff, failed testing.

filsterjisah’s picture

filsterjisah CreditAttribution: filsterjisah commented
Status: Needs work » Needs review
FileSize
openlayers-1962240-fullscreen_map_extent_undefined.diff824 bytes

Removed 'sites/all/modules/contrib/'

Status: Needs review » Needs work

The last submitted patch, openlayers-1962240-fullscreen_map_extent_undefined.diff, failed testing.

Pol’s picture

Pol
the/that
Location Brussels
CreditAttribution: Pol commented
Status: Needs work » Fixed

Thanks, it's fixed.

http://drupalcode.org/project/openlayers.git/commit/f468333

dianacastillo’s picture

dianacastillo CreditAttribution: dianacastillo commented

should I download the dev version or does the beta 5 version have this fix? thanks

Pol’s picture

Pol
the/that
Location Brussels
CreditAttribution: Pol commented

Hi,

Compare the release date from the beta5 and the commit and you'll get the answer.

Status: Fixed » Closed (fixed)

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