Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#16973 closed Bug (duplicate)

Admin messages need capfirst for translations

Reported by: Erik Wognsen Owned by: nobody
Component: contrib.admin Version: master
Severity: Normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


As discussed in the i18n group here, admin messages such as "The %(name)s "%(obj)s" was changed successfully." have translations starting immediately with "%(name)s" in many languages. Since the verbose name of the model is lowercase, the result is a message saying e.g. 'employee "Xyz" was changed successfully' where it should start with 'Employee'.

Attachments (1)

admin_message_capfirst_fix.diff (1.6 KB) - added by Erik Wognsen 9 years ago.

Download all attachments as: .zip

Change History (4)

Changed 9 years ago by Erik Wognsen

comment:1 Changed 9 years ago by Erik Wognsen

The attached patch also fixes the problem that these three translated strings exist:

'The %(name)s "%(obj)s" was added successfully.',
'You may edit it again below.', and the combined
'The %(name)s "%(obj)s" was added successfully. You may edit it again below.'

In response_change where the combined version is used, I have split it into the two already translated strings so the redundant string is removed and it matches the way 'You may edit it again below.' is added to the 'was added successfully' message in response_add.

comment:2 Changed 9 years ago by Julien Phalip

Resolution: duplicate
Status: newclosed

Thanks for this. It has already been reported in #16350, so I'm closing as a duplicate. Please continue the conversation over there. Cheers!

comment:3 Changed 9 years ago by Aymeric Augustin

#17246 was a duplicate.

Note: See TracTickets for help on using tickets.
Back to Top