#24664: TemporaryFileUploadHandler new_file() does not match its parent class
-------------------------------------+-------------------------------------
Reporter: ddanier | Owner: nobody
Type: | Status: new
Cleanup/optimization |
Component: Core (Other) | Version: 1.8
Severity: Normal | Resolution:
Keywords: | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 1 | UI/UX: 0
-------------------------------------+-------------------------------------
Comment (by ddanier):
> [...] but this shouldn't lead to problems as the parameter is not used
It does lead to problems if you call TemporaryFileUploadHandler.new_file()
with named parameters (kwargs). In this case you will pass mutiple values
for the same argument when calling super(). Hit me here:
https://github.com/ddanier/django-git-
lfs/blob/master/django_git_lfs/views.py#L130
(Fixed this by only passing normal args)
--
Ticket URL: <https://code.djangoproject.com/ticket/24664#comment:2>
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 post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/065.953f83620bb4a337606ccef965b595f0%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
