[OpenLayers-Trac] Re: [OpenLayers] #2959: instance of Array + GWT + Buttons + IFRAME = BUG in Panel.js

OpenLayers trac-20090302 at openlayers.org
Thu Jun 16 12:00:07 EDT 2011


#2959: instance of Array + GWT +  Buttons + IFRAME = BUG in Panel.js
-----------------------------------------------+----------------------------
 Reporter:  grigoryevigor                      |       Owner:              
     Type:  bug                                |      Status:  new         
 Priority:  major                              |   Milestone:  2.12 Release
Component:  Control.Panel                      |     Version:  2.10        
 Keywords:  instanceof Array IFRAME GWT Panel  |       State:  Review      
-----------------------------------------------+----------------------------
Changes (by mwootendev):

  * state:  => Review


Comment:

 I have added a patch that should resolve the issue. The patch adds an
 OpenLayers.Util.isArray(a) function that utilizes the toString() method
 mentioned above. I though this might be cleaner than typeOf() === "array".

 I replaced all "x instanceof Array" instances in the library code with
 "OpenLayers.Util.isArray(x)".

 I added a test case for isArray() to Utils.html.

-- 
Ticket URL: <http://trac.openlayers.org/ticket/2959#comment:5>
OpenLayers <http://openlayers.org/>
A free AJAX map viewer


More information about the Trac mailing list