Opened 4 years ago

Closed 4 years ago

#16562 closed Bug (duplicate)

intcomma now (incorrectly) requires USE_THOUSAND_SEPARATOR = True

Reported by: anonymous Owned by: nobody
Component: contrib.humanize Version: master
Severity: Release blocker Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


Prior to r16168, the django.contrib.humanize intcomma filter, when used in a project with settings:

USE_L10N = True

would insert commas into numbers. After r16168, intcomma requires that USE_THOUSAND_SEPARATOR = True in order for it to insert the commas (or other separator for other languages, I'd guess). This is a regression and a bit non-sensical: if I'm specifically coding intcomma into my templates, it's a safe bet I do want the commas added, regardless of the USE_THOUSAND_SEPARATOR setting.

Change History (3)

comment:1 Changed 4 years ago by SmileyChris

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Accepted

comment:2 Changed 4 years ago by bpeschier

Marked #16558 as a duplicate as it describes a symptom.

comment:3 Changed 4 years ago by mtredinnick

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

Looks like a dupe of #16404.

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