Opened 12 years ago

Closed 12 years ago

#2664 closed defect (fixed)

[per-object-permissions] pop admin broken (2)

Reported by: Joeboy Owned by: Chris Long
Component: contrib.admin Version:
Severity: normal Keywords: per-object-permissions, row-level
Cc: django25@… Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:


This snippet from looks wrong:

        if is
            user_rlp_form_list.append({'form':forms.FormWrapper(edit_rlp_manip, data, rlp_errors), 'rlp':r})
        elif is
            group_rlp_form_list.append({'form':forms.FormWrapper(edit_rlp_manip, data, rlp_errors), 'rlp':r})            
            other_rlp_form_list.append({'form':forms.FormWrapper(edit_rlp_manip, data, rlp_errors), 'rlp':r})

as the ifs never evaluate to true. If I change the is's to =='s then they do evaluate to true when expected.

Change History (3)

comment:1 Changed 12 years ago by anonymous

Cc: django25@… added

comment:2 Changed 12 years ago by anonymous

Owner: changed from Adrian Holovaty to Chris Long

comment:3 Changed 12 years ago by Chris Long

Resolution: fixed
Status: newclosed

(In [3731]) [per-object-permissions] Fixes #2664 and #2663

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