X-Git-Url: http://git.ithinksw.org/philo.git/blobdiff_plain/e18b86f47054850e9f49c1cb7aaecd8fdf753e4d..806ef92a88cc844c47e59220ada96477a4ce601a:/admin.py?ds=sidebyside

diff --git a/admin.py b/admin.py
index 178a904..9e991d9 100644
--- a/admin.py
+++ b/admin.py
@@ -8,6 +8,11 @@ from django.utils.safestring import mark_safe
 from django.utils.html import escape
 from django.utils.text import truncate_words
 from philo.models import *
+from django.core.exceptions import ValidationError, ObjectDoesNotExist
+from validators import TreeParentValidator, TreePositionValidator
+
+
+COLLAPSE_CLASSES = ('collapse', 'collapse-closed', 'closed',)
 
 
 class AttributeInline(generic.GenericTabularInline):
@@ -37,12 +42,14 @@ class CollectionMemberInline(admin.TabularInline):
 	fk_name = 'collection'
 	model = CollectionMember
 	extra = 1
-	classes = ('collapse-closed',)
+	classes = COLLAPSE_CLASSES
 	allow_add = True
+	fields = ('member_content_type', 'member_object_id', 'index',)
 
 
 class CollectionAdmin(admin.ModelAdmin):
 	inlines = [CollectionMemberInline]
+	list_display = ('name', 'description', 'get_count')
 
 
 class NodeAdmin(EntityAdmin):
@@ -76,16 +83,77 @@ class ModelLookupWidget(forms.TextInput):
 		return mark_safe(output)
 
 
+class TreeForm(forms.ModelForm):
+	def __init__(self, *args, **kwargs):
+		super(TreeForm, self).__init__(*args, **kwargs)
+		instance = self.instance
+		instance_class = self.get_instance_class()
+		
+		if instance_class is not None:
+			try:
+				self.fields['parent'].queryset = instance_class.objects.exclude(id=instance.id)
+			except ObjectDoesNotExist:
+				pass
+			
+		self.fields['parent'].validators = [TreeParentValidator(*self.get_validator_args())]
+	
+	def get_instance_class(self):
+		return self.instance.__class__
+		
+	def get_validator_args(self):
+		return [self.instance]
+	
+	def clean(self):
+		cleaned_data = self.cleaned_data
+		
+		try:
+			parent = cleaned_data['parent']
+			slug = cleaned_data['slug']
+			obj_class = self.get_instance_class()
+			tpv = TreePositionValidator(parent, slug, obj_class)
+			tpv(self.instance)
+		except KeyError:
+			pass
+		
+		return cleaned_data
+
+
+class NodeForm(TreeForm):
+	def get_instance_class(self):
+		return Node
+		
+	def get_validator_args(self):
+		return [self.instance, 'instance']
+
+
+class PageAdminForm(NodeForm):
+	class Meta:
+		model = Page
+
+
+class RedirectAdminForm(NodeForm):
+	class Meta:
+		model = Redirect
+
+
+class FileAdminForm(NodeForm):
+	class Meta:
+		model = File
+
+
 class RedirectAdmin(NodeAdmin):
 	fieldsets = (
 		(None, {
 			'fields': ('slug', 'target', 'status_code')
 		}),
 		('URL/Tree/Hierarchy', {
-			'classes': ('collapse', 'collapse-closed'),
+			'classes': COLLAPSE_CLASSES,
 			'fields': ('parent',)
 		}),
 	)
+	list_display=('slug', 'target', 'path', 'status_code',)
+	list_filter=('status_code',)
+	form = RedirectAdminForm
 
 
 class FileAdmin(NodeAdmin):
@@ -95,10 +163,12 @@ class FileAdmin(NodeAdmin):
 			'fields': ('file', 'slug', 'mimetype')
 		}),
 		('URL/Tree/Hierarchy', {
-			'classes': ('collapse', 'collapse-closed'),
+			'classes': COLLAPSE_CLASSES,
 			'fields': ('parent',)
 		}),
 	)
+	form=FileAdminForm
+	list_display=('slug', 'mimetype', 'path', 'file',)
 
 
 class PageAdmin(NodeAdmin):
@@ -109,13 +179,14 @@ class PageAdmin(NodeAdmin):
 			'fields': ('title', 'slug', 'template')
 		}),
 		('URL/Tree/Hierarchy', {
-			'classes': ('collapse', 'collapse-closed'),
+			'classes': COLLAPSE_CLASSES,
 			'fields': ('parent',)
 		}),
 	)
 	list_display = ('title', 'path', 'template')
 	list_filter = ('template',)
 	search_fields = ['title', 'slug', 'contentlets__content']
+	form = PageAdminForm
 	
 	def get_fieldsets(self, request, obj=None, **kwargs):
 		fieldsets = list(self.fieldsets)
@@ -157,7 +228,7 @@ class PageAdmin(NodeAdmin):
 				initial_content = None
 				try:
 					initial_content = page.contentreferences.get(name__exact=container_name, content_type=container_content_type).content.pk
-				except ContentReference.DoesNotExist:
+				except (ContentReference.DoesNotExist, AttributeError):
 					pass
 				form.base_fields[('contentreference_container_%s' % container_name)] = forms.ModelChoiceField(label='References', widget=ModelLookupWidget(container_content_type), initial=initial_content, required=False, queryset=container_content_type.model_class().objects.all())
 		return form
@@ -178,10 +249,17 @@ class PageAdmin(NodeAdmin):
 		for container_name, container_content_type in contentreference_containers:
 			if ('contentreference_container_%s' % container_name) in form.cleaned_data:
 				content = form.cleaned_data[('contentreference_container_%s' % container_name)]
-				contentreference, created = page.contentreferences.get_or_create(name=container_name, defaults={'content': content})
-				if not created:
-					contentreference.content = content
-					contentreference.save()
+				try:
+					contentreference = page.contentreferences.get(name=container_name)
+				except ContentReference.DoesNotExist:
+					contentreference = ContentReference(name=container_name, page=page, content_type=container_content_type)
+				
+				if content == None:
+					contentreference.content_id = None
+				else:
+					contentreference.content_id = content.id
+				
+				contentreference.save()
 
 
 class TemplateAdmin(admin.ModelAdmin):
@@ -191,19 +269,21 @@ class TemplateAdmin(admin.ModelAdmin):
 			'fields': ('parent', 'name', 'slug')
 		}),
 		('Documentation', {
-			'classes': ('collapse', 'collapse-closed'),
+			'classes': COLLAPSE_CLASSES,
 			'fields': ('documentation',)
 		}),
 		(None, {
 			'fields': ('code',)
 		}),
 		('Advanced', {
-			'classes': ('collapse','collapse-closed'),
+			'classes': COLLAPSE_CLASSES,
 			'fields': ('mimetype',)
 		}),
 	)
 	save_on_top = True
 	save_as = True
+	list_display = ('__unicode__', 'slug', 'get_path',)
+	form = TreeForm
 
 
 admin.site.register(Collection, CollectionAdmin)