Minor fixes and improvements

flask_admin/contrib/geoa/widgets.py

@@ -10,6 +10,8 @@ def lng(pt):
 
 
 class LeafletWidget(TextArea):
+    data_role = 'leaflet'
+
     """
         `Leaflet <http://leafletjs.com/>`_ styled map widget. Inherits from
         `TextArea` so that geographic data can be stored via the <textarea>
@@ -31,7 +33,7 @@ class LeafletWidget(TextArea):
         self.max_bounds = max_bounds
 
     def __call__(self, field, **kwargs):
-        kwargs.setdefault('data-role', 'leaflet')
+        kwargs.setdefault('data-role', self.data_role)
         gtype = getattr(field, "geometry_type", "GEOMETRY")
         kwargs.setdefault('data-geometry-type', gtype)
 

flask_admin/helpers.py

@@ -56,7 +56,7 @@ def is_form_submitted():
     """
         Check if current method is PUT or POST
     """
-    return request and request.method in ("PUT", "POST")
+    return request and request.method in ('PUT', 'POST')
 
 
 def validate_form_on_submit(form):

flask_admin/model/base.py

@@ -1432,4 +1432,4 @@ class BaseModelView(BaseView, ActionsMixin):
             abort(404)
 
         data = [loader.format(m) for m in loader.get_list(query, offset, limit)]
-        return Response(json.dumps(data), mimetype='application/json')
\ No newline at end of file
+        return Response(json.dumps(data), mimetype='application/json')

flask_admin/model/widgets.py

@@ -26,8 +26,8 @@ class AjaxSelect2Widget(object):
         self.multiple = multiple
 
     def __call__(self, field, **kwargs):
-        kwargs['data-role'] = u'select2-ajax'
-        kwargs['data-url'] = get_url('.ajax_lookup', name=field.loader.name)
+        kwargs.setdefault('data-role', 'select2-ajax')
+        kwargs.setdefault('data-url', get_url('.ajax_lookup', name=field.loader.name))
 
         allow_blank = getattr(field, 'allow_blank', False)
         if allow_blank and not self.multiple:

flask_admin/static/admin/js/form-1.0.0.js

@@ -476,7 +476,7 @@
       * @param {converter} function($el, name)
       */
       this.addFieldConverter = function(converter) {
-          fieldConverters.push(converter);
+        fieldConverters.push(converter);
       };
     };