More updates for Django

is_authenticated is no longer a function

Author: psvenk

analytics/request_counter.py

@@ -43,7 +43,7 @@ def process_response(self, request, response):
         if len(user_agent) > 150:
             user_agent = user_agent[:150]
         tally.user_agent = user_agent
-        if hasattr(request, "user") and request.user and request.user.is_authenticated():
+        if hasattr(request, "user") and request.user and request.user.is_authenticated:
             tally.is_authenticated = True
             try:
                 student = request.user.student

common/decorators.py

@@ -28,7 +28,7 @@ def view_or_basicauth(view, request, test_func, realm = "", *args, **kwargs):
     and returning the view if all goes well, otherwise responding with a 401.
     """
 
-    if request.user is None or not request.user.is_authenticated() or not user_has_student(request.user) or ALWAYS_LOGIN:
+    if request.user is None or not request.user.is_authenticated or not user_has_student(request.user) or ALWAYS_LOGIN:
         key = 'HTTP_AUTHORIZATION'
         if key not in request.META:
             key = 'REDIRECT_HTTP_AUTHORIZATION'
@@ -101,7 +101,7 @@ def wrapper(request, *args, **kwargs):
         if request.method == 'OPTIONS':
             return HttpResponse()
         return view_or_basicauth(func, request,
-                                 lambda u: u.is_authenticated(),
+                                 lambda u: u.is_authenticated,
                                  realm, *args, **kwargs)
     return wrapper
 

requirements/editor.py

@@ -87,7 +87,7 @@ def save_change_request(form, type, list_id="", committed=False):
 
 def is_staff(request):
     """Returns whether or not the request's user is an authenticated staff member."""
-    return request.user is not None and request.user.is_staff and request.user.is_authenticated()
+    return request.user is not None and request.user.is_staff and request.user.is_authenticated
 
 def populate_initial_text(request, params, edit_req):
     params['initial_text'] = edit_req.contents