[hotfix] fixes issue around multidict

apache · Jun 11, 2016 · 327fcee · 327fcee
1 parent 1631137
commit 327fcee
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/caravel/viz.py b/caravel/viz.py
@@ -115,8 +115,14 @@ def get_url(self, **kwargs):
             if d[key] is False:
                 del d[key]
             else:
-                for v in d.getlist(key):
-                    od.add(key, v)
+                if isinstance(d, MultiDict):
+                    v = d.getlist(key)
+                else:
+                    v = d.get(key)
+                if not isinstance(v, list):
+                    v = [v]
+                for item in sorted(v):
+                    od.add(key, item)
         href = Href(
             '/caravel/explore/{self.datasource.type}/'
             '{self.datasource.id}/'.format(**locals()))