Enable Form dbio for a list of fields from one table, allow delaying processing

[laundmo]

I would like to write forms which act on a subset of fields like this:

return Form([
	db.tbl.field1,
	db.tbl.field2
])

but currently, this disabled dbio due to a if isinstance(table, list) check.

But if all fields have the same table, dbio is perfectly safe.

Also, to allow more complex logic including potentially multiple forms on a single page (haven't tested this tho), this makes form processing its own method, analogous to Grid

[mdipierro]

Should this now always run after process() even if auto_process is False?

[laundmo]

oh, you're right

[laundmo]

Wait, no, I was right originally.

The original code looks like this:

if not readonly:
    # [...]
    if post_vars:
        self.submitted = True
        if not self.csrf_protection or self._verify_form(post_vars):
            process = True
    if process:
        # [...]
if self.csrf_protection:
    self._sign_form()

We both probably got turned around by the if not self.csrf_protection - thats been moved into the new process method and rewritten to an early return ((not A) or B inverted to A and (not B))

So this is actually matching the original logic exactly. If CSRF is enabled, the form is always signed. What i can't say if that original logic was correct.

[mdipierro]

Should this now always run after process() even if auto_process is False?