[OpenLayers-Trac] [OpenLayers] #1564: extend and applyDefaults should allow undefined first arg

OpenLayers trac at openlayers.org
Thu Jun 5 16:34:29 EDT 2008


#1564: extend and applyDefaults should allow undefined first arg
----------------------+-----------------------------------------------------
  Reporter:  tschaub  |       Owner:             
      Type:  feature  |      Status:  new        
  Priority:  minor    |   Milestone:  2.7 Release
 Component:  Util     |     Version:  2.6        
Resolution:           |    Keywords:             
     State:  Commit   |  
----------------------+-----------------------------------------------------
Comment (by tschaub):

 Replying to [comment:3 elemoine]:
 > My review:
 >
 >  * in KML.js why don't you just do:
 {{{OpenLayers.Util.extend(feature.style, inlineStyle)}}} and actually
 ignore the return value?

 If feature.style is undefined before the extend, it still will be
 afterwards.

 >  * the above comment applies to your changes in !MapServer.js,
 !WorldWind.js, !KaMap.js

 Ditto (but substitute params).

 >
 > Thanks Tim,

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


More information about the Trac mailing list