Opened 18 years ago
Closed 14 years ago
#3884 closed Uncategorized (wontfix)
humanize filter `intcomma` should be renamed to `numcomma`
| Reported by: | Chris Beaven | Owned by: | Adrian Holovaty |
|---|---|---|---|
| Component: | Contrib apps | Version: | dev |
| Severity: | Normal | Keywords: | |
| Cc: | lidaobing@… | Triage Stage: | Design decision needed |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | yes | UI/UX: | no |
Description
The current filter name doesn't give it justice (it also comma separates floats).
Backwards compatibility is easy, and it'd be good to provide a documentation example of it handling a float.
#3017 was the inspiration for this ticket.
Change History (5)
comment:1 by , 18 years ago
| Triage Stage: | Unreviewed → Design decision needed |
|---|
comment:2 by , 18 years ago
| Resolution: | → wontfix |
|---|---|
| Status: | new → closed |
I'm not necessarily against renaming the filter, but I don't think commaseparate is a great name because of the tendency for people to misspell "separate". Reopen with another name if you can think of one.
comment:3 by , 16 years ago
| Cc: | added |
|---|
comment:4 by , 14 years ago
| Easy pickings: | set |
|---|---|
| Resolution: | wontfix |
| Severity: | → Normal |
| Status: | closed → reopened |
| Summary: | humanize filter `intcomma` should be renamed to `commaseparate` → humanize filter `intcomma` should be renamed to `numcomma` |
| Type: | → Uncategorized |
How about "numcomma"?
comment:5 by , 14 years ago
| Resolution: | → wontfix |
|---|---|
| Status: | reopened → closed |
At this point in Django's life, I'm not sure I see the value in changing the name. It might not be the most accurate name in the world, but it isn't fundamentally broken, and changing it would require a whole bunch of deprecation/backwards compatibility dancing for what is, at the end of the day, not a whole lot of gain.
If the name change was required in the context of some clarification of what intcomma actually does, I might be convinced, but if were doing it just for the sake of theoretical purity, I think we can pass.
What does core think?