Refactor "departments by vendor" grid to use newer-style

tailbone/static/js/tailbone.js

@@ -325,6 +325,7 @@ $(function() {
     });
 
     /*
+     * TODO: this should be deprecated; for old grids only?
      * Add "check all" functionality to tables with checkboxes.
      */
     $('body').on('click', '.grid thead th.checkbox input[type="checkbox"]', function() {

tailbone/views/departments.py

@@ -28,7 +28,7 @@ from __future__ import unicode_literals, absolute_import
 
 from rattail.db import model
 
-from tailbone.views import MasterView, AutocompleteView, AlchemyGridView
+from tailbone.views import MasterView, AutocompleteView
 from tailbone.newgrids import AlchemyGrid, GridAction
 
 
@@ -79,16 +79,11 @@ class DepartmentsView(MasterView):
             kwargs['employees'] = None
         return kwargs
 
-
-class DepartmentsByVendorGrid(AlchemyGridView):
-
-    mapped_class = model.Department
-    config_prefix = 'departments.by_vendor'
-    checkboxes = True
-    partial_only = True
-
-    def query(self):
-        return self.make_query()\
+    def list_by_vendor(self):
+        """
+        View list of departments by vendor
+        """
+        data = self.Session.query(model.Department)\
                            .outerjoin(model.Product)\
                            .join(model.ProductCost)\
                            .join(model.Vendor)\
@@ -96,14 +91,32 @@ class DepartmentsByVendorGrid(AlchemyGridView):
                            .distinct()\
                            .order_by(model.Department.name)
 
-    def grid(self):
-        g = self.make_grid()
-        g.configure(
-            include=[
+        def configure(g):
+            g.configure(include=[
                 g.name,
-                ],
-            readonly=True)
-        return g
+            ], readonly=True)
+
+        def row_attrs(row, i):
+            return {'data-uuid': row.uuid}
+
+        grid = self.make_grid(data=data, sortable=False, filterable=False, pageable=False,
+                              configure=configure, width=None, checkboxes=True,
+                              row_attrs=row_attrs, main_actions=[], more_actions=[])
+        self.request.response.content_type = b'text/html'
+        self.request.response.text = grid.render_grid()
+        return self.request.response
+
+    @classmethod
+    def defaults(cls, config):
+        route_prefix = cls.get_route_prefix()
+        url_prefix = cls.get_url_prefix()
+        permission_prefix = cls.get_permission_prefix()
+
+        config.add_route('{}.by_vendor'.format(route_prefix), '{}/by-vendor'.format(url_prefix))
+        config.add_view(cls, attr='list_by_vendor', route_name='{}.by_vendor'.format(route_prefix),
+                        permission='{}.list'.format(permission_prefix))
+
+        cls._defaults(config)
 
 
 class DepartmentsAutocomplete(AutocompleteView):
@@ -119,9 +132,4 @@ def includeme(config):
     config.add_view(DepartmentsAutocomplete, route_name='departments.autocomplete',
                     renderer='json', permission='departments.list')
 
-    # departments by vendor list
-    config.add_route('departments.by_vendor',           '/departments/by-vendor')
-    config.add_view(DepartmentsByVendorGrid,route_name='departments.by_vendor',
-                    permission='departments.list')
-
     DepartmentsView.defaults(config)

tailbone/views/master.py

@@ -1339,9 +1339,16 @@ class MasterView(View):
         """
         factory = self.get_grid_factory()
         key = self.get_grid_key()
+        if 'data' in kwargs:
+            data = kwargs.pop('data')
+        else:
             data = self.get_data(session=kwargs.get('session'))
+        configure = kwargs.pop('configure', None)
         kwargs = self.make_grid_kwargs(**kwargs)
         grid = factory(key, self.request, data=data, model_class=self.get_model_class(error=False), **kwargs)
+        if configure:
+            configure(grid)
+        else:
             self._preconfigure_grid(grid)
             self.configure_grid(grid)
         grid.load_settings()