stable
Clone or download
This refactoring is needed before introducing the redirections to the planned-iterations app after an iteration creation/update. The main issues with the existing code were: - Duplication of checks in the pre and post artifact creation/edition hooks - The code in the hooks callbacks are specific to the program_management app. It does not ease the handling of the redirections specific to the iteration creations/updates. It appeared difficult to have hexagonal architecture here since we are in a context of tracker events. If you have a better idea to make the target code better, I'll be glad to consider it. -- No functional change expected part of: story #21385 display Program's iterations Change-Id: Ieedcb578d33a678d777f60c798c988c784807599
Modified Files
Name | ||||
---|---|---|---|---|
M | plugins/program_management/include/RedirectParameterInjector.php | +3 | −3 | Go to diff View file |
R | plugins/program_management/include/EventRedirectAfterArtifactCreationOrUpdateHandler.php | Go to diff View file | ||
A | plugins/program_management/include/RedirectToProgramManagementAppManager.php | +64 | −0 | Go to diff View file |
M | plugins/program_management/include/program_managementPlugin.php | +11 | −7 | Go to diff View file |
A | plugins/program_management/tests/unit/RedirectToProgramIncrementAppHandlerTest.php | +109 | −0 | Go to diff View file |
A | plugins/program_management/tests/unit/RedirectToProgramManagementAppManagerTest.php | +57 | −0 | Go to diff View file |