Moved the EventQuerySet to its own class instead of nesting it in the Event model.

[philo.git] / views.py

diff --git a/views.py b/views.py
index ab4c216..f5a2c7f 100644 (file)
--- a/views.py
+++ b/views.py
@@ -1,57 +1,60 @@
-from django.contrib.sites.models import Site

 from django.conf import settings
 from django.conf import settings

-from django.http import Http404, HttpResponse
-from django.template import RequestContext
+from django.core.urlresolvers import resolve
+from django.http import Http404, HttpResponseRedirect

 from django.views.decorators.vary import vary_on_headers
 from django.views.decorators.vary import vary_on_headers

-from philo.models import Node
+from philo.exceptions import MIDDLEWARE_NOT_CONFIGURED

 
 
 @vary_on_headers('Accept')
 def node_view(request, path=None, **kwargs):
 
 
 @vary_on_headers('Accept')
 def node_view(request, path=None, **kwargs):

-       node = None
-       subpath = None
-       if path is None:
-               path = '/'
-       current_site = Site.objects.get_current()
-       try:
-               node, subpath = Node.objects.get_with_path(path, root=current_site.root_node, absolute_result=False)
-       except Node.DoesNotExist:
-               raise Http404
+       if "philo.middleware.RequestNodeMiddleware" not in settings.MIDDLEWARE_CLASSES:
+               raise MIDDLEWARE_NOT_CONFIGURED

        
        

-       if not node:
+       if not request.node:
+               if settings.APPEND_SLASH and request.path != "/":
+                       path = request.path
+                       
+                       if path[-1] == "/":
+                               path = path[:-1]
+                       else:
+                               path += "/"
+                       
+                       view, args, kwargs = resolve(path)
+                       if view != node_view:
+                               return HttpResponseRedirect(path)

                raise Http404
        
                raise Http404
        

-       try:
-               if subpath and not node.accepts_subpath:
+       node = request.node
+       subpath = request.node.subpath
+       
+       # Explicitly disallow trailing slashes if we are otherwise at a node's url.
+       if request.path and request.path != "/" and request.path[-1] == "/" and subpath == "/":
+               return HttpResponseRedirect(node.get_absolute_url())
+       
+       if not node.handles_subpath(subpath):
+               # If the subpath isn't handled, check settings.APPEND_SLASH. If
+               # it's True, try to correct the subpath.
+               if not settings.APPEND_SLASH:

                        raise Http404
                        raise Http404

-               return node.render_to_response(request, path=path, subpath=subpath)
-       except Http404, e:
-               if settings.DEBUG:
-                       raise e

                
                

-               try:
-                       Http404View = node.relationships['Http404']
-               except KeyError:
-                       Http404View = None
+               if subpath[-1] == "/":
+                       subpath = subpath[:-1]
+               else:
+                       subpath += "/"

                
                

-               if not Http404View:
-                       raise e
+               redirect_url = node.construct_url(subpath)

                
                

-               extra_context = {'exception': e}
+               if node.handles_subpath(subpath):
+                       return HttpResponseRedirect(redirect_url)

                
                

-               return Http404View.render_to_response(node, request, path, subpath, extra_context)
-       except Exception, e:
-               if settings.DEBUG:
-                       raise e
+               # Perhaps there is a non-philo view at this address. Can we
+               # resolve *something* there besides node_view? If not,
+               # raise a 404.
+               view, args, kwargs = resolve(redirect_url)

                
                

-               try:
-                       Http500View = node.relationships['Http500']
-                       
-                       if not Http500View:
-                               raise e
-                       
-                       extra_context = {'exception': e}
-                       
-                       return Http500View.render_to_response(node, request, path, subpath, extra_context)
-               except:
-                       raise e
\ No newline at end of file
+               if view == node_view:
+                       raise Http404
+               else:
+                       return HttpResponseRedirect(redirect_url)
+       
+       return node.render_to_response(request, kwargs)
\ No newline at end of file