Patch #33337
closed
Clean-up workflows controller
Added by Vincent Robert over 4 years ago.
Updated over 3 years ago.
Category:
Code cleanup/refactoring
Description
Hello
Here is a patch which aims to refactor the WorkflowsController.
The patch does not change the behavior of the application, but use smaller methods and clearer code. The routes are also updated.
My goal was to use a specific action for each route, and do not use a same method for different purposes (so avoid checking if the request-method is "post" inside actions).
Hope this may help.
Files
- Assignee set to Marius BĂLTEANU
- File 0001-Clean-up-workflows-controller.patch added
- File 0002-Fix-route-name.patch added
- Target version set to Candidate for next major release
- File deleted (
0001-Clean-up-workflows-controller.patch)
- File deleted (
0002-Fix-route-name.patch)
Update the patches for current trunk, the tests still pass.
Setting target to 5.0.0, I think it's a nice refactoring.
- Status changed from New to Closed
- Assignee set to Go MAEDA
Committed the patch. Thank you for your contribution.
- Subject changed from Clean-up Workflows Controller to Clean-up workflows controller
Thank you for updating and committing this patch.
Also available in: Atom
PDF