#34966: Add a check for ModelAdmin.actions functions not taking three arguments
-------------------------------------+-------------------------------------
Reporter: Riccardo | Owner: nobody
Magliocchetti |
Type: New feature | Status: closed
Component: contrib.admin | Version: dev
Severity: Normal | Resolution: wontfix
Keywords: admin actions | Triage Stage:
| Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Riccardo Magliocchetti):
* cc: Riccardo Magliocchetti (added)
Comment:
Hello Natalia, we are already unit testing the admin actions but not
through the testing client, i.e. we are testing the function and not
posting data to the model admin view. I have to say I'm not really
thrilled to start doing so for every action. I would have thought that the
check would have been in the same spirit as the other checks on the fields
though. Thanks.
--
Ticket URL: <https://code.djangoproject.com/ticket/34966#comment:3>
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 django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/0107018bcd7d50de-6d222929-aa30-486d-870e-b92cad171138-000000%40eu-central-1.amazonses.com.
