Fixed issues where 'if user.is_anonymous' was missing () so was always True

wiki/decorators.py

@@ -78,14 +78,14 @@ def get_article(func=None, can_read=True, can_write=False, deleted_contents=Fals
                 return redirect(return_url)
         
         if can_read and not article.can_read(request.user):
-            if request.user.is_anonymous:
+            if request.user.is_anonymous():
                 return redirect(django_settings.LOGIN_URL)
             else:
                 pass
                 # TODO: Return a permission denied page
         
         if can_write and not article.can_write(request.user):
-            if request.user.is_anonymous:
+            if request.user.is_anonymous():
                 return redirect(django_settings.LOGIN_URL)
             else:
                 pass

wiki/plugins/attachments/views.py

@@ -35,7 +35,7 @@ class AttachmentView(ArticleMixin, FormView):
     # WARNING! The below decorator silences other exceptions that may occur!
     #@transaction.commit_manually
     def form_valid(self, form):
-        if self.request.user.is_anonymous and not settings.ANONYMOUS:
+        if self.request.user.is_anonymous() and not settings.ANONYMOUS:
             return redirect(django_settings.LOGIN_URL)
             
         try:
@@ -63,7 +63,7 @@ class AttachmentView(ArticleMixin, FormView):
         kwargs['attachments'] = self.attachments
         kwargs['search_form'] = forms.SearchForm()
         kwargs['selected_tab'] = 'attachments'
-        kwargs['anonymous_disallowed'] = self.request.user.is_anonymous and not settings.ANONYMOUS
+        kwargs['anonymous_disallowed'] = self.request.user.is_anonymous() and not settings.ANONYMOUS
         return super(AttachmentView, self).get_context_data(**kwargs)
 
 

wiki/views/article.py

@@ -59,7 +59,7 @@ class Create(FormView, ArticleMixin):
     def form_valid(self, form):
         user=None
         ip_address = None
-        if not self.request.user.is_anonymous:
+        if not self.request.user.is_anonymous():
             user = self.request.user
             if settings.LOG_IPS_USERS:
                 ip_address = self.request.META.get('REMOTE_ADDR', None)