Adding SQL support on Worflow->Transition condition

Description

Adempiere and iDempiere has not implemented SQL logic in workflow transition.
I proposed following design and discuss with hengsin.

Allowing user to enter SQL which return either Y or N and return value used to check condition.
For security measure, we add method to check that sql entered should not be update, delete or insert.

Environment

None

Attachments

2

Activity

Show:

Heng Sin Low November 15, 2021 at 2:37 AM

Will add the patch here to the PR for

Deepak Pansheriya June 25, 2020 at 10:52 AM
Edited

Triaged by

As suggestion. Needs to mark SQL field as advanced to avoid normal user change it.

Instead of using Value to pass SQL, add additional field SQLLogic, which is visible when operation is SQL. This field is editable for advanced role only. For other Role, it should be read only.

Carlos’ suggestions:

make a readonlylogic to make it readonly when is not an advanced role - sometimes this is better because users can still SEE the configuration but cannot change it

if is possible too - better to make it compatible with the way Heng Sin implemented the @SQL logic for fields

Deepak Pansheriya March 26, 2014 at 8:34 AM

Patch "Handling Exception case for SQL Operation.patch" need to be applied to fix exception saying "No Column defined".

Deepak Pansheriya March 26, 2014 at 6:38 AM

Low,

Patch is attached. Please review.

Duplicate

Details

Assignee

Reporter

Components

Priority

Created March 19, 2014 at 1:16 PM
Updated November 15, 2021 at 2:37 AM
Resolved November 15, 2021 at 2:37 AM