Opened 17 years ago

Closed 17 years ago

#5457 closed (fixed)

Move permissions checking to auth backend

Reported by: Florian Apolloner Owned by: Jacob
Component: Contrib apps Version: newforms-admin
Severity: Keywords: backend permissions sprintsept14
Cc: django@… Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

The included patch moves permission checking to the auth_backends as I am using an ldap server to auth against this would become handy.

The patch is against newforms-admin, but should work in trunk too...

Attachments (6)

backend_user_permissions.diff (144 bytes ) - added by Florian Apolloner 17 years ago.
backend_user_permissions.2.diff (7.5 KB ) - added by Florian Apolloner 17 years ago.
Upps, old one was empty…
doc.diff (1.5 KB ) - added by Florian Apolloner 17 years ago.
I added some docs. Please apologize my bad English.
full_diff (9.0 KB ) - added by Florian Apolloner 17 years ago.
full diff including docs and tests (I hope everything works, as I am really tired now XD)
full_diff.diff (9.0 KB ) - added by Florian Apolloner 17 years ago.
full diff including docs and tests (I hope everything works, as I am really tired now XD)
full_diff.2.diff (11.8 KB ) - added by Florian Apolloner 17 years ago.
full diff including docs and tests (I hope everything works, as I am really tired now XD) [last one missed the tests...]

Download all attachments as: .zip

Change History (12)

by Florian Apolloner, 17 years ago

by Florian Apolloner, 17 years ago

Upps, old one was empty...

comment:1 by Florian Apolloner, 17 years ago

Needs documentation: set
Owner: changed from nobody to Florian Apolloner
Status: newassigned
Triage Stage: UnreviewedDesign decision needed

I'll start writing the docu, and set this to "design decision needed" for now...

comment:2 by anonymous, 17 years ago

Keywords: sprintsept14 added

by Florian Apolloner, 17 years ago

Attachment: doc.diff added

I added some docs. Please apologize my bad English.

comment:3 by Florian Apolloner, 17 years ago

Needs documentation: unset

by Florian Apolloner, 17 years ago

Attachment: full_diff added

full diff including docs and tests (I hope everything works, as I am really tired now XD)

by Florian Apolloner, 17 years ago

Attachment: full_diff.diff added

full diff including docs and tests (I hope everything works, as I am really tired now XD)

by Florian Apolloner, 17 years ago

Attachment: full_diff.2.diff added

full diff including docs and tests (I hope everything works, as I am really tired now XD) [last one missed the tests...]

comment:4 by Florian Apolloner, 17 years ago

Owner: changed from Florian Apolloner to Jacob
Status: assignednew
Triage Stage: Design decision neededReady for checkin

Assigning to jacob and switching to ready for checkin, he is going to review it this weekend :)

-- Florian Apolloner <florian at apolloner.eu>

comment:5 by Florian Apolloner, 17 years ago

The diff file includes the empty init.py but it is not getting applied during patching (and I really don't know why), sry jacob

comment:6 by Jacob, 17 years ago

Resolution: fixed
Status: newclosed

Fixed in [6375].

Note: See TracTickets for help on using tickets.
Back to Top