[Bugs] #2710 UNSP: presenceservice.py has broken string formatting syntax

Sugar Labs Bugs bugtracker-noreply at sugarlabs.org
Mon Mar 21 12:33:45 EDT 2011


#2710: presenceservice.py has broken string formatting syntax
------------------------------------------+---------------------------------
    Reporter:  walter                     |          Owner:  erikos                     
        Type:  defect                     |         Status:  new                        
    Priority:  Unspecified by Maintainer  |      Milestone:  Unspecified by Release Team
   Component:  sugar-toolkit              |        Version:  Git as of bugdate          
    Severity:  Unspecified                |       Keywords:                             
Status_field:  Unconfirmed                |   Distribution:  Unspecified                
   Seeta_dev:                             |  
------------------------------------------+---------------------------------
 There are several places throughout the code that use a comma instead of a
 % so that the format of error messages and log messages break, e.g.:

         if self._activity_cache is not None:
             raise ValueError('Activity %s is already tracked',
                              activity.get_id())

 should be:

         if self._activity_cache is not None:
             raise ValueError('Activity %s is already tracked' % \
                              activity.get_id())

 I'll attach a patch.

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


More information about the Bugs mailing list