Commit e1643e35 authored by Florian Apolloner's avatar Florian Apolloner
Browse files

Don't escape object ids when passing to the contenttypes.shortcut view. 

This commit also changes the string pk to string_pk instead of id, to test
if the admin uses .pk throughout the codebase.
parent 840ffd80

django/contrib/admin/templates/admin/change_form.html

+1 −1
Original line number Diff line number Diff line
@@ -31,7 +31,7 @@ 
  <ul class="object-tools">

    {% block object-tools-items %}

    <li><a href="history/" class="historylink">{% trans "History" %}</a></li>

    {% if has_absolute_url %}<li><a href="../../../r/{{ content_type_id }}/{{ object_id }}/" class="viewsitelink">{% trans "View on site" %}</a></li>{% endif%}

    {% if has_absolute_url %}<li><a href="../../../r/{{ content_type_id }}/{{ original.pk }}/" class="viewsitelink">{% trans "View on site" %}</a></li>{% endif%}

    {% endblock %}

  </ul>

{% endif %}{% endif %}

tests/regressiontests/admin_views/fixtures/string-primary-key.xml

+3 −3
Original line number Diff line number Diff line 
<?xml version="1.0" encoding="utf-8"?>

<django-objects version="1.0">

    <object pk="1" model="admin_views.modelwithstringprimarykey">

        <field type="CharField" name="id"><![CDATA[abcdefghijklmnopqrstuvwxyz ABCDEFGHIJKLMNOPQRSTUVWXYZ 1234567890 -_.!~*'() ;/?:@&=+$, <>#%" {}|\^[]`]]></field>

        <field type="CharField" name="string_pk"><![CDATA[abcdefghijklmnopqrstuvwxyz ABCDEFGHIJKLMNOPQRSTUVWXYZ 1234567890 -_.!~*'() ;/?:@&=+$, <>#%" {}|\^[]`]]></field>

    </object>

</django-objects>

tests/regressiontests/admin_views/models.py

+5 −2
Original line number Diff line number Diff line
@@ -93,10 +93,13 @@ class CustomArticle(models.Model): 




class ModelWithStringPrimaryKey(models.Model):

    id = models.CharField(max_length=255, primary_key=True)

    string_pk = models.CharField(max_length=255, primary_key=True)



    def __unicode__(self):

        return self.id

        return self.string_pk



    def get_absolute_url(self):

        return u'/dummy/%s/' % self.string_pk





class Color(models.Model):

tests/regressiontests/admin_views/tests.py

+11 −3
Original line number Diff line number Diff line
@@ -1403,7 +1403,7 @@ class AdminViewStringPrimaryKeyTest(TestCase): 


    def test_url_conflicts_with_add(self):

        "A model with a primary key that ends with add should be visible"

        add_model = ModelWithStringPrimaryKey(id="i have something to add")

        add_model = ModelWithStringPrimaryKey(pk="i have something to add")

        add_model.save()

        response = self.client.get('/test_admin/admin/admin_views/modelwithstringprimarykey/%s/' % quote(add_model.pk))

        should_contain = """<h1>Change model with string primary key</h1>"""
@@ -1411,7 +1411,7 @@ class AdminViewStringPrimaryKeyTest(TestCase): 


    def test_url_conflicts_with_delete(self):

        "A model with a primary key that ends with delete should be visible"

        delete_model = ModelWithStringPrimaryKey(id="delete")

        delete_model = ModelWithStringPrimaryKey(pk="delete")

        delete_model.save()

        response = self.client.get('/test_admin/admin/admin_views/modelwithstringprimarykey/%s/' % quote(delete_model.pk))

        should_contain = """<h1>Change model with string primary key</h1>"""
@@ -1419,12 +1419,20 @@ class AdminViewStringPrimaryKeyTest(TestCase): 


    def test_url_conflicts_with_history(self):

        "A model with a primary key that ends with history should be visible"

        history_model = ModelWithStringPrimaryKey(id="history")

        history_model = ModelWithStringPrimaryKey(pk="history")

        history_model.save()

        response = self.client.get('/test_admin/admin/admin_views/modelwithstringprimarykey/%s/' % quote(history_model.pk))

        should_contain = """<h1>Change model with string primary key</h1>"""

        self.assertContains(response, should_contain)



    def test_shortcut_view_with_escaping(self):

        "'View on site should' work properly with char fields"

        model = ModelWithStringPrimaryKey(pk='abc_123')

        model.save()

        response = self.client.get('/test_admin/admin/admin_views/modelwithstringprimarykey/%s/' % quote(model.pk))

        should_contain = '/%s/" class="viewsitelink">' % model.pk

        self.assertContains(response, should_contain)





@override_settings(PASSWORD_HASHERS=('django.contrib.auth.hashers.SHA1PasswordHasher',))

class SecureViewTests(TestCase):