[OpenLayers-Trac] [OpenLayers] #992: Allow applyDefaults() to return the modified object

OpenLayers trac at openlayers.org
Fri Nov 9 07:56:45 EST 2007


#992: Allow applyDefaults() to return the modified object
----------------------+-----------------------------------------------------
  Reporter:  fredj    |       Owner:             
      Type:  feature  |      Status:  new        
  Priority:  minor    |   Milestone:  2.6 Release
 Component:  Util     |     Version:  SVN        
Resolution:           |    Keywords:  review     
----------------------+-----------------------------------------------------
Comment (by fredj):

 Replying to [comment:7 elemoine]:
 > Replying to [comment:6 euzuro]:
 > > jaja. this does seem like the kind of thing that i'd complain about.
 as eric puts it, though, it seems to make sense to me. if we return a
 value in extend(), we might as well return one in applyDefaults() as well.
 >
 > yeah, and there's is this anonymous dest object thing mentioned by
 fredj.
 >
 >
 > > In theory, I think it's confusing, because the user might think s/he's
 getting a *new* object back.... but still. i think consistency is more
 important.
 >
 > Tests pass on FF. As usual I don't have IE handy. I'll let fredj run the
 tests on IE.
 All tests pass on IE 6

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


More information about the Trac mailing list