[Bugs] #1966 UNSP: restart of NetworkManager results in shell.log message

[Sugar Labs Bugs]

#1966: restart of NetworkManager results in shell.log message
------------------------------------------+---------------------------------
    Reporter:  quozl                      |          Owner:  tomeu                      
        Type:  defect                     |         Status:  new                        
    Priority:  Unspecified by Maintainer  |      Milestone:  Unspecified by Release Team
   Component:  sugar                      |        Version:  0.84.x                     
    Severity:  Trivial                    |       Keywords:                             
Distribution:  Unspecified                |   Status_field:  Unconfirmed                
------------------------------------------+---------------------------------
 On Sugar 0.84.15, if NetworkManager is restarted while there is at least
 one access point in the mesh view, the shell.log contains a message:

 {{{
 1272267390.519734 ERROR dbus.connection: Exception in handler for D-Bus
 signal:
 Traceback (most recent call last):
   File "/usr/lib/python2.6/site-packages/dbus/connection.py", line 214, in
 maybe_handle_message
     self._handler(*args, **kwargs)
   File "/usr/share/sugar/extensions/deviceicon/network.py", line 309, in
 __ap_properties_changed_cb
     self._update_properties(properties)
   File "/usr/share/sugar/extensions/deviceicon/network.py", line 338, in
 _update_properties
     data = self._name + hex(self._flags)
 TypeError: can only concatenate list (not "str") to list
 }}}

 There is no apparent effect of this message, and there's no indication
 that it causes a delay in reconnecting, but there is that possibility.

 This section of the code has not been materially changed from 0.84 to
 0.88.  A variable renaming happened, is all.  So there is a possibility it
 also may occur in 0.88.

 Adding debugging code shows that ''_update_properties'' is occasionally
 called with a ''Ssid'' value that is not a string.  The debugging code
 emits ''repr(self._name)'' when it is changed in ''_update_properties'':

 {{{
 1272268032.860004 ERROR root: _update_properties: new self._name:
 dbus.ByteArray('quozl.linux.org.au', variant_level=1)
 1272268090.983071 ERROR root: _update_properties: new self._name:
 dbus.Array([dbus.Byte(113), dbus.Byte(117), dbus.Byte(111),
 dbus.Byte(122), dbus.Byte(108), dbus.Byte(46), dbus.Byte(108),
 dbus.Byte(105), dbus.Byte(110), dbus.Byte(117), dbus.Byte(120),
 dbus.Byte(46), dbus.Byte(111), dbus.Byte(114), dbus.Byte(103),
 dbus.Byte(46), dbus.Byte(97), dbus.Byte(117)],
 signature=dbus.Signature('y'), variant_level=1)
 }}}

-- 
Ticket URL: <http://bugs.sugarlabs.org/ticket/1966>
Sugar Labs <http://sugarlabs.org/>
Sugar Labs bug tracking system