Ticket #2301 (new enhancement) — at Version 1

Opened 2 years ago

Last modified 23 months ago

Replace old-style string formatting with format() method (at least in strings marked for translation)

Reported by: seanh Owned by: seanh
Priority: minor Milestone: ckan-future
Component: ckan Keywords: i18n
Cc: Repository: ckan
Theme: none

Description (last modified by seanh) (diff)

Some strings in CKAN are passed to gettext with more than one %s in them, this is no good as translators may need to change the order of substituted words.

Strings should not use the old style % substitution, they should use the new format method in Python 2.6+, e.g.:

'This {food} is {adjective}.'.format(food='spam', adjective='absolutely horrible')

​http://docs.python.org/release/2.6.8/tutorial/inputoutput.html#fancier-output-formatting

Change History

comment:1 Changed 2 years ago by seanh

  • Description modified (diff)
  • Summary changed from Fix substitutions in strings marked for translation to Replace old-style string formatting with format() method (at least in strings marked for translation)
Note: See TracTickets for help on using tickets.