Ticket #16790: 16790.2.diff

django/contrib/gis/admin/options.py

@@ -39 +39 @@
     debug = False
     widget = OpenLayersWidget
 
-    def _media(self):
+    @property
+    def media(self):
         "Injects OpenLayers JavaScript into the admin."
-        media = super(GeoModelAdmin, self)._media()
+        media = super(GeoModelAdmin, self).media
         media.add_js([self.openlayers_url])
         media.add_js(self.extra_js)
         return media
-    media = property(_media)
 
     def formfield_for_dbfield(self, db_field, **kwargs):
         """

django/contrib/gis/tests/__init__.py

@@ -29 +29 @@
 
     # The following GeoDjango test apps depend on GDAL support.
     if HAS_GDAL:
+        # Geographic admin requires GDAL
+        apps.append('geoadmin')
+
         # 3D apps use LayerMapping, which uses GDAL.
         if connection.ops.postgis and GEOS_PREPARE:
             apps.append('geo3d')

new file django/contrib/gis/tests/geoadmin/models.py

@@ +1 @@
+from django.contrib.gis.db import models
+from django.contrib.gis import admin
+
+class City(models.Model):
+    name = models.CharField(max_length=30)
+    point = models.PointField()
+    objects = models.GeoManager()
+    def __unicode__(self): return self.name
+
+admin.site.register(City, admin.OSMGeoAdmin)

new file django/contrib/gis/tests/geoadmin/tests.py

@@ +1 @@
+from django.test import TestCase
+from django.contrib.gis import admin
+from models import City
+
+class GeoAdminTest(TestCase):
+    urls = 'django.contrib.gis.tests.geoadmin.urls'
+
+    def test01_ensure_geographic_media(self):
+        geoadmin = admin.site._registry[City]
+        # TODO: Ensure `geoadmin.media` has what we expect.

new file django/contrib/gis/tests/geoadmin/urls.py

@@ +1 @@
+from django.conf.urls.defaults import *
+from django.contrib import admin
+
+urlpatterns = patterns('',
+    (r'^admin/', include(admin.site.urls)),
+)