But implementation of InlineModelAdmin._has_add_permission() says obj will
be required in django 3.0:
{{{
def _has_add_permission(self, request, obj):
# RemovedInDjango30Warning: obj will be a required argument.
args = get_func_args(self.has_add_permission)
return self.has_add_permission(request, obj) if 'obj' in args else
self.has_add_permission(request)
}}}
So, code will crush at **{{{self.has_add_permission(request)}}}** in
return statement with {{{missing 1 required positional argument: 'obj'}}}.
--
Ticket URL: <https://code.djangoproject.com/ticket/30097>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
* has_patch: 0 => 1
--
Ticket URL: <https://code.djangoproject.com/ticket/30097#comment:1>
* cc: Jon, Dufresne (added)
* severity: Normal => Release blocker
* stage: Unreviewed => Accepted
Comment:
This looks related to #27991. The new test case on the PR looks right and
fails without the patch, so pending review I'm going to accept this as a
Release Blocker on v2.1.
--
Ticket URL: <https://code.djangoproject.com/ticket/30097#comment:2>
* status: new => assigned
* owner: nobody => Maxim Zemskov
--
Ticket URL: <https://code.djangoproject.com/ticket/30097#comment:3>
* cc: Jon, Dufresne (removed)
* stage: Accepted => Ready for checkin
--
Ticket URL: <https://code.djangoproject.com/ticket/30097#comment:4>
* status: assigned => closed
* resolution: => fixed
Comment:
In [changeset:"3c01fe30f3dd4dc1c8bb4fec816bd277d1ae5fa6" 3c01fe30]:
{{{
#!CommitTicketReference repository=""
revision="3c01fe30f3dd4dc1c8bb4fec816bd277d1ae5fa6"
Fixed #30097 -- Made 'obj' arg of InlineModelAdmin.has_add_permission()
optional.
Restored backwards compatibility after refs #27991.
Regression in be6ca89396c031619947921c81b8795d816e3285.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/30097#comment:5>
Comment (by Tim Graham <timograham@…>):
In [changeset:"3df13847d583f0e439d03dd85e4d8d2a36e6de73" 3df13847]:
{{{
#!CommitTicketReference repository=""
revision="3df13847d583f0e439d03dd85e4d8d2a36e6de73"
[2.1.x] Fixed #30097 -- Made 'obj' arg of
InlineModelAdmin.has_add_permission() optional.
Restored backwards compatibility after refs #27991.
Regression in be6ca89396c031619947921c81b8795d816e3285.
Backport of 3c01fe30f3dd4dc1c8bb4fec816bd277d1ae5fa6 from master.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/30097#comment:6>
Comment (by Tim Graham <timograham@…>):
In [changeset:"ee9bd8c31024ec424157798b2a60a7f52f9353ee" ee9bd8c]:
{{{
#!CommitTicketReference repository=""
revision="ee9bd8c31024ec424157798b2a60a7f52f9353ee"
[2.2.x] Refs #30097 -- Fixed typos in
InlineModelAdmin.has_add_permission() deprecation comments.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/30097#comment:7>
Comment (by Tim Graham <timograham@…>):
In [changeset:"1bb31708a0b1fee1a903dacca0123205a6e3690a" 1bb31708]:
{{{
#!CommitTicketReference repository=""
revision="1bb31708a0b1fee1a903dacca0123205a6e3690a"
[2.1.x] Refs #30097 -- Fixed typos in
InlineModelAdmin.has_add_permission() deprecation comments.
Backport of ee9bd8c31024ec424157798b2a60a7f52f9353ee from stable/2.2.x.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/30097#comment:8>