Modify

Opened 7 years ago

Closed 7 years ago

#8021 closed defect (fixed)

[Patch] String formats should be `%(name)s` directive instead of `%s`

Reported by: Jun Omae Owned by: Steffen Hoffmann
Priority: normal Component: DynamicFieldsPlugin
Severity: normal Keywords: i18n
Cc: Trac Release: 0.12

Description

DynamicFieldsPlugin now uses %s in the messages catalog. It makes the translations difficult. The formats should be %(name)s directive instead of %s.

directive-name.diff

Attachments (1)

directive-name.diff (3.4 KB) - added by Jun Omae 7 years ago.

Download all attachments as: .zip

Change History (4)

Changed 7 years ago by Jun Omae

Attachment: directive-name.diff added

comment:1 Changed 7 years ago by Jun Omae

Summary: String formats should be `%(name)s` directive instead of `%s`[Patch] String formats should be `%(name)s` directive instead of `%s`

comment:2 Changed 7 years ago by Steffen Hoffmann

Status: newassigned

Looks saner, more understandable indeed. Thanks for the suggestion. You are a hurricane running through all the translations available by now, right? ;-) This is most appreciated.

comment:3 Changed 7 years ago by Steffen Hoffmann

Resolution: fixed
Status: assignedclosed

(In [9417]) DynamicFieldsPlugin: Change strings aka msgid's containing variable parts for web UI, closes #8021.

Replacements (%s) are rewritten with named variables like %(variable)s. This helps a lot when translating without looking at the source code. Thanks to Jun Omae for both, suggestion and providing actually better code.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Steffen Hoffmann.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.