[Django] #25217: cleanup - M2M shouldn't take a 'default' kwarg

3 views
Skip to first unread message

Django

unread,
Aug 3, 2015, 9:25:17 PM8/3/15
to django-...@googlegroups.com
#25217: cleanup - M2M shouldn't take a 'default' kwarg
--------------------------------------+--------------------
Reporter: mjlock | Owner: nobody
Type: Cleanup/optimization | Status: new
Component: Core (Other) | Version: 1.8
Severity: Normal | Keywords:
Triage Stage: Unreviewed | Has patch: 0
Easy pickings: 0 | UI/UX: 0
--------------------------------------+--------------------
Many to Many field shouldn't take a 'default' kwarg as it doesn't use it.
Should raise a warning if one is given.

--
Ticket URL: <https://code.djangoproject.com/ticket/25217>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

unread,
Aug 3, 2015, 9:25:43 PM8/3/15
to django-...@googlegroups.com
#25217: cleanup - M2M shouldn't take a 'default' kwarg
-------------------------------------+-------------------------------------
Reporter: mjlock | Owner: mjlock
Type: | Status: assigned
Cleanup/optimization |

Component: Core (Other) | Version: 1.8
Severity: Normal | Resolution:
Keywords: | Triage Stage:
| Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by mjlock):

* owner: nobody => mjlock
* needs_docs: => 0
* status: new => assigned
* needs_tests: => 0
* needs_better_patch: => 0


--
Ticket URL: <https://code.djangoproject.com/ticket/25217#comment:1>

Django

unread,
Aug 3, 2015, 11:22:43 PM8/3/15
to django-...@googlegroups.com
#25217: cleanup - M2M shouldn't take a 'default' kwarg
-------------------------------------+-------------------------------------
Reporter: mjlock | Owner: mjlock
Type: | Status: assigned
Cleanup/optimization |
Component: Core (Other) | Version: 1.8
Severity: Normal | Resolution:
Keywords: | Triage Stage:
| Unreviewed

Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by mjlock):

submitted pull request for this ticket:
https://github.com/django/django/pull/5098

--
Ticket URL: <https://code.djangoproject.com/ticket/25217#comment:2>

Django

unread,
Aug 4, 2015, 6:57:12 AM8/4/15
to django-...@googlegroups.com
#25217: cleanup - M2M shouldn't take a 'default' kwarg
-------------------------------------+-------------------------------------
Reporter: mjlock | Owner: mjlock
Type: | Status: closed
Cleanup/optimization |

Component: Core (Other) | Version: 1.8
Severity: Normal | Resolution: duplicate
Keywords: | Triage Stage:
| Unreviewed

Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by timgraham):

* status: assigned => closed
* resolution: => duplicate


Comment:

Duplicate of #2750. Adding a check as done here was tried there, but
doesn't work.

--
Ticket URL: <https://code.djangoproject.com/ticket/25217#comment:3>

Reply all
Reply to author
Forward
0 new messages