rattail/tailbone
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

Add some smarts when making batch execution form schema

Browse source 
in some cases `has_execution_options()` may return True but the base
view class may not need to provide any options itself (i.e. subclass
is responsible for declaring the view has options).
This commit is contained in:
Lance Edgar 2021-12-15 15:02:28 -06:00
parent ca57bd3572
commit f49fdebd98
1 changed files with 16 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
tailbone/views/batch/core.py
View file

@ -786,10 +786,12 @@ class BatchMasterView(MasterView):
route_prefix = self.get_route_prefix() route_prefix = self.get_route_prefix()
use_buefy = self.get_use_buefy() use_buefy = self.get_use_buefy()
schema = None
if self.has_execution_options(batch): if self.has_execution_options(batch):
if batch is None: if batch is None:
batch = self.model_class batch = self.model_class
schema = self.make_execute_schema(batch) schema = self.make_execute_schema(batch)
if schema:
for field in schema: for field in schema:
# if field does not yet have a default, maybe provide one from session storage # if field does not yet have a default, maybe provide one from session storage
@ -807,7 +809,7 @@ class BatchMasterView(MasterView):
if use_buefy and isinstance(field.widget, forms.widgets.PlainSelectWidget): if use_buefy and isinstance(field.widget, forms.widgets.PlainSelectWidget):
field.widget = dfwidget.SelectWidget(values=field.widget.values) field.widget = dfwidget.SelectWidget(values=field.widget.values)
else: if not schema:
schema = colander.Schema() schema = colander.Schema()
kwargs['use_buefy'] = use_buefy kwargs['use_buefy'] = use_buefy