Umm, the problem with late commits is that you forget what you

jschrewe · jschrewe · commit 450b702867ec · 2011-08-29T16:06:46.000+02:00
changed. This is a late commit. Sorry.
diff --git a/mongodbforms/documents.py b/mongodbforms/documents.py
@@ -74,7 +74,6 @@ def save_instance(form, instance, fields=None, fail_message='saved',
     if commit and hasattr(instance, 'save'):
         # see BaseDocumentForm._post_clean for an explanation
         if hasattr(form, '_delete_before_save'):
-            print "saving and ignoring stuff. cool, eh?"
             fields = instance._fields
             new_fields = {}
             for n, f in fields.iteritems():
@@ -84,7 +83,6 @@ def save_instance(form, instance, fields=None, fail_message='saved',
             instance.save()
             instance._fields = fields
         else:
-            print "_delete_before_save not there..."
             instance.save()
         
     return instance
@@ -305,7 +303,6 @@ def _post_clean(self):
 
         # Clean the model instance's fields.
         to_delete = []
-        print exclude
         try:
             for f in self.instance._fields.itervalues():
                 value = getattr(self.instance, f.name)
@@ -314,7 +311,7 @@ def _post_clean(self):
                 elif value == '':
                     # mongoengine chokes on empty strings for fields
                     # that are not required. Clean them up here, though
-                    # is maybe not the right place :-)
+                    # this is maybe not the right place :-)
                     to_delete.append(f.name)
         except ValidationError, e:
             err = {f.name: [e.message]}
@@ -406,31 +403,21 @@ class EmbeddedDocumentForm(BaseDocumentForm):
     
     def __init__(self, parent_document, *args, **kwargs):
         super(EmbeddedDocumentForm, self).__init__(*args, **kwargs)
-        # FIXME: Check if parent is a real object and not a type?
         self.parent_document = parent_document
         if self._meta.embedded_field is not None and \
                 not hasattr(self.parent_document, self._meta.embedded_field):
             raise FieldError("Parent document must have field %s" % self._meta.embedded_field)
         
-        # always ignore kwargs instance
-        self.instance = self._meta.document
-        
     def save(self, commit=True):
         if self.errors:
             raise ValueError("The %s could not be saved because the data didn't"
                          " validate." % self.instance.__class__.__name__)
         
         if commit:
-            print "embedded field:"
-            print self._meta.embedded_field
-            print getattr(self.parent_document, self._meta.embedded_field)
             l = getattr(self.parent_document, self._meta.embedded_field)
-            print type(self.instance)
-            print type(self)
             l.append(self.instance)
             setattr(self.parent_document, self._meta.embedded_field, l)
-            print getattr(self.parent_document, self._meta.embedded_field)
-            self.parent_document.save()
+            self.parent_document.save() 
         
         return self.instance
 
@@ -439,7 +426,6 @@ class BaseDocumentFormSet(BaseFormSet):
     """
     A ``FormSet`` for editing a queryset and/or adding new objects to it.
     """
-    #model = None
 
     def __init__(self, data=None, files=None, auto_id='id_%s', prefix=None,
                  queryset=None, **kwargs):
diff --git a/readme.md b/readme.md
@@ -1,49 +1,18 @@
-# django mongoforms
+# django mongodbforms
 
-This is an implementation of django's model forms that uses mongoengine documents.
+This is an implementation of django's model forms for mongoengine documents.
 
 ## Requirements
 
  * mongoengine
 
 ## Usage
 
-Add mongoadmin to `INSTALLED_APPS` settings
 
-	INSTALLED_APPS = (
-		...
-    	'mongoadmin',
-    	'django.contrib.admin',
-		...
-	)
-
-Add mongoadmin to `urls.py`
-
-	from django.contrib import admin
-	admin.autodiscover()
-
-	from mongoadmin import site
-
-	urlpatterns = patterns('',
-    	# Uncomment the next line to enable the admin:
-    	url(r'^admin/', include(site.urls)),
-	)
-
-The `admin.py` for your app needs to use mongoadmin instead of django's admin:
-
-	from mongoadmin import site, DocumentAdmin
-
-	from app.models import AppDocument
-	
-	class AppDocumentAdmin(DocumentAdmin):
-	    pass
-	site.register(AppDocument, AppDocumentAdmin)
-	
-Now the document should appear as usual in django's admin.
 
 ## What works and doesn't work
 
-django-mongoadmin currently only supports the most basic things and even they are not really tested.
+django-mongodbforms currently only supports the most basic things and even they are not really tested.
 
 Changelists only support basic listings you probably won't be able to use fieldlists and every other feature that django supports for changelists (search, etc.).
 
