#7856 HIGH 8.2.0 (: notify::active behaviour change

[Zarro Boogs per Child]

#7856: notify::active behaviour change
----------------------+-----------------------------------------------------
   Reporter:  dsd     |       Owner:  tomeu                        
       Type:  defect  |      Status:  new                          
   Priority:  high    |   Milestone:  8.2.0 (was Update.2)         
  Component:  sugar   |     Version:  not specified                
 Resolution:          |    Keywords:  blocks-:8.2.0 polish:8.2.0 r+
Next_action:  review  |    Verified:  0                            
  Blockedby:          |    Blocking:  7467, 7771                   
----------------------+-----------------------------------------------------
Changes (by tomeu):

  * keywords:  blocks-:8.2.0 polish:8.2.0 r? => blocks-:8.2.0 polish:8.2.0
               r+


Comment:

 {{{
             if self._service != None:
                 self.set_active(True)
 }}}

 No need to test it here as we are testing inside the method.

 {{{
         if self._service:
             self._service.SetActive(state,
 }}}

 Better use if self._service is None:

-- 
Ticket URL: <http://dev.laptop.org/ticket/7856#comment:10>
One Laptop Per Child <http://laptop.org/>
OLPC bug tracking system