[OpenLayers-Trac] [OpenLayers] #1234: select feature control calls onUnselect twice

OpenLayers trac at openlayers.org
Thu Jan 3 03:41:41 EST 2008


#1234: select feature control calls onUnselect twice
------------------------------------+---------------------------------------
  Reporter:  tschaub                |       Owner:  tschaub    
      Type:  bug                    |      Status:  assigned   
  Priority:  major                  |   Milestone:  2.6 Release
 Component:  Control.SelectFeature  |     Version:  2.5        
Resolution:                         |    Keywords:             
     State:  Review                 |  
------------------------------------+---------------------------------------
Comment (by elemoine):

 Tim,

 Important thing:

  * in unselectAll() you must check that options isn't null before
 dereferencing it (the clickoutFeature method fails with your patch)

 Unimportant thing:

  * in unselectAll() you define the feature var outside the for loop. Is
 that on purpose (for performance reason)?

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


More information about the Trac mailing list