#33679: "empty" suffix gets added to last (original) admin inline form row for
readonly views
-----------------------------------------+------------------------
Reporter: Daniel Hahler | Owner: nobody
Type: Bug | Status: new
Component: contrib.admin | Version: 4.0
Severity: Normal | Keywords:
Triage Stage: Unreviewed | Has patch: 0
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
-----------------------------------------+------------------------
`django/contrib/admin/templates/admin/edit_inline/tabular.html` has the
following code:
{{{
<tr class="form-row {% if inline_admin_form.original or
inline_admin_form.show_url %}has_original{% endif %}{% if forloop.last and
inline_admin_formset.has_add_permission %} empty-form{% endif %}"
id="{{ inline_admin_formset.formset.prefix }}-{% if not
forloop.last %}{{ forloop.counter0 }}{% else %}empty{% endif %}">
}}}
The condition for adding the "empty" suffix to the id does not work when
the object is readonly (no `has_add_permission`).
I think it should use the same condition as with the handling of the
`empty-form` class.
The empty row gets added based on this in the first place:
https://github.com/django/django/blob/f97401d1b184406d2e24f11eddbdaca8bbc360e3/django/contrib/admin/helpers.py#L299-L304
ref (added in):
https://github.com/django/django/commit/825f0beda804e48e9197fcf3b0d909f9f548aa47
#diff-3bed8380f4461efe8e7ec596937f85def59b1e38dcd6edd6e7f76c7a45d53f02
--
Ticket URL: <https://code.djangoproject.com/ticket/33679>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/010701808e7ba62b-2b661780-3317-4def-a4ce-26737b6adb4d-000000%40eu-central-1.amazonses.com.
