Cleanup: move legalpage prefixes to constant

There's not much use in having this as a setting and actually the
approach should be revisited altogether.

Refs. #318.

CHANGES.md

@@ -4,6 +4,7 @@ Zing Changelog
 v.next (not released yet)
 -------------------------
 
+* Moved `LEGALPAGE_NOCHECK_PREFIXES` into a constant.
 * Removed captcha middleware and associated setting.
 * Removed `ZING_INSTANCE_ID`. This can easily be implemented by anyone.
 * Cleaned up setting names and removed obsoletes:

docs/ref-settings.md

@@ -48,16 +48,6 @@ Controls whether user sign ups are allowed or not. If set to `False`,
 administrators will still be able to create new user accounts.
 
 
-### `ZING_LEGALPAGE_NOCHECK_PREFIXES`
-
-Default: `('/about', '/accounts', '/admin', '/contact', '/jsi18n', '/pages', )`
-
-List of path prefixes where the `LegalAgreementMiddleware` will check
-if the current logged-in user has agreed all the legal documents defined
-for the Zing instance. Don't change this unless you know what you're
-doing.
-
-
 ### `ZING_REPORTS_MARK_FUNC`
 
 Default: _""_ (unset)

pootle/apps/staticpages/context_processors.py

@@ -6,16 +6,24 @@
 # or later license. See the LICENSE file for a copy of the license and the
 # AUTHORS file for copyright and authorship information.
 
-from django.conf import settings
-
 from .models import LegalPage
 
 
+NOCHECK_PREFIXES = (
+    '/about',
+    '/accounts',
+    '/admin',
+    '/contact',
+    '/jsi18n',
+    '/pages',
+    '/xhr',
+)
+
+
 def agreement(request):
     """Returns whether the agreement box should be displayed or not."""
     request_path = request.META['PATH_INFO']
-    nocheck = filter(lambda x: request_path.startswith(x),
-                     settings.ZING_LEGALPAGE_NOCHECK_PREFIXES)
+    nocheck = filter(lambda x: request_path.startswith(x), NOCHECK_PREFIXES)
 
     display_agreement = False
     if (request.user.is_authenticated and not nocheck and

pootle/settings/40-apps.conf

@@ -1,26 +1,6 @@
 # -*- coding: utf-8 -*-
 
-"""Configuration settings for applications used by Pootle."""
-
-#
-# Pootle
-#
-
-
-# Legal Page Agreements
-#
-# List of paths prefixes where no checks for legal document agreements
-# will be enforced
-ZING_LEGALPAGE_NOCHECK_PREFIXES = (
-    '/about',
-    '/accounts',
-    '/admin',
-    '/contact',
-    '/jsi18n',
-    '/pages',
-    '/xhr',
-)
-
+"""Configuration settings for applications."""
 
 #
 # webassets