feat(edit-content) : Allowing Users to add separators between workflow actions

[jcastro-dotcms]

Proposed Changes

🤖 Generated by Copilot at 6bdd0f6

This pull request adds a new feature to the workflow system, which allows creating separator actions that can be used to organize and group other actions in a workflow step. It also refactors and cleans up some of the existing code related to workflow actions, and adds a startup task and a database column to support JSONB metadata for actions. The main files affected are WorkflowAction, WorkflowActionForm, WorkflowFactoryImpl, WorkflowSQL, WorkflowHelper, WorkflowAPITest, view_steps_filtered.jsp, and Task231009AddMetadataColumnToWorkflowAction.

🤖 Generated by Copilot at 6bdd0f6

• Add a new metadata field to the WorkflowAction class and its related classes, to store additional configuration properties for the action (link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link)
• Add a new constant for the separator actionId, which is a special type of action that groups other actions in the UI, and a new method to create a separator action form in the WorkflowActionForm builder (link, link)
• Refactor the save method in the WfActionAjax class, to use the WorkflowActionForm builder and the separator method to create a separator action form, and to use the ExceptionUtil class to get the error message from an exception (link)
• Add a new method to save a separator action in the WfActionAjax class, by calling the save method with the separator actionId and the schemeId and stepId parameters (link)
• Add a new script tag and a div element in the view_steps_filtered.jsp file, to define a function to add a separator action to a workflow step, and to display a button to call the function (link, link)
• Reorder the parameters of the addWorkflowAction method calls in the WorkflowAPITest class, to match the new signature of the method that takes the nextStep and stepId parameters before the whoCanUse and schemeId parameters (link)
• Add a new method to create a WorkflowAction with a subtractTwoNumbers name and a subtract actionlet class in the WorkflowAPITest class, for testing purposes (link)
• Add a new test method that verifies that a WorkflowAction with a separator subtype can be created and retrieved, and that the separator is included in the available actions for a contentlet in the WorkflowAPITest class (link)
• Update the javadoc of the saveAction method in the WorkflowHelper class, to include the user parameter and to clarify the behavior of the stepId parameter (link)
• Improve the error logging and handling in the saveAction method in the WorkflowHelper class, by using the ExceptionUtil class to get the error message, and by including the actionId and actionName in the log message (link)
• Remove the unused imports and queries from the WorkflowHelper, WorkflowFactoryImpl, WorkflowSQL, and WfActionAjax classes, to clean up the code and avoid confusion (link, link, link, link, link, link, link, link, link, link, link)
• Reorder the imports for the WorkflowHelper, WorkflowFactoryImpl, and WfActionAjax classes, to follow the alphabetical order and the grouping by package (link, link, link)
• Add the final modifier to the row parameter in the convertListToObjects method in the WorkflowFactoryImpl class, to follow the code style and avoid reassignment (link)
• Add the metadata column to the workflow_action table in the postgres.sql script, to create the JSONB column in the database (link)
• Add javadoc comments for the WorkflowSQL and WfActionAjax classes, to document their purpose and usage, and to mark the WfActionAjax class as deprecated for removal, since it is replaced by the REST endpoints (link, link)

Screenshot

[jdotcms]

Very good job

[dotcms-sonarqube]

SonarQube Quality Gate

0 Bugs
0 Vulnerabilities
0 Security Hotspots
3 Code Smells

0.6% Coverage
0.3% Duplication

[github-actions]

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days.

[github-actions]

This PR was closed because it has been stalled with no activity.

[dotcms-sonarqube]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
0.6% Coverage on New Code
0.3% Duplication on New Code

See analysis details on SonarQube