X-Git-Url: http://git.ithinksw.org/philo.git/blobdiff_plain/1dba84d30c4323acb722de2011ff7a0bff9148aa..048812e5eaeff5eb9ba3b1877cdb9fb3a84662aa:/models/nodes.py?ds=inline

diff --git a/models/nodes.py b/models/nodes.py
index b16521b..526400c 100644
--- a/models/nodes.py
+++ b/models/nodes.py
@@ -5,14 +5,14 @@ from django.contrib.sites.models import Site
 from django.http import HttpResponse, HttpResponseServerError, HttpResponseRedirect
 from django.core.exceptions import ViewDoesNotExist
 from django.core.servers.basehttp import FileWrapper
-from django.core.urlresolvers import resolve, clear_url_caches, reverse
+from django.core.urlresolvers import resolve, clear_url_caches, reverse, NoReverseMatch
 from django.template import add_to_builtins as register_templatetags
 from inspect import getargspec
 from philo.exceptions import MIDDLEWARE_NOT_CONFIGURED
 from philo.models.base import TreeEntity, Entity, QuerySetMapper, register_value_model
 from philo.utils import ContentTypeSubclassLimiter
 from philo.validators import RedirectValidator
-from philo.exceptions import ViewDoesNotProvideSubpaths, AncestorDoesNotExist
+from philo.exceptions import ViewCanNotProvideSubpath, ViewDoesNotProvideSubpaths, AncestorDoesNotExist
 from philo.signals import view_about_to_render, view_finished_rendering
 
 
@@ -62,7 +62,18 @@ class View(Entity):
 	accepts_subpath = False
 	
 	def get_subpath(self, obj):
-		raise ViewDoesNotProvideSubpaths
+		if not self.accepts_subpath:
+			raise ViewDoesNotProvideSubpaths
+		
+		view_name, args, kwargs = self.get_reverse_params(obj)
+		try:
+			return reverse(view_name, args=args, kwargs=kwargs, urlconf=self)
+		except NoReverseMatch:
+			raise ViewCanNotProvideSubpath
+	
+	def get_reverse_params(self, obj):
+		"""This method should return a view_name, args, kwargs tuple suitable for reversing a url for the given obj using self as the urlconf."""
+		raise NotImplementedError("View subclasses must implement get_reverse_params to support subpaths.")
 	
 	def attributes_with_node(self, node):
 		return QuerySetMapper(self.attribute_set, passthrough=node.attributes)
@@ -78,7 +89,7 @@ class View(Entity):
 		return response
 	
 	def actually_render_to_response(self, request, extra_context=None):
-		raise NotImplementedError('View subclasses must implement render_to_response.')
+		raise NotImplementedError('View subclasses must implement actually_render_to_response.')
 	
 	class Meta:
 		abstract = True
@@ -91,13 +102,9 @@ class MultiView(View):
 	accepts_subpath = True
 	
 	@property
-	def urlpatterns(self, obj):
+	def urlpatterns(self):
 		raise NotImplementedError("MultiView subclasses must implement urlpatterns.")
 	
-	def get_reverse_params(self, obj):
-		"""This method should return a view_name, args, kwargs tuple suitable for reversing a url for the given obj using self as the urlconf."""
-		raise NotImplementedError("MultiView subclasses must implement get_subpath.")
-	
 	def actually_render_to_response(self, request, extra_context=None):
 		clear_url_caches()
 		subpath = request.node.subpath
@@ -112,6 +119,39 @@ class MultiView(View):
 			kwargs['extra_context'] = extra_context
 		return view(request, *args, **kwargs)
 	
+	def reverse(self, view_name, args=None, kwargs=None, node=None):
+		"""Shortcut method to handle the common pattern of getting the absolute url for a multiview's
+		subpaths."""
+		subpath = reverse(view_name, urlconf=self, args=args or [], kwargs=kwargs or {})
+		if node is not None:
+			return '/%s/%s/' % (node.get_absolute_url().strip('/'), subpath.strip('/'))
+		return subpath
+	
+	def get_context(self):
+		"""Hook for providing instance-specific context - such as the value of a Field - to all views."""
+		return {}
+	
+	def basic_view(self, field_name):
+		"""
+		Given the name of a field on ``self``, accesses the value of
+		that field and treats it as a ``View`` instance. Creates a
+		basic context based on self.get_context() and any extra_context
+		that was passed in, then calls the ``View`` instance's
+		render_to_response() method. This method is meant to be called
+		to return a view function appropriate for urlpatterns.
+		"""
+		field = self._meta.get_field(field_name)
+		view = getattr(self, field.name, None)
+		
+		def inner(request, extra_context=None, **kwargs):
+			if not view:
+				raise Http404
+			context = self.get_context()
+			context.update(extra_context or {})
+			return view.render_to_response(request, extra_context=context)
+		
+		return inner
+	
 	class Meta:
 		abstract = True
 
@@ -133,7 +173,6 @@ class Redirect(View):
 		app_label = 'philo'
 
 
-# Why does this exist?
 class File(View):
 	""" For storing arbitrary files """