Changeset 5409

Timestamp:

06/01/07 08:39:08 (1 year ago)

Author:

russellm

Message:

Fixed #3466 -- Fixed problem with specifyin a 'fields' argument to a JSON serializer. Also added documenation for the 'fields' argument.

Files:

• django/trunk/django/core/serializers/json.py (modified) (1 diff)
• django/trunk/django/core/serializers/pyyaml.py (modified) (1 diff)
• django/trunk/docs/serialization.txt (modified) (2 diffs)
• django/trunk/tests/modeltests/serializers/models.py (modified) (1 diff)
• django/trunk/tests/regressiontests/serializers_regress/models.py (modified) (1 diff)
• django/trunk/tests/regressiontests/serializers_regress/tests.py (modified) (2 diffs)

django/trunk/django/core/serializers/json.py

@@ -22 +22 @@
-    """
-    def end_serialization(self):
+        self.options.pop('stream', None)
+        self.options.pop('fields', None)
-        simplejson.dump(self.objects, self.stream, cls=DjangoJSONEncoder, **self.options)
-

django/trunk/django/core/serializers/pyyaml.py

@@ -19 +19 @@
-    """
-    def end_serialization(self):
+        self.options.pop('stream', None)
+        self.options.pop('fields', None)
-        yaml.dump(self.objects, self.stream, **self.options)
-        

django/trunk/docs/serialization.txt

@@ -44 +44 @@
-
-.. _HTTPResponse: ../request_response/#httpresponse-objects
+
+Subset of fields
+~~~~~~~~~~~~~~~~
+
+If you only want a subset of fields to be serialized, you can 
+specify a `fields` argument to the serializer::
+
+    from django.core import serializers
+    data = serializers.serialize('xml', SomeModel.objects.all(), fields=('name','size'))
+
+In this example, only the `name` and `size` attributes of each model will
+be serialized. 
+
+.. note::
+
+    Depending on your model, you may find that it is not possible to deserialize
+    a model that only serializes a subset of it's fields. If a serialized object
+    doesn't specify all the fields that are required by a model, the deserializer
+    will not be able to save deserialized instances.
-
-Deserializing data
@@ -93 +112 @@
-                strings, etc.).  Not really all that useful on its own, but
-                used as a base for other serializers.
+    
+    ``yaml``    Serializes to YAML (Yet Another Markup Lanuage). This 
+                serializer will only be made available if PyYAML_ is installed.
+                
-    ==========  ==============================================================
-
-.. _json: http://json.org/
-.. _simplejson: http://undefined.org/python/#simplejson
+.. _PyYAML: http://www.pyyaml.org/
-
-Notes for specific serialization formats

django/trunk/tests/modeltests/serializers/models.py

@@ -160 +160 @@
-<Author: Agnes>
-
+# Serializer output can be restricted to a subset of fields
+>>> print serializers.serialize("json", Article.objects.all(), fields=('headline','pub_date'))
+[{"pk": "1", "model": "serializers.article", "fields": {"headline": "Just kidding; I love TV poker", "pub_date": "2006-06-16 11:00:00"}}, {"pk": "2", "model": "serializers.article", "fields": {"headline": "Time to reform copyright", "pub_date": "2006-06-16 13:00:00"}}, {"pk": "3", "model": "serializers.article", "fields": {"headline": "Forward references pose no problem", "pub_date": "2006-06-16 15:00:00"}}]
+
-"""}

django/trunk/tests/regressiontests/serializers_regress/models.py

@@ -206 +206 @@
-#     data = models.XMLField(primary_key=True)
-
+class ComplexModel(models.Model):
+    field1 = models.CharField(maxlength=10)
+    field2 = models.CharField(maxlength=10)
+    field3 = models.CharField(maxlength=10)

django/trunk/tests/regressiontests/serializers_regress/tests.py

@@ -10 +10 @@
-
-import unittest, datetime
+from cStringIO import StringIO
-
-from django.utils.functional import curry
@@ -279 +280 @@
-        func[1](self, pk, klass, datum)
-
+def fieldsTest(format, self):
+    # Clear the database first
+    management.flush(verbosity=0, interactive=False)
+
+    obj = ComplexModel(field1='first',field2='second',field3='third')
+    obj.save()
+    
+    # Serialize then deserialize the test database
+    serialized_data = serializers.serialize(format, [obj], indent=2, fields=('field1','field3'))
+    result = serializers.deserialize(format, serialized_data).next()
+    
+    # Check that the deserialized object contains data in only the serialized fields.
+    self.assertEqual(result.object.field1, 'first')
+    self.assertEqual(result.object.field2, '')
+    self.assertEqual(result.object.field3, 'third')
+
+def streamTest(format, self):
+    # Clear the database first
+    management.flush(verbosity=0, interactive=False)
+
+    obj = ComplexModel(field1='first',field2='second',field3='third')
+    obj.save()
+    
+    # Serialize the test database to a stream
+    stream = StringIO()    
+    serializers.serialize(format, [obj], indent=2, stream=stream)
+    
+    # Serialize normally for a comparison
+    string_data = serializers.serialize(format, [obj], indent=2)
+
+    # Check that the two are the same
+    self.assertEqual(string_data, stream.buffer())
+    stream.close()
+    
-for format in serializers.get_serializer_formats():
-    setattr(SerializerTests, 'test_'+format+'_serializer', curry(serializerTest, format))
+    setattr(SerializerTests, 'test_'+format+'_serializer_fields', curry(fieldsTest, format))
+    setattr(SerializerTests, 'test_'+format+'_serializer_stream', curry(fieldsTest, format))