Bugfix in admin forms using PermissionInline as inlines.

jazzband / django-authority

A Django app that provides generic per-object-permissions for Django's auth app and helpers to create custom permission checks.

http://django-authority.readthedocs.org

BSD 3-Clause "New" or "Revised" License

292 stars 57 forks source link

Bugfix in admin forms using PermissionInline as inlines. #2

Closed rmoorman closed 10 years ago

rmoorman commented 11 years ago

Hello,

This is basically the same fix as suggested by the following pull-request on bitbucket:

https://bitbucket.org/jezdez/django-authority/pull-request/1/bugfix-in-admin-forms-using

by Ratnadeep Debnath

It really is an issue because the modeladmins which want to use the inline are rendered useless this way.

I hope this fix can be incorporated and released to pypi soon.

Thank you very much in advance,

Best regards,

Rico

winhamwr commented 10 years ago

Hi Rico,

Thanks so much for the pull request and I'm sorry that I haven't responded here until now.

I'm going to go ahead and close the pull request, as the same fix was submitted as #11 and we need to get some kind of simple unit test on this to include it. I'll definitely add you to the AUTHORS file once that is merged in though.

Thanks again

-Wes

rmoorman commented 10 years ago

Hello Wes,

Thank you for taking the time to incorporate the change and I am glad to hear that this issue will be taken care of! Keep up the good work

Best regards,

Rico