[Django] #36984: Prevent extremely long validation messages in inlines

[Django]

#36984: Prevent extremely long validation messages in inlines
-------------------------------------+-------------------------------------
Reporter: esperonus-karolis | Type:
| Uncategorized
Status: new | Component:
| contrib.admin
Version: 6.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
-------------------------------------+-------------------------------------
Currently Django inlines generate a DeleteProtectedModelForm with
`hand_clean_DELETE` that validates for protected items. If the amount of
protected items is large, the error message is huge as all protected
objects are listed. The message spanning thousands of entries is
impossible to read as is not structured, and nobody is going to check
thousands of items manually either. An image showing what it looks like
(excuse my locale of the message): [[Image(https://ibb.co/4RKCK5Nb)]]

Therefore, I propose to limit the amount of protected items shown in
inline validation, listing at most N items where N is subject to a
discussion, and then adding "and X more" to indicate that the list is not
extensive.

Here is a rough idea on how this could be implemented:

{{{
def hand_clean_DELETE(self):
"""
We don't validate the 'DELETE' field itself because on
templates it's not rendered using the field information, but
just using a generic "deletion_field" of the InlineModelAdmin.
"""
if self.cleaned_data.get(DELETION_FIELD_NAME, False):
using = router.db_for_write(self._meta.model)
collector = NestedObjects(using=using)
if self.instance._state.adding:
return
collector.collect([self.instance])
if collector.protected:
objs = []
for p in collector.protected[:MAX_VALIDATION_OUTPUT_ITEMS]:
objs.append(
# Translators: Model verbose name and instance
# representation, suitable to be an item in a
# list.
_("%(class_name)s %(instance)s")
% {"class_name": p._meta.verbose_name, "instance": p}
)
params = {
"class_name": self._meta.model._meta.verbose_name,
"instance": self.instance,
}
if len(collector.protected) > MAX_VALIDATION_OUTPUT_ITEMS:
params['related_objects'] = get_text_list(objs, _(", ")) +
_(' and %d more') % (len(collector.protected) -
MAX_VALIDATION_OUTPUT_ITEMS)
else:
params['related_objects'] = get_text_list(objs, _(" and
"))
msg = _(
"Deleting %(class_name)s %(instance)s would require "
"deleting the following protected related objects: "
"%(related_objects)s"
)
raise ValidationError(
msg, code="deleting_protected", params=params
)
}}}

If everyone is happy about this, I could implement the change.
--
Ticket URL: <https://code.djangoproject.com/ticket/36984>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

[Django]

#36984: Prevent extremely long validation messages in inlines
-------------------------------------+-------------------------------------

Reporter: Karolis Ryselis | Owner: Karolis
Type: | Ryselis
Cleanup/optimization | Status: assigned
Component: contrib.admin | Version: 6.0
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted

Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Changes (by Jacob Walls):

* owner: (none) => Karolis Ryselis
* stage: Unreviewed => Accepted
* status: new => assigned
* type: Uncategorized => Cleanup/optimization

Comment:

Thanks, this is similar in spirit to #10919. Thanks for offering a PR as
well.
--
Ticket URL: <https://code.djangoproject.com/ticket/36984#comment:1>