Opened 10 years ago

Closed 7 years ago

#2710 closed defect (fixed)

presenceservice.py has broken string formatting syntax

Reported by: walter Owned by: erikos
Priority: Urgent Milestone: Unspecified
Component: Sugar Version: Git as of bugdate
Severity: Unspecified Keywords:
Cc: Distribution/OS: Unspecified
Bug Status: New

Description

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.

Change History (3)

comment:1 Changed 8 years ago by dnarvaez

  • Component changed from sugar-toolkit to Sugar

comment:2 Changed 7 years ago by dnarvaez

  • Bug Status changed from Unconfirmed to New
  • Priority changed from Unspecified by Maintainer to Urgent

comment:3 Changed 7 years ago by dnarvaez

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.