Changed URLField to CharField in GeoJSON and GeoRSS in base synchroni…

…zers (related to #178) The default django implementation of URLField is too restrictive and doesn't follow the URL parsing standard, therefore we switch back to CharField.
ninuxorg · Oct 9, 2014 · 49c295a · 49c295a
1 parent 49604d4
commit 49c295a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/nodeshot/interop/sync/synchronizers/base.py b/nodeshot/interop/sync/synchronizers/base.py
@@ -169,8 +169,9 @@ class GenericGisSynchronizer(HttpRetrieverMixin, BaseSynchronizer):
     SCHEMA = [
         {
             'name': 'url',
-            'class': 'URLField',
+            'class': 'CharField',
             'kwargs': {
+                'max_length': 128,
                 'help_text': _('URL containing geographical data')
             }
         },

diff --git a/nodeshot/interop/sync/synchronizers/georss.py b/nodeshot/interop/sync/synchronizers/georss.py
@@ -10,9 +10,10 @@ class GeoRss(XMLParserMixin, GenericGisSynchronizer):
     SCHEMA = [
         {
             'name': 'url',
-            'class': 'URLField',
+            'class': 'CharField',
             'kwargs': {
-                'help_text': _('URL containing geographical data')
+                'help_text': _('URL containing geographical data'),
+                'max_length': 128,
             }
         },
         {