Fix our LocalSite permission wrapper on Django 1.6.
Review Request #5420 — Created Feb. 7, 2014 and submitted — Latest diff uploaded
The
LocalSite
permission wrapper broke on Django 1.6, due to
PermLookupDict
renamingmodule_name
toapp_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.