Fix our LocalSite permission wrapper on Django 1.6.

Review Request #5420 — Created Feb. 7, 2014 and submitted

Information

Review Board
master

Reviewers

The LocalSite permission wrapper broke on Django 1.6, due to
PermLookupDict renaming module_name to app_label.

This updates our calls to use the new name. It also adds unit tests to
ensure we catch any breakages in the future.

Ran just the unit tests on the master branch, without the fix, and got
the same errors that were silently triggering our permission lookup
failures.

Ran with the fix, and the tests passed. I was also able to choose
Delete Permanently, and close another user's review requests.

david
  1. Ship It!

  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Loading...