Merge lp://staging/~bhavesh-goyal093/postorius/fixed-postorius-method-decorators into lp://staging/postorius

Proposed by Bhavesh Goyal
Status: Merged
Merged at revision: 221
Proposed branch: lp://staging/~bhavesh-goyal093/postorius/fixed-postorius-method-decorators
Merge into: lp://staging/postorius
Diff against target: 38 lines (+9/-4)
2 files modified
src/postorius/auth/decorators.py (+8/-3)
src/postorius/templates/postorius/menu/list_nav.html (+1/-1)
To merge this branch: bzr merge lp://staging/~bhavesh-goyal093/postorius/fixed-postorius-method-decorators
Reviewer Review Type Date Requested Status
Terri Approve
Review via email: mp+256037@code.staging.launchpad.net

Description of the change

Fixed Bug #1435062 which caused postorius to show incorrect nav bar view for members posessing different roles.
The solution was found in correcting permissions for list_nav template and fixing method decorators for list_moderator_required which earlier didn't set is_list_owner to be True even if the user was the owner and as a result the view was shown to be incorrect.
Now the attributes are set correctly according to the roles and the nav bar sows correct view or different and multiple roles.

To post a comment you must log in.
Revision history for this message
Terri (terriko) wrote :

Whoops! Looks like you used tabs instead of spaces. I'll fix it when I do the merge, assuming my tests pass.

Revision history for this message
Bhavesh Goyal (bhavesh-goyal093) wrote :

Sorry for the tabs! I ll take care of them next time onwards...

Revision history for this message
Terri (terriko) wrote :

The update seems to have fixed the test errors I saw earlier; thanks!

Revision history for this message
Terri (terriko) wrote :

Also, it would be awesome if I had some tests for this behaviour. I'll open a separate bug for this.

Revision history for this message
Terri (terriko) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches