From d327c5f531b341ad980d20454211b02b87f34d0e Mon Sep 17 00:00:00 2001
From: Tom Christie <tom@tomchristie.com>
Date: Thu, 1 Nov 2012 23:04:13 +0000
Subject: [PATCH 01/16] Relational field support in browseable API.

Add slug relational fields.
Add quickstart.
---
 djangorestframework.egg-info/PKG-INFO         |  19 ++++
 djangorestframework.egg-info/SOURCES.txt      |  86 ++++++++++++++
 .../dependency_links.txt                      |   1 +
 djangorestframework.egg-info/top_level.txt    |   7 ++
 docs/index.md                                 |   2 -
 docs/template.html                            |   2 +-
 docs/tutorial/quickstart.md                   |   9 +-
 rest_framework/fields.py                      | 107 +++++++++++++++++-
 rest_framework/renderers.py                   |  25 ++--
 .../static/rest_framework/css/default.css     |   7 ++
 .../templates/rest_framework/base.html        |   4 +-
 11 files changed, 244 insertions(+), 25 deletions(-)
 create mode 100644 djangorestframework.egg-info/PKG-INFO
 create mode 100644 djangorestframework.egg-info/SOURCES.txt
 create mode 100644 djangorestframework.egg-info/dependency_links.txt
 create mode 100644 djangorestframework.egg-info/top_level.txt

diff --git a/djangorestframework.egg-info/PKG-INFO b/djangorestframework.egg-info/PKG-INFO
new file mode 100644
index 000000000..04eafbde9
--- /dev/null
+++ b/djangorestframework.egg-info/PKG-INFO
@@ -0,0 +1,19 @@
+Metadata-Version: 1.0
+Name: djangorestframework
+Version: 2.0.0
+Summary: A lightweight REST framework for Django.
+Home-page: http://django-rest-framework.org
+Author: Tom Christie
+Author-email: tom@tomchristie.com
+License: BSD
+Download-URL: http://pypi.python.org/pypi/rest_framework/
+Description: UNKNOWN
+Platform: UNKNOWN
+Classifier: Development Status :: 4 - Beta
+Classifier: Environment :: Web Environment
+Classifier: Framework :: Django
+Classifier: Intended Audience :: Developers
+Classifier: License :: OSI Approved :: BSD License
+Classifier: Operating System :: OS Independent
+Classifier: Programming Language :: Python
+Classifier: Topic :: Internet :: WWW/HTTP
diff --git a/djangorestframework.egg-info/SOURCES.txt b/djangorestframework.egg-info/SOURCES.txt
new file mode 100644
index 000000000..298d1c333
--- /dev/null
+++ b/djangorestframework.egg-info/SOURCES.txt
@@ -0,0 +1,86 @@
+MANIFEST.in
+setup.py
+djangorestframework.egg-info/PKG-INFO
+djangorestframework.egg-info/SOURCES.txt
+djangorestframework.egg-info/dependency_links.txt
+djangorestframework.egg-info/top_level.txt
+rest_framework/__init__.py
+rest_framework/authentication.py
+rest_framework/compat.py
+rest_framework/decorators.py
+rest_framework/exceptions.py
+rest_framework/fields.py
+rest_framework/generics.py
+rest_framework/mixins.py
+rest_framework/models.py
+rest_framework/negotiation.py
+rest_framework/pagination.py
+rest_framework/parsers.py
+rest_framework/permissions.py
+rest_framework/renderers.py
+rest_framework/request.py
+rest_framework/response.py
+rest_framework/reverse.py
+rest_framework/serializers.py
+rest_framework/settings.py
+rest_framework/status.py
+rest_framework/throttling.py
+rest_framework/urlpatterns.py
+rest_framework/urls.py
+rest_framework/views.py
+rest_framework/authtoken/__init__.py
+rest_framework/authtoken/models.py
+rest_framework/authtoken/views.py
+rest_framework/authtoken/migrations/0001_initial.py
+rest_framework/authtoken/migrations/__init__.py
+rest_framework/runtests/__init__.py
+rest_framework/runtests/runcoverage.py
+rest_framework/runtests/runtests.py
+rest_framework/runtests/settings.py
+rest_framework/runtests/urls.py
+rest_framework/static/rest_framework/css/bootstrap-tweaks.css
+rest_framework/static/rest_framework/css/bootstrap.min.css
+rest_framework/static/rest_framework/css/default.css
+rest_framework/static/rest_framework/css/prettify.css
+rest_framework/static/rest_framework/img/glyphicons-halflings-white.png
+rest_framework/static/rest_framework/img/glyphicons-halflings.png
+rest_framework/static/rest_framework/img/grid.png
+rest_framework/static/rest_framework/js/bootstrap.min.js
+rest_framework/static/rest_framework/js/default.js
+rest_framework/static/rest_framework/js/jquery-1.8.1-min.js
+rest_framework/static/rest_framework/js/prettify-min.js
+rest_framework/templates/rest_framework/api.html
+rest_framework/templates/rest_framework/base.html
+rest_framework/templates/rest_framework/login.html
+rest_framework/templatetags/__init__.py
+rest_framework/templatetags/rest_framework.py
+rest_framework/tests/__init__.py
+rest_framework/tests/authentication.py
+rest_framework/tests/breadcrumbs.py
+rest_framework/tests/decorators.py
+rest_framework/tests/description.py
+rest_framework/tests/files.py
+rest_framework/tests/genericrelations.py
+rest_framework/tests/generics.py
+rest_framework/tests/htmlrenderer.py
+rest_framework/tests/hyperlinkedserializers.py
+rest_framework/tests/models.py
+rest_framework/tests/modelviews.py
+rest_framework/tests/negotiation.py
+rest_framework/tests/pagination.py
+rest_framework/tests/parsers.py
+rest_framework/tests/renderers.py
+rest_framework/tests/request.py
+rest_framework/tests/response.py
+rest_framework/tests/reverse.py
+rest_framework/tests/serializer.py
+rest_framework/tests/status.py
+rest_framework/tests/testcases.py
+rest_framework/tests/tests.py
+rest_framework/tests/throttling.py
+rest_framework/tests/validators.py
+rest_framework/tests/views.py
+rest_framework/utils/__init__.py
+rest_framework/utils/breadcrumbs.py
+rest_framework/utils/encoders.py
+rest_framework/utils/mediatypes.py
\ No newline at end of file
diff --git a/djangorestframework.egg-info/dependency_links.txt b/djangorestframework.egg-info/dependency_links.txt
new file mode 100644
index 000000000..8b1378917
--- /dev/null
+++ b/djangorestframework.egg-info/dependency_links.txt
@@ -0,0 +1 @@
+
diff --git a/djangorestframework.egg-info/top_level.txt b/djangorestframework.egg-info/top_level.txt
new file mode 100644
index 000000000..7e18534d8
--- /dev/null
+++ b/djangorestframework.egg-info/top_level.txt
@@ -0,0 +1,7 @@
+rest_framework/authtoken
+rest_framework/utils
+rest_framework/tests
+rest_framework/runtests
+rest_framework/templatetags
+rest_framework
+rest_framework/authtoken/migrations
diff --git a/docs/index.md b/docs/index.md
index 75a1cf6e6..5e0868724 100644
--- a/docs/index.md
+++ b/docs/index.md
@@ -66,11 +66,9 @@ If you're intending to use the browseable API you'll want to add REST framework'
 
 Note that the URL path can be whatever you want, but you must include `rest_framework.urls` with the `rest_framework` namespace.
 
-<!--
 ## Quickstart
 
 Can't wait to get started?  The [quickstart guide][quickstart] is the fastest way to get up and running with REST framework.
--->
 
 ## Tutorial
 
diff --git a/docs/template.html b/docs/template.html
index 94fc269f4..c428dff3a 100644
--- a/docs/template.html
+++ b/docs/template.html
@@ -53,7 +53,7 @@
               <li class="dropdown">
                 <a href="#" class="dropdown-toggle" data-toggle="dropdown">Tutorial <b class="caret"></b></a>
                 <ul class="dropdown-menu">
-                  <!--<li><a href="{{ base_url }}/tutorial/quickstart{{ suffix }}">Quickstart</a></li>-->
+                  <li><a href="{{ base_url }}/tutorial/quickstart{{ suffix }}">Quickstart</a></li>
                   <li><a href="{{ base_url }}/tutorial/1-serialization{{ suffix }}">1 - Serialization</a></li>
                   <li><a href="{{ base_url }}/tutorial/2-requests-and-responses{{ suffix }}">2 - Requests and responses</a></li>
                   <li><a href="{{ base_url }}/tutorial/3-class-based-views{{ suffix }}">3 - Class based views</a></li>
diff --git a/docs/tutorial/quickstart.md b/docs/tutorial/quickstart.md
index 6bde725b3..93da1a594 100644
--- a/docs/tutorial/quickstart.md
+++ b/docs/tutorial/quickstart.md
@@ -19,12 +19,19 @@ First up we're going to define some serializers in `quickstart/serializers.py` t
     
     
     class GroupSerializer(serializers.HyperlinkedModelSerializer):
+        permissions = serializers.ManySlugRelatedField(
+            slug_field='codename',
+            queryset=Permission.objects.all()
+        )
+
         class Meta:
             model = Group
             fields = ('url', 'name', 'permissions')
 
 Notice that we're using hyperlinked relations in this case, with `HyperlinkedModelSerializer`.  You can also use primary key and various other relationships, but hyperlinking is good RESTful design.
 
+We've also overridden the `permission` field on the `GroupSerializer`.  In this case we don't want to use a hyperlinked representation, but instead use the list of permission codenames associated with the group, so we've used a `ManySlugRelatedField`, using the `codename` field for the representation.
+
 ## Views
 
 Right, we'd better write some views then.  Open `quickstart/views.py` and get typing.
@@ -152,7 +159,7 @@ We can now access our API, both from the command-line, using tools like `curl`..
             }, 
             {
                 "email": "tom@example.com", 
-                "groups": [], 
+                "groups": [                ], 
                 "url": "http://127.0.0.1:8000/users/2/", 
                 "username": "tom"
             }
diff --git a/rest_framework/fields.py b/rest_framework/fields.py
index 1d6d760e3..6d858087b 100644
--- a/rest_framework/fields.py
+++ b/rest_framework/fields.py
@@ -8,6 +8,7 @@ from django.core.exceptions import ObjectDoesNotExist, ValidationError
 from django.core.urlresolvers import resolve, get_script_prefix
 from django.conf import settings
 from django.forms import widgets
+from django.forms.models import ModelChoiceIterator
 from django.utils.encoding import is_protected_type, smart_unicode
 from django.utils.translation import ugettext_lazy as _
 from rest_framework.reverse import reverse
@@ -232,11 +233,72 @@ class ModelField(WritableField):
 class RelatedField(WritableField):
     """
     Base class for related model fields.
+
+    If not overridden, this represents a to-one relatinship, using the unicode
+    representation of the target.
     """
+    widget = widgets.Select
+    cache_choices = False
+    empty_label = None
+
     def __init__(self, *args, **kwargs):
         self.queryset = kwargs.pop('queryset', None)
         super(RelatedField, self).__init__(*args, **kwargs)
 
+    ### We need this stuff to make form choices work...
+
+    # def __deepcopy__(self, memo):
+    #     result = super(RelatedField, self).__deepcopy__(memo)
+    #     result.queryset = result.queryset
+    #     return result
+
+    def prepare_value(self, obj):
+        return self.to_native(obj)
+
+    def label_from_instance(self, obj):
+        """
+        Return a readable representation for use with eg. select widgets.
+        """
+        desc = smart_unicode(obj)
+        ident = self.to_native(obj)
+        if desc == ident:
+            return desc
+        return "%s - %s" % (desc, ident)
+
+    def _get_queryset(self):
+        return self._queryset
+
+    def _set_queryset(self, queryset):
+        self._queryset = queryset
+        self.widget.choices = self.choices
+
+    queryset = property(_get_queryset, _set_queryset)
+
+    def _get_choices(self):
+        # If self._choices is set, then somebody must have manually set
+        # the property self.choices. In this case, just return self._choices.
+        if hasattr(self, '_choices'):
+            return self._choices
+
+        # Otherwise, execute the QuerySet in self.queryset to determine the
+        # choices dynamically. Return a fresh ModelChoiceIterator that has not been
+        # consumed. Note that we're instantiating a new ModelChoiceIterator *each*
+        # time _get_choices() is called (and, thus, each time self.choices is
+        # accessed) so that we can ensure the QuerySet has not been consumed. This
+        # construct might look complicated but it allows for lazy evaluation of
+        # the queryset.
+        return ModelChoiceIterator(self)
+
+    def _set_choices(self, value):
+        # Setting choices also sets the choices on the widget.
+        # choices can be any iterable, but we call list() on it because
+        # it will be consumed more than once.
+        self._choices = self.widget.choices = list(value)
+
+    choices = property(_get_choices, _set_choices)
+
+    ### Regular serializier stuff...
+
     def field_to_native(self, obj, field_name):
         value = getattr(obj, self.source or field_name)
         return self.to_native(value)
@@ -253,6 +315,8 @@ class ManyRelatedMixin(object):
     """
     Mixin to convert a related field to a many related field.
     """
+    widget = widgets.SelectMultiple
+
     def field_to_native(self, obj, field_name):
         value = getattr(obj, self.source or field_name)
         return [self.to_native(item) for item in value.all()]
@@ -276,6 +340,9 @@ class ManyRelatedMixin(object):
 class ManyRelatedField(ManyRelatedMixin, RelatedField):
     """
     Base class for related model managers.
+
+    If not overridden, this represents a to-many relatinship, using the unicode
+    representations of the target, and is read-only.
     """
     pass
 
@@ -284,7 +351,7 @@ class ManyRelatedField(ManyRelatedMixin, RelatedField):
 
 class PrimaryKeyRelatedField(RelatedField):
     """
-    Serializes a related field or related object to a pk value.
+    Represents a to-one relationship as a pk value.
     """
 
     def to_native(self, pk):
@@ -313,7 +380,7 @@ class PrimaryKeyRelatedField(RelatedField):
 
 class ManyPrimaryKeyRelatedField(ManyRelatedField):
     """
-    Serializes a to-many related field or related manager to a pk value.
+    Represents a to-many relationship as a pk value.
     """
     def to_native(self, pk):
         return pk
@@ -329,10 +396,36 @@ class ManyPrimaryKeyRelatedField(ManyRelatedField):
         # Forward relationship
         return [self.to_native(item.pk) for item in queryset.all()]
 
+### Slug relationships
+
+
+class SlugRelatedField(RelatedField):
+    def __init__(self, *args, **kwargs):
+        self.slug_field = kwargs.pop('slug_field', None)
+        assert self.slug_field, 'slug_field is required'
+        super(SlugRelatedField, self).__init__(*args, **kwargs)
+
+    def to_native(self, obj):
+        return getattr(obj, self.slug_field)
+
+    def from_native(self, data):
+        try:
+            return self.queryset.get(**{self.slug_field: data})
+        except ObjectDoesNotExist:
+            raise ValidationError('Object with %s=%s does not exist.' %
+                                  (self.slug_field, unicode(data)))
+
+
+class ManySlugRelatedField(ManyRelatedMixin, SlugRelatedField):
+    pass
+
 
 ### Hyperlinked relationships
 
 class HyperlinkedRelatedField(RelatedField):
+    """
+    Represents a to-one relationship, using hyperlinking.
+    """
     pk_url_kwarg = 'pk'
     slug_url_kwarg = 'slug'
     slug_field = 'slug'
@@ -417,16 +510,20 @@ class HyperlinkedRelatedField(RelatedField):
 
 
 class ManyHyperlinkedRelatedField(ManyRelatedMixin, HyperlinkedRelatedField):
+    """
+    Represents a to-many relationship, using hyperlinking.
+    """
     pass
 
 
 class HyperlinkedIdentityField(Field):
     """
-    A field that represents the model's identity using a hyperlink.
+    Represents the instance, or a property on the instance, using hyperlinking.
     """
+
     def __init__(self, *args, **kwargs):
-        # TODO: Make this mandatory, and have the HyperlinkedModelSerializer
-        # set it on-the-fly
+        # TODO: Make view_name mandatory, and have the
+        # HyperlinkedModelSerializer set it on-the-fly
         self.view_name = kwargs.pop('view_name', None)
         self.format = kwargs.pop('format', None)
         super(HyperlinkedIdentityField, self).__init__(*args, **kwargs)
diff --git a/rest_framework/renderers.py b/rest_framework/renderers.py
index 8dff0c773..63578b14d 100644
--- a/rest_framework/renderers.py
+++ b/rest_framework/renderers.py
@@ -282,10 +282,12 @@ class BrowsableAPIRenderer(BaseRenderer):
             serializers.EmailField: forms.EmailField,
             serializers.CharField: forms.CharField,
             serializers.BooleanField: forms.BooleanField,
-            serializers.PrimaryKeyRelatedField: forms.ModelChoiceField,
-            serializers.ManyPrimaryKeyRelatedField: forms.ModelMultipleChoiceField,
-            serializers.HyperlinkedRelatedField: forms.ModelChoiceField,
-            serializers.ManyHyperlinkedRelatedField: forms.ModelMultipleChoiceField
+            serializers.PrimaryKeyRelatedField: forms.ChoiceField,
+            serializers.ManyPrimaryKeyRelatedField: forms.MultipleChoiceField,
+            serializers.SlugRelatedField: forms.ChoiceField,
+            serializers.ManySlugRelatedField: forms.MultipleChoiceField,
+            serializers.HyperlinkedRelatedField: forms.ChoiceField,
+            serializers.ManyHyperlinkedRelatedField: forms.MultipleChoiceField
         }
 
         fields = {}
@@ -296,19 +298,14 @@ class BrowsableAPIRenderer(BaseRenderer):
             kwargs = {}
             kwargs['required'] = v.required
 
-            if getattr(v, 'queryset', None):
-                kwargs['queryset'] = v.queryset
+            #if getattr(v, 'queryset', None):
+            #    kwargs['queryset'] = v.queryset
+
+            if getattr(v, 'choices', None) is not None:
+                kwargs['choices'] = v.choices
 
             if getattr(v, 'widget', None):
                 widget = copy.deepcopy(v.widget)
-                # If choices have friendly readable names,
-                # then add in the identities too
-                if getattr(widget, 'choices', None):
-                    choices = widget.choices
-                    if any([ident != desc for (ident, desc) in choices]):
-                        choices = [(ident, "%s (%s)" % (desc, ident))
-                                   for (ident, desc) in choices]
-                    widget.choices = choices
                 kwargs['widget'] = widget
 
             if getattr(v, 'default', None) is not None:
diff --git a/rest_framework/static/rest_framework/css/default.css b/rest_framework/static/rest_framework/css/default.css
index e29da395a..fdf456596 100644
--- a/rest_framework/static/rest_framework/css/default.css
+++ b/rest_framework/static/rest_framework/css/default.css
@@ -36,6 +36,13 @@ ul.breadcrumb {
   margin: 58px 0 0 0;
 }
 
+form select, form input {
+  width: 90%;
+}
+
+form select[multiple] {
+  height: 150px;
+}
 /* To allow tooltips to work on disabled elements */
 .disabled-tooltip-shield {
     position: absolute;
diff --git a/rest_framework/templates/rest_framework/base.html b/rest_framework/templates/rest_framework/base.html
index e0f794814..9b0a0dcab 100644
--- a/rest_framework/templates/rest_framework/base.html
+++ b/rest_framework/templates/rest_framework/base.html
@@ -134,7 +134,7 @@
                                 <div class="control-group {% if field.errors %}error{% endif %}">
                                     {{ field.label_tag|add_class:"control-label" }}
                                     <div class="controls">
-                                        {{ field|add_class:"input-xlarge" }}
+                                        {{ field }}
                                         <span class="help-inline">{{ field.help_text }}</span>
                                         {{ field.errors|add_class:"help-block" }}
                                     </div>
@@ -159,7 +159,7 @@
                                 <div class="control-group {% if field.errors %}error{% endif %}">
                                     {{ field.label_tag|add_class:"control-label" }}
                                     <div class="controls">
-                                        {{ field|add_class:"input-xlarge" }}
+                                        {{ field }}
                                         <span class='help-inline'>{{ field.help_text }}</span>
                                         {{ field.errors|add_class:"help-block" }}
                                     </div>

From 09d70958bd09b65687aabbdfa999a7431d37897f Mon Sep 17 00:00:00 2001
From: Tom Christie <tom@tomchristie.com>
Date: Thu, 1 Nov 2012 23:15:36 +0000
Subject: [PATCH 02/16] Update release notes

---
 docs/topics/release-notes.md | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/docs/topics/release-notes.md b/docs/topics/release-notes.md
index b336aeab9..41a230f06 100644
--- a/docs/topics/release-notes.md
+++ b/docs/topics/release-notes.md
@@ -6,6 +6,8 @@
 
 ## Master
 
+* Add support for relational fields in the browsable API.
+* Added SlugRelatedField and ManySlugRelatedField.
 * If PUT creates an instance return '201 Created', instead of '200 OK'.
 
 ## 2.0.0

From 062f5caef3dcb76e19f9eed59779f45849a166a1 Mon Sep 17 00:00:00 2001
From: Tom Christie <tom@tomchristie.com>
Date: Thu, 1 Nov 2012 23:40:34 +0000
Subject: [PATCH 03/16] Tweaks fields docs, and fix 2.0.1 version.

---
 docs/api-guide/fields.md     | 42 ++++++++++++++++++++----------------
 docs/topics/release-notes.md |  2 +-
 rest_framework/__init__.py   |  2 +-
 3 files changed, 25 insertions(+), 21 deletions(-)

diff --git a/docs/api-guide/fields.md b/docs/api-guide/fields.md
index 8c3df0678..86460b4b6 100644
--- a/docs/api-guide/fields.md
+++ b/docs/api-guide/fields.md
@@ -235,44 +235,48 @@ Then an example output format for a Bookmark instance would be:
         'url': u'https://www.djangoproject.com/'
     }
 
-## PrimaryKeyRelatedField
+## PrimaryKeyRelatedField / ManyPrimaryKeyRelatedField
 
-This field can be applied to any "to-one" relationship, such as a `ForeignKey` field.
+`PrimaryKeyRelatedField` and `ManyPrimaryKeyRelatedField` will represent the target of the relationship using it's primary key.
 
-`PrimaryKeyRelatedField` will represent the target of the field using it's primary key.
+Be default these fields read-write, although you can change this behaviour using the `read_only` flag.
 
-Be default, `PrimaryKeyRelatedField` is read-write, although you can change this behaviour using the `read_only` flag.
+**Arguments**:
 
-## ManyPrimaryKeyRelatedField
+* `queryset` - All relational fields must either set a queryset, or set `read_only=True`
 
-This field can be applied to any "to-many" relationship, such as a `ManyToManyField` field, or a reverse `ForeignKey` relationship.
+## SlugRelatedField / ManySlugRelatedField
 
-`PrimaryKeyRelatedField` will represent the targets of the field using their primary key.
+`SlugRelatedField` and `ManySlugRelatedField` will represent the target of the relationship using a unique slug.
 
-Be default, `ManyPrimaryKeyRelatedField` is read-write, although you can change this behaviour using the `read_only` flag.
+Be default these fields read-write, although you can change this behaviour using the `read_only` flag.
 
-## HyperlinkedRelatedField
+**Arguments**:
 
-This field can be applied to any "to-one" relationship, such as a `ForeignKey` field.
+* `slug_field` - The field on the target that should used as the representation.  This should be a field that uniquely identifies any given instance.  For example, `username`.
+* `queryset` - All relational fields must either set a queryset, or set `read_only=True`
 
-`HyperlinkedRelatedField` will represent the target of the field using a hyperlink.  You must include a named URL pattern in your URL conf, with a name like `'{model-name}-detail'` that corresponds to the target of the hyperlink.
+## HyperlinkedRelatedField / ManyHyperlinkedRelatedField
+
+`HyperlinkedRelatedField` and `ManyHyperlinkedRelatedField` will represent the target of the relationship using a hyperlink.
 
 Be default, `HyperlinkedRelatedField` is read-write, although you can change this behaviour using the `read_only` flag.
 
-## ManyHyperlinkedRelatedField
+**Arguments**:
 
-This field can be applied to any "to-many" relationship, such as a `ManyToManyField` field, or a reverse `ForeignKey` relationship.
-
-`ManyHyperlinkedRelatedField` will represent the targets of the field using hyperlinks.  You must include a named URL pattern in your URL conf, with a name like `'{model-name}-detail'` that corresponds to the target of the hyperlink.
-
-Be default, `ManyHyperlinkedRelatedField` is read-write, although you can change this behaviour using the `read_only` flag.
+* `view_name` - The view name that should be used as the target of the relationship.  **required**.
+* `format` - If using format suffixes, hyperlinked fields will use the same format suffix for the target unless overridden by using the `format` argument.
+* `queryset` - All relational fields must either set a queryset, or set `read_only=True`
 
 ## HyperLinkedIdentityField
 
 This field can be applied as an identity relationship, such as the `'url'` field on  a HyperlinkedModelSerializer.
 
-You must include a named URL pattern in your URL conf, with a name like `'{model-name}-detail'` that corresponds to the model.
-
 This field is always read-only.
 
+**Arguments**:
+
+* `view_name` - The view name that should be used as the target of the relationship.  **required**.
+* `format` - If using format suffixes, hyperlinked fields will use the same format suffix for the target unless overridden by using the `format` argument.
+
 [cite]: http://www.python.org/dev/peps/pep-0020/
diff --git a/docs/topics/release-notes.md b/docs/topics/release-notes.md
index 41a230f06..1744dc031 100644
--- a/docs/topics/release-notes.md
+++ b/docs/topics/release-notes.md
@@ -4,7 +4,7 @@
 >
 > &mdash; Eric S. Raymond, [The Cathedral and the Bazaar][cite].
 
-## Master
+## 2.0.1
 
 * Add support for relational fields in the browsable API.
 * Added SlugRelatedField and ManySlugRelatedField.
diff --git a/rest_framework/__init__.py b/rest_framework/__init__.py
index 557f5943f..d6e762862 100644
--- a/rest_framework/__init__.py
+++ b/rest_framework/__init__.py
@@ -1,3 +1,3 @@
-__version__ = '2.0.0'
+__version__ = '2.0.1'
 
 VERSION = __version__  # synonym

From b8648c08bee3dc67565a5ac3da73395c1185320b Mon Sep 17 00:00:00 2001
From: Tom Christie <tom@tomchristie.com>
Date: Thu, 1 Nov 2012 23:48:25 +0000
Subject: [PATCH 04/16] gitignore *.egg-info

---
 .gitignore | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.gitignore b/.gitignore
index 96186b465..25113d09d 100644
--- a/.gitignore
+++ b/.gitignore
@@ -7,7 +7,7 @@ html/
 coverage/
 build/
 dist/
-rest_framework.egg-info/
+*.egg-info/
 MANIFEST
 
 !.gitignore

From 2dec2b9c1346d16ae746defb47b624953bf5f0d0 Mon Sep 17 00:00:00 2001
From: Tom Christie <tom@tomchristie.com>
Date: Thu, 1 Nov 2012 23:50:00 +0000
Subject: [PATCH 05/16] Remove accidental egg-info files

---
 djangorestframework.egg-info/PKG-INFO         | 19 ----
 djangorestframework.egg-info/SOURCES.txt      | 86 -------------------
 .../dependency_links.txt                      |  1 -
 djangorestframework.egg-info/top_level.txt    |  7 --
 4 files changed, 113 deletions(-)
 delete mode 100644 djangorestframework.egg-info/PKG-INFO
 delete mode 100644 djangorestframework.egg-info/SOURCES.txt
 delete mode 100644 djangorestframework.egg-info/dependency_links.txt
 delete mode 100644 djangorestframework.egg-info/top_level.txt

diff --git a/djangorestframework.egg-info/PKG-INFO b/djangorestframework.egg-info/PKG-INFO
deleted file mode 100644
index 04eafbde9..000000000
--- a/djangorestframework.egg-info/PKG-INFO
+++ /dev/null
@@ -1,19 +0,0 @@
-Metadata-Version: 1.0
-Name: djangorestframework
-Version: 2.0.0
-Summary: A lightweight REST framework for Django.
-Home-page: http://django-rest-framework.org
-Author: Tom Christie
-Author-email: tom@tomchristie.com
-License: BSD
-Download-URL: http://pypi.python.org/pypi/rest_framework/
-Description: UNKNOWN
-Platform: UNKNOWN
-Classifier: Development Status :: 4 - Beta
-Classifier: Environment :: Web Environment
-Classifier: Framework :: Django
-Classifier: Intended Audience :: Developers
-Classifier: License :: OSI Approved :: BSD License
-Classifier: Operating System :: OS Independent
-Classifier: Programming Language :: Python
-Classifier: Topic :: Internet :: WWW/HTTP
diff --git a/djangorestframework.egg-info/SOURCES.txt b/djangorestframework.egg-info/SOURCES.txt
deleted file mode 100644
index 298d1c333..000000000
--- a/djangorestframework.egg-info/SOURCES.txt
+++ /dev/null
@@ -1,86 +0,0 @@
-MANIFEST.in
-setup.py
-djangorestframework.egg-info/PKG-INFO
-djangorestframework.egg-info/SOURCES.txt
-djangorestframework.egg-info/dependency_links.txt
-djangorestframework.egg-info/top_level.txt
-rest_framework/__init__.py
-rest_framework/authentication.py
-rest_framework/compat.py
-rest_framework/decorators.py
-rest_framework/exceptions.py
-rest_framework/fields.py
-rest_framework/generics.py
-rest_framework/mixins.py
-rest_framework/models.py
-rest_framework/negotiation.py
-rest_framework/pagination.py
-rest_framework/parsers.py
-rest_framework/permissions.py
-rest_framework/renderers.py
-rest_framework/request.py
-rest_framework/response.py
-rest_framework/reverse.py
-rest_framework/serializers.py
-rest_framework/settings.py
-rest_framework/status.py
-rest_framework/throttling.py
-rest_framework/urlpatterns.py
-rest_framework/urls.py
-rest_framework/views.py
-rest_framework/authtoken/__init__.py
-rest_framework/authtoken/models.py
-rest_framework/authtoken/views.py
-rest_framework/authtoken/migrations/0001_initial.py
-rest_framework/authtoken/migrations/__init__.py
-rest_framework/runtests/__init__.py
-rest_framework/runtests/runcoverage.py
-rest_framework/runtests/runtests.py
-rest_framework/runtests/settings.py
-rest_framework/runtests/urls.py
-rest_framework/static/rest_framework/css/bootstrap-tweaks.css
-rest_framework/static/rest_framework/css/bootstrap.min.css
-rest_framework/static/rest_framework/css/default.css
-rest_framework/static/rest_framework/css/prettify.css
-rest_framework/static/rest_framework/img/glyphicons-halflings-white.png
-rest_framework/static/rest_framework/img/glyphicons-halflings.png
-rest_framework/static/rest_framework/img/grid.png
-rest_framework/static/rest_framework/js/bootstrap.min.js
-rest_framework/static/rest_framework/js/default.js
-rest_framework/static/rest_framework/js/jquery-1.8.1-min.js
-rest_framework/static/rest_framework/js/prettify-min.js
-rest_framework/templates/rest_framework/api.html
-rest_framework/templates/rest_framework/base.html
-rest_framework/templates/rest_framework/login.html
-rest_framework/templatetags/__init__.py
-rest_framework/templatetags/rest_framework.py
-rest_framework/tests/__init__.py
-rest_framework/tests/authentication.py
-rest_framework/tests/breadcrumbs.py
-rest_framework/tests/decorators.py
-rest_framework/tests/description.py
-rest_framework/tests/files.py
-rest_framework/tests/genericrelations.py
-rest_framework/tests/generics.py
-rest_framework/tests/htmlrenderer.py
-rest_framework/tests/hyperlinkedserializers.py
-rest_framework/tests/models.py
-rest_framework/tests/modelviews.py
-rest_framework/tests/negotiation.py
-rest_framework/tests/pagination.py
-rest_framework/tests/parsers.py
-rest_framework/tests/renderers.py
-rest_framework/tests/request.py
-rest_framework/tests/response.py
-rest_framework/tests/reverse.py
-rest_framework/tests/serializer.py
-rest_framework/tests/status.py
-rest_framework/tests/testcases.py
-rest_framework/tests/tests.py
-rest_framework/tests/throttling.py
-rest_framework/tests/validators.py
-rest_framework/tests/views.py
-rest_framework/utils/__init__.py
-rest_framework/utils/breadcrumbs.py
-rest_framework/utils/encoders.py
-rest_framework/utils/mediatypes.py
\ No newline at end of file
diff --git a/djangorestframework.egg-info/dependency_links.txt b/djangorestframework.egg-info/dependency_links.txt
deleted file mode 100644
index 8b1378917..000000000
--- a/djangorestframework.egg-info/dependency_links.txt
+++ /dev/null
@@ -1 +0,0 @@
-
diff --git a/djangorestframework.egg-info/top_level.txt b/djangorestframework.egg-info/top_level.txt
deleted file mode 100644
index 7e18534d8..000000000
--- a/djangorestframework.egg-info/top_level.txt
+++ /dev/null
@@ -1,7 +0,0 @@
-rest_framework/authtoken
-rest_framework/utils
-rest_framework/tests
-rest_framework/runtests
-rest_framework/templatetags
-rest_framework
-rest_framework/authtoken/migrations

From b9bff2a984767c4258cad4860addc53a9623bb11 Mon Sep 17 00:00:00 2001
From: Tom Christie <tom@tomchristie.com>
Date: Fri, 2 Nov 2012 14:05:14 +0000
Subject: [PATCH 06/16] Fix issues with pk related fields in the browsable API.

---
 README.md                    | 16 +++++++++++++++
 docs/topics/release-notes.md | 10 +++++++++
 rest_framework/__init__.py   |  2 +-
 rest_framework/fields.py     | 39 +++++++++++++++++++++++++++++++++++-
 4 files changed, 65 insertions(+), 2 deletions(-)

diff --git a/README.md b/README.md
index 4b2245cca..b1e86d288 100644
--- a/README.md
+++ b/README.md
@@ -57,8 +57,24 @@ To run the tests.
 
 # Changelog
 
+## 2.0.2
+
+**Date**: 2nd Nov 2012
+
+* Fix issues with pk related fields in the browsable API.
+
+## 2.0.1
+
+**Date**: 1st Nov 2012
+
+* Add support for relational fields in the browsable API.
+* Added SlugRelatedField and ManySlugRelatedField.
+* If PUT creates an instance return '201 Created', instead of '200 OK'.
+
 ## 2.0.0
 
+**Date**: 30th Oct 2012
+
 * Redesign of core components.
 * Fix **all of the things**.
 
diff --git a/docs/topics/release-notes.md b/docs/topics/release-notes.md
index 1744dc031..b9107e968 100644
--- a/docs/topics/release-notes.md
+++ b/docs/topics/release-notes.md
@@ -4,14 +4,24 @@
 >
 > &mdash; Eric S. Raymond, [The Cathedral and the Bazaar][cite].
 
+## 2.0.2
+
+**Date**: 2nd Nov 2012
+
+* Fix issues with pk related fields in the browsable API.
+
 ## 2.0.1
 
+**Date**: 1st Nov 2012
+
 * Add support for relational fields in the browsable API.
 * Added SlugRelatedField and ManySlugRelatedField.
 * If PUT creates an instance return '201 Created', instead of '200 OK'.
 
 ## 2.0.0
 
+**Date**: 30th Oct 2012
+
 * **Fix all of the things.**  (Well, almost.)
 * For more information please see the [2.0 migration guide][migration].
 
diff --git a/rest_framework/__init__.py b/rest_framework/__init__.py
index d6e762862..56b9a35ca 100644
--- a/rest_framework/__init__.py
+++ b/rest_framework/__init__.py
@@ -1,3 +1,3 @@
-__version__ = '2.0.1'
+__version__ = '2.0.2'
 
 VERSION = __version__  # synonym
diff --git a/rest_framework/fields.py b/rest_framework/fields.py
index b7d022349..375d7a465 100644
--- a/rest_framework/fields.py
+++ b/rest_framework/fields.py
@@ -230,6 +230,7 @@ class ModelField(WritableField):
 ##### Relational fields #####
 
 
+# Not actually Writable, but subclasses may need to be.
 class RelatedField(WritableField):
     """
     Base class for related model fields.
@@ -240,10 +241,12 @@ class RelatedField(WritableField):
     widget = widgets.Select
     cache_choices = False
     empty_label = None
+    default_read_only = True  # TODO: Remove this
 
     def __init__(self, *args, **kwargs):
         self.queryset = kwargs.pop('queryset', None)
         super(RelatedField, self).__init__(*args, **kwargs)
+        self.read_only = self.default_read_only
 
     ### We need this stuff to make form choices work...
 
@@ -260,7 +263,7 @@ class RelatedField(WritableField):
         Return a readable representation for use with eg. select widgets.
         """
         desc = smart_unicode(obj)
-        ident = self.to_native(obj)
+        ident = smart_unicode(self.to_native(obj))
         if desc == ident:
             return desc
         return "%s - %s" % (desc, ident)
@@ -353,7 +356,23 @@ class PrimaryKeyRelatedField(RelatedField):
     """
     Represents a to-one relationship as a pk value.
     """
+    default_read_only = False
 
+    # TODO: Remove these field hacks...
+    def prepare_value(self, obj):
+        return self.to_native(obj.pk)
+
+    def label_from_instance(self, obj):
+        """
+        Return a readable representation for use with eg. select widgets.
+        """
+        desc = smart_unicode(obj)
+        ident = smart_unicode(self.to_native(obj.pk))
+        if desc == ident:
+            return desc
+        return "%s - %s" % (desc, ident)
+
+    # TODO: Possibly change this to just take `obj`, through prob less performant
     def to_native(self, pk):
         return pk
 
@@ -382,6 +401,21 @@ class ManyPrimaryKeyRelatedField(ManyRelatedField):
     """
     Represents a to-many relationship as a pk value.
     """
+    default_read_only = False
+
+    def prepare_value(self, obj):
+        return self.to_native(obj.pk)
+
+    def label_from_instance(self, obj):
+        """
+        Return a readable representation for use with eg. select widgets.
+        """
+        desc = smart_unicode(obj)
+        ident = smart_unicode(self.to_native(obj.pk))
+        if desc == ident:
+            return desc
+        return "%s - %s" % (desc, ident)
+
     def to_native(self, pk):
         return pk
 
@@ -400,6 +434,8 @@ class ManyPrimaryKeyRelatedField(ManyRelatedField):
 
 
 class SlugRelatedField(RelatedField):
+    default_read_only = False
+
     def __init__(self, *args, **kwargs):
         self.slug_field = kwargs.pop('slug_field', None)
         assert self.slug_field, 'slug_field is required'
@@ -429,6 +465,7 @@ class HyperlinkedRelatedField(RelatedField):
     pk_url_kwarg = 'pk'
     slug_url_kwarg = 'slug'
     slug_field = 'slug'
+    default_read_only = False
 
     def __init__(self, *args, **kwargs):
         try:

From e84ce60a0da77c0e07e0d6e5f627694ae3f4422f Mon Sep 17 00:00:00 2001
From: Tom Christie <tom@tomchristie.com>
Date: Fri, 2 Nov 2012 19:11:40 +0000
Subject: [PATCH 07/16] Initial PK relationship tests

---
 rest_framework/tests/pk_relations.py | 86 ++++++++++++++++++++++++++++
 1 file changed, 86 insertions(+)
 create mode 100644 rest_framework/tests/pk_relations.py

diff --git a/rest_framework/tests/pk_relations.py b/rest_framework/tests/pk_relations.py
new file mode 100644
index 000000000..eca534c2f
--- /dev/null
+++ b/rest_framework/tests/pk_relations.py
@@ -0,0 +1,86 @@
+from django.db import models
+from django.test import TestCase
+from rest_framework import serializers
+
+
+class Target(models.Model):
+    name = models.CharField(max_length=100)
+
+
+class Source(models.Model):
+    name = models.CharField(max_length=100)
+    targets = models.ManyToManyField(Target, related_name='sources')
+
+
+class TargetSerializer(serializers.ModelSerializer):
+    sources = serializers.ManyPrimaryKeyRelatedField()
+
+    class Meta:
+        fields = ('id', 'name', 'sources')
+        model = Target
+
+
+class SourceSerializer(serializers.ModelSerializer):
+    class Meta:
+        model = Source
+
+
+# TODO: Add test that .data cannot be accessed prior to .is_valid
+
+class PrimaryKeyManyToManyTests(TestCase):
+    def setUp(self):
+        for idx in range(1, 4):
+            target = Target(name='target-%d' % idx)
+            target.save()
+            source = Source(name='source-%d' % idx)
+            source.save()
+            for target in Target.objects.all():
+                source.targets.add(target)
+
+    def test_many_to_many_retrieve(self):
+        serializer = SourceSerializer(instance=Source.objects.all())
+        expected = [
+                {'id': 1, 'name': u'source-1', 'targets': [1]},
+                {'id': 2, 'name': u'source-2', 'targets': [1, 2]},
+                {'id': 3, 'name': u'source-3', 'targets': [1, 2, 3]}
+        ]
+        self.assertEquals(serializer.data, expected)
+
+    def test_reverse_many_to_many_retrieve(self):
+        serializer = TargetSerializer(instance=Target.objects.all())
+        expected = [
+            {'id': 1, 'name': u'target-1', 'sources': [1, 2, 3]},
+            {'id': 2, 'name': u'target-2', 'sources': [2, 3]},
+            {'id': 3, 'name': u'target-3', 'sources': [3]}
+        ]
+        self.assertEquals(serializer.data, expected)
+
+    def test_many_to_many_update(self):
+        data = {'id': 1, 'name': u'source-1', 'targets': [1, 2, 3]}
+        serializer = SourceSerializer(data, instance=Source.objects.get(pk=1))
+        self.assertTrue(serializer.is_valid())
+        self.assertEquals(serializer.data, data)
+
+        # Ensure source 1 is updated, and everything else is as expected
+        serializer = SourceSerializer(instance=Source.objects.all())
+        expected = [
+                {'id': 1, 'name': u'source-1', 'targets': [1, 2, 3]},
+                {'id': 2, 'name': u'source-2', 'targets': [1, 2]},
+                {'id': 3, 'name': u'source-3', 'targets': [1, 2, 3]}
+        ]
+        self.assertEquals(serializer.data, expected)
+
+    def test_reverse_many_to_many_update(self):
+        data = {'id': 1, 'name': u'target-0', 'sources': [1]}
+        serializer = TargetSerializer(data, instance=Target.objects.get(pk=1))
+        self.assertTrue(serializer.is_valid())
+        self.assertEquals(serializer.data, data)
+
+        # Ensure target 1 is updated, and everything else is as expected
+        serializer = TargetSerializer(instance=Target.objects.all())
+        expected = [
+            {'id': 1, 'name': u'target-1', 'sources': [1]},
+            {'id': 2, 'name': u'target-2', 'sources': [2, 3]},
+            {'id': 3, 'name': u'target-3', 'sources': [3]}
+        ]
+        self.assertEquals(serializer.data, expected)

From 3de61d200ccfe51a8e7e2fd900838c9c1c91b0f2 Mon Sep 17 00:00:00 2001
From: Jacob Magnusson <m@jacobian.se>
Date: Fri, 2 Nov 2012 20:55:58 +0100
Subject: [PATCH 08/16] Don't convert dictionary into a string representation

---
 rest_framework/fields.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/rest_framework/fields.py b/rest_framework/fields.py
index 375d7a465..6dfe74580 100644
--- a/rest_framework/fields.py
+++ b/rest_framework/fields.py
@@ -90,6 +90,8 @@ class Field(object):
             return value
         elif hasattr(value, '__iter__') and not isinstance(value, (dict, basestring)):
             return [self.to_native(item) for item in value]
+        elif isinstance(value, dict):
+            return dict((k, self.to_native(v)) for k, v in value.items())
         return smart_unicode(value)
 
     def attributes(self):

From a6806f03078fbdda598e10260f9d9bcdf07c1dce Mon Sep 17 00:00:00 2001
From: Jacob Magnusson <m@jacobian.se>
Date: Fri, 2 Nov 2012 21:48:16 +0100
Subject: [PATCH 09/16] Apply to_native on dictionary keys as well

---
 rest_framework/fields.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/rest_framework/fields.py b/rest_framework/fields.py
index 6dfe74580..515ccbcf5 100644
--- a/rest_framework/fields.py
+++ b/rest_framework/fields.py
@@ -91,7 +91,7 @@ class Field(object):
         elif hasattr(value, '__iter__') and not isinstance(value, (dict, basestring)):
             return [self.to_native(item) for item in value]
         elif isinstance(value, dict):
-            return dict((k, self.to_native(v)) for k, v in value.items())
+            return dict(map(self.to_native, (k, v)) for k, v in value.items())
         return smart_unicode(value)
 
     def attributes(self):

From 6eaec7a0eccabb3e1b010d07633632e8a3ecd86f Mon Sep 17 00:00:00 2001
From: Tom Christie <tom@tomchristie.com>
Date: Fri, 2 Nov 2012 20:53:33 +0000
Subject: [PATCH 10/16] foreign key tests

---
 rest_framework/fields.py             |  13 ++-
 rest_framework/tests/pk_relations.py | 139 +++++++++++++++++++++++----
 2 files changed, 132 insertions(+), 20 deletions(-)

diff --git a/rest_framework/fields.py b/rest_framework/fields.py
index 375d7a465..965e22c4d 100644
--- a/rest_framework/fields.py
+++ b/rest_framework/fields.py
@@ -383,7 +383,8 @@ class PrimaryKeyRelatedField(RelatedField):
         try:
             return self.queryset.get(pk=data)
         except ObjectDoesNotExist:
-            raise ValidationError('Invalid hyperlink - object does not exist.')
+            msg = "Invalid pk '%s' - object does not exist." % smart_unicode(data)
+            raise ValidationError(msg)
 
     def field_to_native(self, obj, field_name):
         try:
@@ -430,6 +431,16 @@ class ManyPrimaryKeyRelatedField(ManyRelatedField):
         # Forward relationship
         return [self.to_native(item.pk) for item in queryset.all()]
 
+    def from_native(self, data):
+        if self.queryset is None:
+            raise Exception('Writable related fields must include a `queryset` argument')
+
+        try:
+            return self.queryset.get(pk=data)
+        except ObjectDoesNotExist:
+            msg = "Invalid pk '%s' - object does not exist." % smart_unicode(data)
+            raise ValidationError(msg)
+
 ### Slug relationships
 
 
diff --git a/rest_framework/tests/pk_relations.py b/rest_framework/tests/pk_relations.py
index eca534c2f..a09343448 100644
--- a/rest_framework/tests/pk_relations.py
+++ b/rest_framework/tests/pk_relations.py
@@ -3,26 +3,50 @@ from django.test import TestCase
 from rest_framework import serializers
 
 
-class Target(models.Model):
+# ManyToMany
+
+class ManyToManyTarget(models.Model):
     name = models.CharField(max_length=100)
 
 
-class Source(models.Model):
+class ManyToManySource(models.Model):
     name = models.CharField(max_length=100)
-    targets = models.ManyToManyField(Target, related_name='sources')
+    targets = models.ManyToManyField(ManyToManyTarget, related_name='sources')
 
 
-class TargetSerializer(serializers.ModelSerializer):
-    sources = serializers.ManyPrimaryKeyRelatedField()
+class ManyToManyTargetSerializer(serializers.ModelSerializer):
+    sources = serializers.ManyPrimaryKeyRelatedField(queryset=ManyToManySource.objects.all())
 
     class Meta:
-        fields = ('id', 'name', 'sources')
-        model = Target
+        model = ManyToManyTarget
 
 
-class SourceSerializer(serializers.ModelSerializer):
+class ManyToManySourceSerializer(serializers.ModelSerializer):
     class Meta:
-        model = Source
+        model = ManyToManySource
+
+
+# ForeignKey
+
+class ForeignKeyTarget(models.Model):
+    name = models.CharField(max_length=100)
+
+
+class ForeignKeySource(models.Model):
+    name = models.CharField(max_length=100)
+    target = models.ForeignKey(ForeignKeyTarget, related_name='sources')
+
+
+class ForeignKeyTargetSerializer(serializers.ModelSerializer):
+    sources = serializers.ManyPrimaryKeyRelatedField(queryset=ForeignKeySource.objects.all())
+
+    class Meta:
+        model = ForeignKeyTarget
+
+
+class ForeignKeySourceSerializer(serializers.ModelSerializer):
+    class Meta:
+        model = ForeignKeySource
 
 
 # TODO: Add test that .data cannot be accessed prior to .is_valid
