fixed blank field string reversing

simon-the-shark · May 11, 2020 · 539ccf7 · 539ccf7
1 parent ead5f5b
commit 539ccf7
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/mapbox_location_field/forms.py b/mapbox_location_field/forms.py
@@ -6,6 +6,8 @@
 
 
 def reverse_tuple_string(location_string):
+    if location_string == "":
+        return location_string
     args = location_string.split(",")
     return args[1] + "," + args[0]
 

diff --git a/mapbox_location_field/tests/test_forms.py b/mapbox_location_field/tests/test_forms.py
@@ -1,5 +1,4 @@
 from django.contrib.gis.geos import Point
-from django.core.exceptions import ValidationError
 from django.test import TestCase
 
 from mapbox_location_field.forms import LocationField, AddressAutoHiddenField, reverse_tuple_string
@@ -25,6 +24,7 @@ def test_clean(self):
         field = LocationField()
         self.assertEqual("11,12", field.clean("12, 11"))
 
+
 class SpatialLocationFieldTests(TestCase):
 
     def test_widget(self):
@@ -67,6 +67,10 @@ def test_to_python(self):
     def test_reverse_tuple_string(self):
         self.assertEqual(reverse_tuple_string("1,2"), "2,1")
 
+    def test_empty_reverse_tuple_string(self):
+        self.assertEqual(reverse_tuple_string(""), "")
+
+
 class AddressAutoHiddenFieldTests(TestCase):
     def test_widget(self):
         field = AddressAutoHiddenField()