Opened 16 years ago
Closed 16 years ago
#9030 closed (duplicate)
GenericInlineModelAdmin ignores max_num
| Reported by: | Owned by: | nobody | |
|---|---|---|---|
| Component: | Contrib apps | Version: | 1.0 |
| Severity: | Keywords: | inline, generic | |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
InlineModelAdmin takes the argument "max_num", but GenericInlineModelAdmin ignores this argument.
just add follwoing line in django/trunk/django/contrib/contenttypes/generic.py :
"max_num": self.max_num,
between line 370 and 380.
is it right that a single-line patch doesn't need a patch file?
Change History (3)
comment:1 by , 16 years ago
| Has patch: | unset |
|---|---|
| Triage Stage: | Unreviewed → Accepted |
comment:2 by , 16 years ago
It also ignores the "exclude" argument. I opened #9122 with a patch for both before seeing this ticket. Should this one be closed, since mine is broader and includes a patch?
comment:3 by , 16 years ago
| Resolution: | → duplicate |
|---|---|
| Status: | new → closed |
Closing as a dupe of #9122.
Note:
See TracTickets
for help on using tickets.
In general, yes, *always* attach a patch. They are precise in exactly you want changed and makes it easier for us to apply, test and commit. Next time do so :)