stable
Clone or download
In order to prepare the field for future contributions, we should have a net safety by extracting logic in dedicated object with better test coverage. Redirection was introduced in planning view v1, but mostly disappeared with introduction of pv2 and its artifact modal. Redirection was used to be able to create/edit artifact in tracker and being redirected to the agiledashboard in the right pane, with autolinking. Today, this redirection mechanism is only used in details (overview) pane (and in legacy cardwall, but its irrelevant for the current contribution). I'm not sure if all redirect features are still used, but I blindly extracted the whole stuff in order to focus on the next addition: redirection from the +New dropdown that will use more or less the same mechanism. No functional changes are expected. Part of story #16206: ยซ + New ยป button, better contextualisation Change-Id: I95e2b5076f5548dcab16411acad898219d960c07
Modified Files
Name | ||||
---|---|---|---|---|
A | plugins/agiledashboard/include/AgileDashboard/Artifact/EventRedirectAfterArtifactCreationOrUpdateHandler.php | +142 | โ0 | Go to diff View file |
A | plugins/agiledashboard/include/AgileDashboard/Artifact/RedirectParameterInjector.php | +71 | โ0 | Go to diff View file |
M | plugins/agiledashboard/include/agiledashboardPlugin.php | +15 | โ78 | Go to diff View file |
A | plugins/agiledashboard/tests/unit/AgileDashboard/Artifact/EventRedirectAfterArtifactCreationOrUpdateHandlerTest.php | +287 | โ0 | Go to diff View file |
A | plugins/agiledashboard/tests/unit/AgileDashboard/Artifact/RedirectParameterInjectorTest.php | +149 | โ0 | Go to diff View file |