wiki:S3/S3AAA/RecordApproval

Version 2 (modified by Dominic König, 12 years ago) ( diff )

--

Record Approval

The S3 Record Approval Framework hides records which have not yet been reviewed and approved by an authorized user.

Unapproved Records

Whether a record is approved or not is encoded as approved_by field in the record. The field is either None (unapproved record) or holds the user ID of the approving user (approved record).

For tables which require approval, this field is mandatory (field type "integer" or "reference auth_user"). The default value for this field is None, i.e. all new records are unapproved by default. This field should not be editable, i.e. writable=False.

Workflow

The intended workflow for approval is:

  • user creates a new record => record is unapproved by default (i.e. approved_by=None)
  • unapproved records are treated as invalid and excluded from any kind of workflow (i.e. invisible) except for review/approval
  • user with approval role reviews the record and either approves or rejects it
  • approve sets the approved_by field of the record to the approving users ID
  • reject deletes the record and all its dependencies
  • approved records are now available for all workflows, except for review/approval

Developer notes:

  • it should not be possible to import pre-approved records, otherwise this would allow unauthorized users to circumvent the approval framework
  • unapproved records should be disregarded in Sync, because otherwise this introduces the same hole as above: the user could approve the record in a remote instance where he is Admin and synchronize it back. Apart from that, circulation of unapproved records turns the purpose of approval somewhat upside down.

Configuration

The Record Approval Framework can be turned on/off globally by a deployment setting:

settings.auth.record_approval = True

Default is False.

In addition to activating record approval, you will also need to set the role which is permitted to review/approve/reject records:

settings.auth.record_approver_role = <role_uid>

The permission to review/approve/reject records of this role is realm-limited (see S3AAA/OrgAuth for more details).

Admin can always see all records, regardless whether they are approved or not. Other users can only see approved records, unless they have the approver role and use one of the approval methods "review", "approve", "reject".

Whether record approval is required can be configured per table like:

s3db.configure(tablename, requires_approval=True)

Default is False.

For a deployment-specific approval configuration, you can override all table-specific settings with:

settings.auth.record_approval_required_for = [tablename, tablename]

Record approval will then only be applied to those tables in the list. If you set this to None, it will fall back to the table-specific settings.

Methods to Approve or Reject Records

S3Resource implements two low-level methods for record approval:

resource.approve()

...to approve all records in a resource (and all its components), and

resource.reject()

...to reject (=delete) all unapproved records in a resource.

S3CRUD shall implement the following methods:

/controller/function/review

...to see all unapproved records in a resource, or:

/controller/function/XY/review

...to review a particular record, which then gives the user the option to either:

POST /controller/function/XY/approve

...approve the record, or:

POST /controller/function/XY/reject

...reject it.

NOTE: resource.reject() is much more radical about record deletion than delete() - it will try to bypass any RESTRICTs and rigirously delete any dependency of the rejected records so they can not leave any undesired legacy behind. However, that means that reject() must always be properly authorized and strictly limited to unapproved records in tables which require approval.

Developer note: GET to approve or reject shall forward to review (for REST compliance)

Developer note: If approve or reject are called without record ID, they shall raise a "Record Not Found" error rather than to approve/reject all unapproved records as per the requirement that a record can only be approved/rejects after it has been reviewed (=no bulk approval/reject).

Callbacks

For both, resource.approve() and resource.reject(), there are hooks which will be called for each record that gets approved/rejected:

s3db.configure(tablename, onapprove=function)
s3db.configure(tablename, onreject=function)

The hook function must table (table, row) as parameter, and doesn't return anything.

Note: See TracWiki for help on using the wiki.