Public

Project privacy set to public. By default, its content is available to everyone (authenticated or not). Please note that more restrictive permissions might exist on some items.

stable

Clone or download

Read-only

feat: forbid move when there are no fields to move

Download Browse

part of story #32280 Complete the "move artifact" feature Note: On the backend-side, the move is forbidden only for the duck typed move. Since the behavior already existed for the semantic move, and that this move type will be removed, I think it can be left like that. How to test: - Have a two trackers with incompatible fields - Pick up an artifact in one of these trackers - Try to move it to the second tracker --> The dry run steps says the artifact can't be moved --> The submit button is disabled - Using the web console, remove the "disable" attribute from the submit button, then click it --> A rest error says it cannot be moved. Change-Id: Ia33e844a4856a80fbdd76cc2366c77b457480c40

+221 −13
Thomas Gorka (tgorka) 2023-07-12 17:02
Thomas Gorka (tgorka) 2023-07-13 13:27
3db336d6001ef2df2a1927c84f72b32554adb48d
657ca92f3b45ac50d9fbb5f640bab648c00ea695
git #tuleap/stable/3db336d6001ef2df2a1927c84f72b32554adb48d

Modified Files

Side by side diff
Inline diff
Name
M plugins/tracker/include/REST/v1/Move/MovePatchAction.php +6 −1 Go to diff View file
M plugins/tracker/include/REST/v1/Move/MoveRestArtifact.php +2 −0 Go to diff View file
M plugins/tracker/include/REST/v1/Move/RestArtifactMover.php +9 −0 Go to diff View file
A plugins/tracker/include/Tracker/Exception/MoveArtifactNoValuesToProcessException.php +34 −0 Go to diff View file
M plugins/tracker/scripts/move-artifact-action/po/fr_FR.po +8 −0 Go to diff View file
M plugins/tracker/scripts/move-artifact-action/po/pt_BR.po +6 −0 Go to diff View file
M plugins/tracker/scripts/move-artifact-action/src/components/DryRunNotMigratedFieldState.vue +13 −2 Go to diff View file
M plugins/tracker/scripts/move-artifact-action/src/components/MoveModal.vue +2 −1 Go to diff View file
M plugins/tracker/scripts/move-artifact-action/src/store/actions.js +6 −1 Go to diff View file
M plugins/tracker/scripts/move-artifact-action/src/store/actions.test.js +28 −0 Go to diff View file
M plugins/tracker/scripts/move-artifact-action/src/store/mutations.js +3 −0 Go to diff View file
M plugins/tracker/scripts/move-artifact-action/src/store/state.js +1 −0 Go to diff View file
M plugins/tracker/site-content/fr_FR/LC_MESSAGES/tuleap-tracker.po +5 −0 Go to diff View file
M plugins/tracker/site-content/pt_BR/LC_MESSAGES/tuleap-tracker.po +3 −0 Go to diff View file
M plugins/tracker/tests/unit/Stub/MoveRestArtifactStub.php +22 −6 Go to diff View file
M plugins/tracker/tests/unit/Tracker/REST/v1/Move/MovePatchActionTest.php +24 −0 Go to diff View file
M plugins/tracker/tests/unit/Tracker/REST/v1/Move/RestArtifactMoverTest.php +49 −2 Go to diff View file