@@ -30,15 +54,16 @@ class SourceSerializer(serializers.ModelSerializer):
 class PrimaryKeyManyToManyTests(TestCase):
     def setUp(self):
         for idx in range(1, 4):
-            target = Target(name='target-%d' % idx)
+            target = ManyToManyTarget(name='target-%d' % idx)
             target.save()
-            source = Source(name='source-%d' % idx)
+            source = ManyToManySource(name='source-%d' % idx)
             source.save()
-            for target in Target.objects.all():
+            for target in ManyToManyTarget.objects.all():
                 source.targets.add(target)
 
     def test_many_to_many_retrieve(self):
-        serializer = SourceSerializer(instance=Source.objects.all())
+        queryset = ManyToManySource.objects.all()
+        serializer = ManyToManySourceSerializer(instance=queryset)
         expected = [
                 {'id': 1, 'name': u'source-1', 'targets': [1]},
                 {'id': 2, 'name': u'source-2', 'targets': [1, 2]},
@@ -47,7 +72,8 @@ class PrimaryKeyManyToManyTests(TestCase):
         self.assertEquals(serializer.data, expected)
 
     def test_reverse_many_to_many_retrieve(self):
-        serializer = TargetSerializer(instance=Target.objects.all())
+        queryset = ManyToManyTarget.objects.all()
+        serializer = ManyToManyTargetSerializer(instance=queryset)
         expected = [
             {'id': 1, 'name': u'target-1', 'sources': [1, 2, 3]},
             {'id': 2, 'name': u'target-2', 'sources': [2, 3]},
@@ -57,12 +83,15 @@ class PrimaryKeyManyToManyTests(TestCase):
 
     def test_many_to_many_update(self):
         data = {'id': 1, 'name': u'source-1', 'targets': [1, 2, 3]}
-        serializer = SourceSerializer(data, instance=Source.objects.get(pk=1))
+        instance = ManyToManySource.objects.get(pk=1)
+        serializer = ManyToManySourceSerializer(data, instance=instance)
         self.assertTrue(serializer.is_valid())
         self.assertEquals(serializer.data, data)
+        serializer.save()
 
         # Ensure source 1 is updated, and everything else is as expected
-        serializer = SourceSerializer(instance=Source.objects.all())
+        queryset = ManyToManySource.objects.all()
+        serializer = ManyToManySourceSerializer(instance=queryset)
         expected = [
                 {'id': 1, 'name': u'source-1', 'targets': [1, 2, 3]},
                 {'id': 2, 'name': u'source-2', 'targets': [1, 2]},
@@ -71,16 +100,88 @@ class PrimaryKeyManyToManyTests(TestCase):
         self.assertEquals(serializer.data, expected)
 
     def test_reverse_many_to_many_update(self):
-        data = {'id': 1, 'name': u'target-0', 'sources': [1]}
-        serializer = TargetSerializer(data, instance=Target.objects.get(pk=1))
+        data = {'id': 1, 'name': u'target-1', 'sources': [1]}
+        instance = ManyToManyTarget.objects.get(pk=1)
+        serializer = ManyToManyTargetSerializer(data, instance=instance)
         self.assertTrue(serializer.is_valid())
         self.assertEquals(serializer.data, data)
+        serializer.save()
 
         # Ensure target 1 is updated, and everything else is as expected
-        serializer = TargetSerializer(instance=Target.objects.all())
+        queryset = ManyToManyTarget.objects.all()
+        serializer = ManyToManyTargetSerializer(instance=queryset)
         expected = [
             {'id': 1, 'name': u'target-1', 'sources': [1]},
             {'id': 2, 'name': u'target-2', 'sources': [2, 3]},
             {'id': 3, 'name': u'target-3', 'sources': [3]}
         ]
         self.assertEquals(serializer.data, expected)
+
+
+class PrimaryKeyForeignKeyTests(TestCase):
+    def setUp(self):
+        target = ForeignKeyTarget(name='target-1')
+        target.save()
+        new_target = ForeignKeyTarget(name='target-2')
+        new_target.save()
+        for idx in range(1, 4):
+            source = ForeignKeySource(name='source-%d' % idx, target=target)
+            source.save()
+
+    def test_foreign_key_retrieve(self):
+        queryset = ForeignKeySource.objects.all()
+        serializer = ForeignKeySourceSerializer(instance=queryset)
+        expected = [
+            {'id': 1, 'name': u'source-1', 'target': 1},
+            {'id': 2, 'name': u'source-2', 'target': 1},
+            {'id': 3, 'name': u'source-3', 'target': 1}
+        ]
+        self.assertEquals(serializer.data, expected)
+
+    def test_reverse_foreign_key_retrieve(self):
+        queryset = ForeignKeyTarget.objects.all()
+        serializer = ForeignKeyTargetSerializer(instance=queryset)
+        expected = [
+            {'id': 1, 'name': u'target-1', 'sources': [1, 2, 3]},
+            {'id': 2, 'name': u'target-2', 'sources': []},
+        ]
+        self.assertEquals(serializer.data, expected)
+
+    def test_foreign_key_update(self):
+        data = {'id': 1, 'name': u'source-1', 'target': 2}
+        instance = ForeignKeySource.objects.get(pk=1)
+        serializer = ForeignKeySourceSerializer(data, instance=instance)
+        self.assertTrue(serializer.is_valid())
+        self.assertEquals(serializer.data, data)
+        serializer.save()
+
+        # # Ensure source 1 is updated, and everything else is as expected
+        queryset = ForeignKeySource.objects.all()
+        serializer = ForeignKeySourceSerializer(instance=queryset)
+        expected = [
+            {'id': 1, 'name': u'source-1', 'target': 2},
+            {'id': 2, 'name': u'source-2', 'target': 1},
+            {'id': 3, 'name': u'source-3', 'target': 1}
+        ]
+        self.assertEquals(serializer.data, expected)
+
+    # reverse foreign keys MUST be read_only
+    # In the general case they do not provide .remove() or .clear()
+    # and cannot be arbitrarily set.
+
+    # def test_reverse_foreign_key_update(self):
+    #     data = {'id': 1, 'name': u'target-1', 'sources': [1]}
+    #     instance = ForeignKeyTarget.objects.get(pk=1)
+    #     serializer = ForeignKeyTargetSerializer(data, instance=instance)
+    #     self.assertTrue(serializer.is_valid())
+    #     self.assertEquals(serializer.data, data)
+    #     serializer.save()
+
+    #     # Ensure target 1 is updated, and everything else is as expected
+    #     queryset = ForeignKeyTarget.objects.all()
+    #     serializer = ForeignKeyTargetSerializer(instance=queryset)
+    #     expected = [
+    #         {'id': 1, 'name': u'target-1', 'sources': [1]},
+    #         {'id': 2, 'name': u'target-2', 'sources': []},
+    #     ]
+    #     self.assertEquals(serializer.data, expected)

From 8ec54e6a9fe8e46955098ca8e7d728d8bb4f7053 Mon Sep 17 00:00:00 2001
From: Tom Christie <tom@tomchristie.com>
Date: Fri, 2 Nov 2012 20:56:51 +0000
Subject: [PATCH 11/16] Tweaks

---
 rest_framework/fields.py               | 2 +-
 rest_framework/runtests/runcoverage.py | 4 ++--
 rest_framework/tests/pk_relations.py   | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/rest_framework/fields.py b/rest_framework/fields.py
index 965e22c4d..661d02dee 100644
--- a/rest_framework/fields.py
+++ b/rest_framework/fields.py
@@ -344,7 +344,7 @@ class ManyRelatedField(ManyRelatedMixin, RelatedField):
     """
     Base class for related model managers.
 
-    If not overridden, this represents a to-many relatinship, using the unicode
+    If not overridden, this represents a to-many relationship, using the unicode
     representations of the target, and is read-only.
     """
     pass
diff --git a/rest_framework/runtests/runcoverage.py b/rest_framework/runtests/runcoverage.py
index ea2e3d458..0ce379ebe 100755
--- a/rest_framework/runtests/runcoverage.py
+++ b/rest_framework/runtests/runcoverage.py
@@ -32,10 +32,10 @@ def main():
             'Function-based test runners are deprecated. Test runners should be classes with a run_tests() method.',
             DeprecationWarning
         )
-        failures = TestRunner(['rest_framework'])
+        failures = TestRunner(['tests'])
     else:
         test_runner = TestRunner()
-        failures = test_runner.run_tests(['rest_framework'])
+        failures = test_runner.run_tests(['tests'])
     cov.stop()
 
     # Discover the list of all modules that we should test coverage for
diff --git a/rest_framework/tests/pk_relations.py b/rest_framework/tests/pk_relations.py
index a09343448..9095dcd81 100644
--- a/rest_framework/tests/pk_relations.py
+++ b/rest_framework/tests/pk_relations.py
@@ -38,7 +38,7 @@ class ForeignKeySource(models.Model):
 
 
 class ForeignKeyTargetSerializer(serializers.ModelSerializer):
-    sources = serializers.ManyPrimaryKeyRelatedField(queryset=ForeignKeySource.objects.all())
+    sources = serializers.ManyPrimaryKeyRelatedField(read_only=True)
 
     class Meta:
         model = ForeignKeyTarget

From 59c1e0933f1f7896bb6f6f248aca07acdfb35ae7 Mon Sep 17 00:00:00 2001
From: Tom Christie <tom@tomchristie.com>
Date: Fri, 2 Nov 2012 21:13:29 +0000
Subject: [PATCH 12/16] Update release notes

---
 README.md                    | 4 ++++
 docs/topics/release-notes.md | 4 ++++
 2 files changed, 8 insertions(+)

diff --git a/README.md b/README.md
index b1e86d288..3d8dce948 100644
--- a/README.md
+++ b/README.md
@@ -57,6 +57,10 @@ To run the tests.
 
 # Changelog
 
+## Master
+
+* Minor field improvements (don't stringify dicts, more robust many-pk fields)
+
 ## 2.0.2
 
 **Date**: 2nd Nov 2012
diff --git a/docs/topics/release-notes.md b/docs/topics/release-notes.md
index b9107e968..0bf46430a 100644
--- a/docs/topics/release-notes.md
+++ b/docs/topics/release-notes.md
@@ -4,6 +4,10 @@
 >
 > &mdash; Eric S. Raymond, [The Cathedral and the Bazaar][cite].
 
+## Master
+
+* Minor field improvements (don't stringify dicts, more robust many-pk fields)
+
 ## 2.0.2
 
 **Date**: 2nd Nov 2012

From 19a218f85973b4c478ec0b8373e971e1d787281a Mon Sep 17 00:00:00 2001
From: Jacob Magnusson <m@jacobian.se>
Date: Sat, 3 Nov 2012 00:27:56 +0100
Subject: [PATCH 13/16] Bye bye star import

---
 rest_framework/tests/serializer.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/rest_framework/tests/serializer.py b/rest_framework/tests/serializer.py
index d4b43862f..9febc9c8a 100644
--- a/rest_framework/tests/serializer.py
+++ b/rest_framework/tests/serializer.py
@@ -1,7 +1,9 @@
 import datetime
 from django.test import TestCase
 from rest_framework import serializers
-from rest_framework.tests.models import *
+from rest_framework.tests.models import (ActionItem, Anchor, BasicModel,
+    BlankFieldModel, BlogPost, CallableDefaultValueModel, DefaultValueModel,
+    ManyToManyModel, Person, ReadOnlyManyToManyModel)
 
 
 class SubComment(object):

From 1b49615c00793b658f40d08617965316a3f71a7e Mon Sep 17 00:00:00 2001
From: Jacob Magnusson <m@jacobian.se>
Date: Sat, 3 Nov 2012 00:32:02 +0100
Subject: [PATCH 14/16] Added test that makes sure that fields with
 dictionaries as data are returned as expected and not turned into string
 representations

---
 rest_framework/tests/models.py     |  7 +++++++
 rest_framework/tests/serializer.py | 13 +++++++++++++
 2 files changed, 20 insertions(+)

diff --git a/rest_framework/tests/models.py b/rest_framework/tests/models.py
index 415e4d062..1a0078e8b 100644
--- a/rest_framework/tests/models.py
+++ b/rest_framework/tests/models.py
@@ -122,6 +122,13 @@ class Person(RESTFrameworkModel):
     name = models.CharField(max_length=10)
     age = models.IntegerField(null=True, blank=True)
 
+    @property
+    def info(self):
+        return {
+            'name': self.name,
+            'age': self.age,
+        }
+
 
 # Model for issue #324
 class BlankFieldModel(RESTFrameworkModel):
diff --git a/rest_framework/tests/serializer.py b/rest_framework/tests/serializer.py
index 9febc9c8a..5d46ff68f 100644
--- a/rest_framework/tests/serializer.py
+++ b/rest_framework/tests/serializer.py
@@ -46,8 +46,11 @@ class ActionItemSerializer(serializers.ModelSerializer):
 
 
 class PersonSerializer(serializers.ModelSerializer):
+    info = serializers.Field(source='info')
+
     class Meta:
         model = Person
+        fields = ('name', 'age', 'info')
 
 
 class BasicTests(TestCase):
@@ -69,6 +72,9 @@ class BasicTests(TestCase):
             'created': datetime.datetime(2012, 1, 1),
             'sub_comment': 'And Merry Christmas!'
         }
+        self.person_data = {'name': 'dwight', 'age': 35}
+        self.person = Person(**self.person_data)
+        self.person.save()
 
     def test_empty(self):
         serializer = CommentSerializer()
@@ -100,6 +106,13 @@ class BasicTests(TestCase):
         self.assertTrue(serializer.object is expected)
         self.assertEquals(serializer.data['sub_comment'], 'And Merry Christmas!')
 
+    def test_field_with_dictionary(self):
+        """ Make sure that dictionaries from fields are left intact
+        """
+        serializer = PersonSerializer(instance=self.person)
+        expected = self.person_data
+        self.assertEquals(serializer.data['info'], expected)
+
 
 class ValidationTests(TestCase):
     def setUp(self):

From 38af6107b5fb26e2dffdeeadad80df70e256a5f3 Mon Sep 17 00:00:00 2001
From: Jacob Magnusson <m@jacobian.se>
Date: Sat, 3 Nov 2012 00:32:08 +0100
Subject: [PATCH 15/16] Added test for ModelSerializer Meta fields returning as
 expected

---
 rest_framework/tests/serializer.py | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/rest_framework/tests/serializer.py b/rest_framework/tests/serializer.py
index 5d46ff68f..3fd490644 100644
--- a/rest_framework/tests/serializer.py
+++ b/rest_framework/tests/serializer.py
@@ -105,6 +105,14 @@ class BasicTests(TestCase):
         self.assertEquals(serializer.object, expected)
         self.assertTrue(serializer.object is expected)
         self.assertEquals(serializer.data['sub_comment'], 'And Merry Christmas!')
+    
+    def test_model_fields_as_expected(self):
+        """ Make sure that the fields returned are the same as defined
+        in the Meta data
+        """
+        serializer = PersonSerializer(instance=self.person)
+        self.assertEquals(set(serializer.data.keys()),
+                          set(['name', 'age', 'info']))
 
     def test_field_with_dictionary(self):
         """ Make sure that dictionaries from fields are left intact

From 3e0319389a4e7714e0658bd38cb1e9fb01cf662d Mon Sep 17 00:00:00 2001
From: Tom Christie <tom@tomchristie.com>
Date: Fri, 2 Nov 2012 21:17:44 +0000
Subject: [PATCH 16/16] Added @jmagnusson.  Thanks!

---
 docs/topics/credits.md | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/docs/topics/credits.md b/docs/topics/credits.md
index df0223410..0e8399b31 100644
--- a/docs/topics/credits.md
+++ b/docs/topics/credits.md
@@ -53,6 +53,7 @@ The following people have helped make REST framework great.
 * Stephan Groß - [minddust]
 * Pavel Savchenko - [asfaltboy]
 * Otto Yiu - [ottoyiu]
+* Jacob Magnusson - [jmagnusson]
 
 Many thanks to everyone who's contributed to the project.
 
@@ -141,3 +142,4 @@ To contact the author directly:
 [minddust]: https://github.com/minddust
 [asfaltboy]: https://github.com/asfaltboy
 [ottoyiu]: https://github.com/OttoYiu
+[jmagnusson]: https://github.com/jmagnusson