Opened 7 years ago

Closed 4 years ago

#15575 closed defect (patchwelcome)

[regression] dojox/geo/openlayers/widget/Map not working in conjunction with OpenLayers 2.11

Reported by: pags Owned by: Patrick Ruzand
Priority: undecided Milestone: 1.13
Component: DojoX Widgets Version: 1.8.0b1
Keywords: Cc: Kitson Kelly
Blocked By: Blocking:

Description

Receiving:

Uncaught TypeError?: Cannot read property 'w' of null OpenLayers?.js:563

It appears the underlying OpenLayers? map's size object isn't initialized by the time it is needed.

Change History (8)

comment:1 Changed 7 years ago by Kitson Kelly

Cc: Kitson Kelly added

comment:2 Changed 7 years ago by pags

Map is being initialized within a templated widget.

postCreate : function() {
            this.inherited(arguments);
            this.mapWidget = new Map({
                baseLayerType : dojox.geo.openlayers.BaseLayerType.OSM,
                style : "border:1px solid #DDD;height:100%;width:100%",
                initialLocation : {
                    position : [-74.00597, 40.71427]
                }
            }, this.map);}
Last edited 7 years ago by Kitson Kelly (previous) (diff)

comment:3 Changed 7 years ago by pags

This was working in 1.7.2 and 1.7.3 - broke after 1.8 beta.

comment:4 Changed 7 years ago by Kitson Kelly

Milestone: tbd1.8.1
Summary: dojox/geo/openlayers/widget/Map not working in conjunction with OpenLayers 2.11[regression] dojox/geo/openlayers/widget/Map not working in conjunction with OpenLayers 2.11

comment:5 Changed 7 years ago by cjolif

Owner: changed from dante to pags
Status: newpending

This is probably following #15526 and in particular [28935]. The dojox/geo code was hardcoding the size of the map div inside its component. This hacky code was removed thus revealing your issue. If you correctly size your div in CSS you should not have the problem anymore. At least we were able to make all our demos / samples work with OL by doing so (see the ticket).

If you still get it please provide us with a full & simple example reproducing the issue to accelerate its investigation.

comment:6 Changed 7 years ago by cjolif

Owner: changed from pags to Patrick Ruzand
Status: pendingassigned

comment:7 Changed 7 years ago by cjolif

Milestone: 1.8.1tbd

comment:8 Changed 4 years ago by dylan

Milestone: tbd1.12
Resolution: patchwelcome
Status: assignedclosed

Given that no one has shown interest in creating a patch in the past 3+ years, I'm closing this as patchwelcome.

Note: See TracTickets for help on using tickets.