X-Git-Url: http://git.ithinksw.org/philo.git/blobdiff_plain/f314a8ddceb543e2bb4711d50bdd2060452689b1..46d04058ae4f5a7fa50d76c28f328c972466743f:/contrib/gilbert/plugins/models.py

diff --git a/contrib/gilbert/plugins/models.py b/contrib/gilbert/plugins/models.py
index b765482..e3d559a 100644
--- a/contrib/gilbert/plugins/models.py
+++ b/contrib/gilbert/plugins/models.py
@@ -1,15 +1,16 @@
-import operator
-from django.forms.models import ModelForm, modelform_factory
+from django.conf import settings
+from django.contrib.admin.util import lookup_field, label_for_field, display_for_field, NestedObjects
+from django.core.exceptions import PermissionDenied
 from django.db.models import Q
 from django.db.models.fields.related import ManyToOneRel
 from django.db.models.fields.files import FieldFile, ImageFieldFile, FileField
-from django.contrib.admin.util import lookup_field, label_for_field, display_for_field, NestedObjects
+from django.forms.models import ModelForm, modelform_factory
+from django.template.defaultfilters import capfirst
+from django.utils import simplejson as json
 from django.utils.encoding import smart_unicode
-import staticmedia
 from .base import Plugin
 from ..extdirect import ext_action, ext_method
-from django.core.exceptions import PermissionDenied
-from django.utils import simplejson as json
+import operator, staticmedia
 
 
 @ext_action(name='models')
@@ -64,7 +65,7 @@ class ModelAdmin(Plugin):
 	def data_serialize_model_instance(cls, obj):
 		return {
 			'app_label': obj._meta.app_label,
-			'name': obj._meta.object_name,
+			'name': obj._meta.module_name,
 			'pk': obj.pk,
 			'__unicode__': unicode(obj),
 		}
@@ -126,7 +127,7 @@ class ModelAdmin(Plugin):
 				'editable': False,
 			}
 			header, attr = label_for_field(field_name, self.model, model_admin=self, return_attr=True)
-			column['header'] = header
+			column['header'] = capfirst(header)
 			if (field_name in self.sortable_fields) or (getattr(attr, 'admin_order_field', None) in self.sortable_fields):
 				column['sortable'] = True
 			if field_name in self.data_editable_columns:
@@ -170,17 +171,20 @@ class ModelAdmin(Plugin):
 			raise PermissionDenied
 		return self.model._default_manager.all()
 	
+	def queryset(self, request):
+		return self.model._default_manager.get_query_set()
+	
 	@ext_method
 	def filter(self, request, **kwargs):
 		if not self.has_read_permission(request):
 			raise PermissionDenied
-		return self.model._default_manager.all().filter(**kwargs)
+		return self.queryset(request).filter(**kwargs)
 	
 	@ext_method
 	def get(self, request, **kwargs):
 		if not self.has_read_permission(request):
 			raise PermissionDenied
-		return self.model._default_manager.all().values().get(**kwargs)
+		return self.queryset(request).values().get(**kwargs)
 	
 	@property
 	def form_class(self):
@@ -215,10 +219,10 @@ class ModelAdmin(Plugin):
 		
 		form = self.form_class(request.POST, request.FILES, instance=instance)
 		
-		try:
+		if form.is_valid():
 			saved = form.save()
 			return True, None, saved.pk
-		except ValueError:
+		else:
 			return False, form.errors
 	
 	def data_serialize_object(self, obj):