Opened 13 years ago
Closed 10 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 10 years ago by dnarvaez
- Component changed from sugar-toolkit to Sugar
comment:2 Changed 10 years ago by dnarvaez
- Bug Status changed from Unconfirmed to New
- Priority changed from Unspecified by Maintainer to Urgent
comment:3 Changed 10 years ago by dnarvaez
- Resolution set to fixed
- Status changed from new to closed
Note: See
TracTickets for help on using
tickets.