Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#16403 closed Bug (fixed)

contrib.comments uses wrong pluralization of profanity message

Reported by: poirier Owned by: nobody
Component: contrib.comments Version: 1.3
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

contrib.comments says "Watch your mouth! The word %s is not allowed here." for multiple profanities and "Watch your mouth! The words %s are not allowed here." for single ones.

This is because ungettext is being used incorrectly (just like case #5011 from four years ago) - the third arg should be the number of items, not a boolean.

Patch attached.

Attachments (2)

django-comments-ungettext.diff (904 bytes) - added by poirier 4 years ago.
django-comments-ungettext.2.diff (1.0 KB) - added by Ferry Jérémie <ferryjeremie@…> 4 years ago.

Download all attachments as: .zip

Change History (6)

Changed 4 years ago by poirier

comment:1 Changed 4 years ago by jezdez

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Ready for checkin

comment:2 Changed 4 years ago by jezdez

  • Resolution set to fixed
  • Status changed from new to closed

In [16505]:

Fixed #16403 -- Correctly use ungettext in comments form. Thanks, poirier.

Changed 4 years ago by Ferry Jérémie <ferryjeremie@…>

comment:3 Changed 4 years ago by Ferry Jérémie <ferryjeremie@…>

missind also i18n "and" string...

comment:4 Changed 4 years ago by claudep

Latest comment added as new ticket #17597.

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