Kaydet (Commit) 19d1cb14 authored tarafından Tim Graham's avatar Tim Graham

Fixed #20415 -- Ensured srid isn't localized in OpenLayers JavaScript.

üst 5aa53286
<style type="text/css">{% block map_css %}{% load i18n static %}{% get_current_language_bidi as LANGUAGE_BIDI %} {% load i18n l10n static %}
<style type="text/css">{% block map_css %}{% get_current_language_bidi as LANGUAGE_BIDI %}
#{{ id }}_map { width: {{ map_width }}px; height: {{ map_height }}px; } #{{ id }}_map { width: {{ map_width }}px; height: {{ map_height }}px; }
#{{ id }}_map .aligned label { float: inherit; } #{{ id }}_map .aligned label { float: inherit; }
#{{ id }}_div_map { position: relative; vertical-align: top; float: {{ LANGUAGE_BIDI|yesno:"right,left" }}; } #{{ id }}_div_map { position: relative; vertical-align: top; float: {{ LANGUAGE_BIDI|yesno:"right,left" }}; }
...@@ -25,7 +26,7 @@ ...@@ -25,7 +26,7 @@
id: '{{ id }}', id: '{{ id }}',
map_id: '{{ id }}_map', map_id: '{{ id }}_map',
map_options: map_options, map_options: map_options,
map_srid: {{ map_srid }}, map_srid: {{ map_srid|unlocalize }},
name: '{{ name }}' name: '{{ name }}'
}; };
{% endblock %} {% endblock %}
......
...@@ -34,3 +34,6 @@ Bugfixes ...@@ -34,3 +34,6 @@ Bugfixes
* Fixed a regression in Django 1.8.5 that broke copying a ``SimpleLazyObject`` * Fixed a regression in Django 1.8.5 that broke copying a ``SimpleLazyObject``
with ``copy.copy()`` (:ticket:`26122`). with ``copy.copy()`` (:ticket:`26122`).
* Fixed the ``contrib.gis`` map widgets when using
``USE_THOUSAND_SEPARATOR=True`` (:ticket:`20415`).
...@@ -88,3 +88,6 @@ Bugfixes ...@@ -88,3 +88,6 @@ Bugfixes
* Always included ``geometry_field`` in the GeoJSON serializer output regardless * Always included ``geometry_field`` in the GeoJSON serializer output regardless
of the ``fields`` parameter (:ticket:`26138`). of the ``fields`` parameter (:ticket:`26138`).
* Fixed the ``contrib.gis`` map widgets when using
``USE_THOUSAND_SEPARATOR=True`` (:ticket:`20415`).
...@@ -4,7 +4,7 @@ from django.contrib.gis import forms ...@@ -4,7 +4,7 @@ from django.contrib.gis import forms
from django.contrib.gis.gdal import HAS_GDAL from django.contrib.gis.gdal import HAS_GDAL
from django.contrib.gis.geos import GEOSGeometry from django.contrib.gis.geos import GEOSGeometry
from django.forms import ValidationError from django.forms import ValidationError
from django.test import SimpleTestCase, skipUnlessDBFeature from django.test import SimpleTestCase, override_settings, skipUnlessDBFeature
from django.utils import six from django.utils import six
from django.utils.html import escape from django.utils.html import escape
...@@ -152,6 +152,7 @@ class SpecializedFieldTest(SimpleTestCase): ...@@ -152,6 +152,7 @@ class SpecializedFieldTest(SimpleTestCase):
self.assertTrue(form_instance.is_valid()) self.assertTrue(form_instance.is_valid())
rendered = form_instance.as_p() rendered = form_instance.as_p()
self.assertIn('new MapWidget(options);', rendered) self.assertIn('new MapWidget(options);', rendered)
self.assertIn('map_srid: 4326,', rendered)
self.assertIn('gis/js/OLMapWidget.js', str(form_instance.media)) self.assertIn('gis/js/OLMapWidget.js', str(form_instance.media))
def assertTextarea(self, geom, rendered): def assertTextarea(self, geom, rendered):
...@@ -161,6 +162,8 @@ class SpecializedFieldTest(SimpleTestCase): ...@@ -161,6 +162,8 @@ class SpecializedFieldTest(SimpleTestCase):
self.assertIn('required', rendered) self.assertIn('required', rendered)
self.assertIn(geom.wkt, rendered) self.assertIn(geom.wkt, rendered)
# map_srid in operlayers.html template must not be localized.
@override_settings(USE_L10N=True, USE_THOUSAND_SEPARATOR=True)
def test_pointfield(self): def test_pointfield(self):
class PointForm(forms.Form): class PointForm(forms.Form):
p = forms.PointField() p = forms.PointField()
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment