X-Git-Url: http://git.ithinksw.org/philo.git/blobdiff_plain/781cf86e935c66069acd17878a9f3de7cc6ec231..872b260781ffcc72f81549bc92bde8ca4eb2155e:/models/nodes.py

diff --git a/models/nodes.py b/models/nodes.py
index 23be42b..c75700c 100644
--- a/models/nodes.py
+++ b/models/nodes.py
@@ -3,14 +3,16 @@ from django.contrib.contenttypes.models import ContentType
 from django.contrib.contenttypes import generic
 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
+from django.core.urlresolvers import resolve, clear_url_caches, reverse
 from django.template import add_to_builtins as register_templatetags
 from inspect import getargspec
 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.signals import view_about_to_render, view_finished_rendering
 
 
 _view_content_type_limiter = ContentTypeSubclassLimiter(None)
@@ -31,10 +33,18 @@ class Node(TreeEntity):
 		return self.view.render_to_response(self, request, path, subpath, extra_context)
 	
 	def get_absolute_url(self):
-		root = Site.objects.get_current().root_node
 		try:
-			return '/%s' % self.get_path(root=root)
-		except AncestorDoesNotExist:
+			root = Site.objects.get_current().root_node
+		except Site.DoesNotExist:
+			root = None
+		
+		try:
+			path = self.get_path(root=root)
+			if path:
+				path += '/'
+			root_url = reverse('philo-root')
+			return '%s%s' % (root_url, path)
+		except AncestorDoesNotExist, ViewDoesNotExist:
 			return None
 	
 	class Meta:
@@ -60,6 +70,13 @@ class View(Entity):
 		return QuerySetMapper(self.relationship_set, passthrough=node.relationships)
 	
 	def render_to_response(self, node, request, path=None, subpath=None, extra_context=None):
+		extra_context = extra_context or {}
+		view_about_to_render.send(sender=self, node=node, request=request, path=path, subpath=subpath, extra_context=extra_context)
+		response = self.actually_render_to_response(node, request, path, subpath, extra_context)
+		view_finished_rendering.send(sender=self, response=response)
+		return response
+	
+	def actually_render_to_response(self, node, request, path=None, subpath=None, extra_context=None):
 		raise NotImplementedError('View subclasses must implement render_to_response.')
 	
 	class Meta:
@@ -72,9 +89,15 @@ _view_content_type_limiter.cls = View
 class MultiView(View):
 	accepts_subpath = True
 	
-	urlpatterns = []
+	@property
+	def urlpatterns(self, obj):
+		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 render_to_response(self, node, request, path=None, subpath=None, extra_context=None):
+	def actually_render_to_response(self, node, request, path=None, subpath=None, extra_context=None):
 		clear_url_caches()
 		if not subpath:
 			subpath = ""
@@ -101,7 +124,7 @@ class Redirect(View):
 	target = models.CharField(max_length=200, validators=[RedirectValidator()])
 	status_code = models.IntegerField(choices=STATUS_CODES, default=302, verbose_name='redirect type')
 	
-	def render_to_response(self, node, request, path=None, subpath=None, extra_context=None):
+	def actually_render_to_response(self, node, request, path=None, subpath=None, extra_context=None):
 		response = HttpResponseRedirect(self.target)
 		response.status_code = self.status_code
 		return response
@@ -116,7 +139,7 @@ class File(View):
 	mimetype = models.CharField(max_length=255)
 	file = models.FileField(upload_to='philo/files/%Y/%m/%d')
 	
-	def render_to_response(self, node, request, path=None, subpath=None, extra_context=None):
+	def actually_render_to_response(self, node, request, path=None, subpath=None, extra_context=None):
 		wrapper = FileWrapper(self.file)
 		response = HttpResponse(wrapper, content_type=self.mimetype)
 		response['Content-Length'] = self.file.size