stable
Clone or download
Read-only
part story #32280 Complete the "move artifact" feature Fields are moved based on their name either they exist are are moved, either they are ignored This is the basic first step, inconsistencies remains (for list, date...) In order to don't commit breaking change on master and to limit the patch size, everything is behind a feature flag, code is designed to be easily removed (original or new code) Activate the feature glag with: tuleap config-set feature_flag_enable_complete_move_artifact 1 Change-Id: I38bf54980085c8901e6413b980f05c15e1044c28
Modified Files
| Name | ||||
|---|---|---|---|---|
| M | plugins/tracker/include/REST/v1/ArtifactPatchDryRunFieldsResponseRepresentation.php | +32 | −11 | Go to diff View file |
| M | plugins/tracker/include/REST/v1/ArtifactPatchDryRunResponseRepresentation.php | +12 | −4 | Go to diff View file |
| M | plugins/tracker/include/REST/v1/ArtifactPatchResponseRepresentation.php | +10 | −3 | Go to diff View file |
| M | plugins/tracker/include/REST/v1/ArtifactsResource.class.php | +87 | −102 | Go to diff View file |
| A | plugins/tracker/include/REST/v1/Move/AddPostMoveArtifactFeedback.php | +31 | −0 | Go to diff View file |
| A | plugins/tracker/include/REST/v1/Move/BeforeMoveChecker.php | +74 | −0 | Go to diff View file |
| A | plugins/tracker/include/REST/v1/Move/CheckBeforeMove.php | +32 | −0 | Go to diff View file |
| A | plugins/tracker/include/REST/v1/Move/DryRunMover.php | +64 | −0 | Go to diff View file |
| A | plugins/tracker/include/REST/v1/Move/HeaderForMoveSender.php | +34 | −0 | Go to diff View file |
| A | plugins/tracker/include/REST/v1/Move/MoveDryRun.php | +38 | −0 | Go to diff View file |
| A | plugins/tracker/include/REST/v1/Move/MovePatchAction.php | +98 | −0 | Go to diff View file |
| A | plugins/tracker/include/REST/v1/Move/MoveRestArtifact.php | +43 | −0 | Go to diff View file |
| R | plugins/tracker/include/REST/v1/PostMoveArticfactRESTAction.php | Go to diff View file | ||
| A | plugins/tracker/include/REST/v1/Move/RestArtifactMover.php | +95 | −0 | Go to diff View file |
| A | plugins/tracker/include/REST/v1/MoveArtifactCompleteFeatureFlag.php | +39 | −0 | Go to diff View file |
| A | plugins/tracker/include/Tracker/Action/CheckFieldTypeCompatibility.php | +31 | −0 | Go to diff View file |
| A | plugins/tracker/include/Tracker/Action/CheckMoveArtifact.php | +42 | −0 | Go to diff View file |
| A | plugins/tracker/include/Tracker/Action/CollectDryRunTypingField.php | +28 | −0 | Go to diff View file |
| A | plugins/tracker/include/Tracker/Action/DryRunDuckTypingFieldCollector.php | +60 | −0 | Go to diff View file |
| A | plugins/tracker/include/Tracker/Action/DuckTypedMoveFieldCollection.php | +48 | −0 | Go to diff View file |
| A | plugins/tracker/include/Tracker/Action/FieldMapping.php | +38 | −0 | Go to diff View file |
| A | plugins/tracker/include/Tracker/Action/FieldTypeCompatibilityChecker.php | +53 | −0 | Go to diff View file |
| A | plugins/tracker/include/Tracker/Action/MegaMoverArtifact.php | +141 | −0 | Go to diff View file |
| A | plugins/tracker/include/Tracker/Action/MegaMoverArtifactByDuckTyping.php | +118 | −0 | Go to diff View file |
| M | plugins/tracker/include/Tracker/Action/Move/FeedbackFieldCollector.php | +7 | −7 | Go to diff View file |
| M | plugins/tracker/include/Tracker/Action/Move/FeedbackFieldCollectorInterface.php | +19 | −4 | Go to diff View file |
| M | plugins/tracker/include/Tracker/Action/Move/NoFeedbackFieldCollector.php | +19 | −4 | Go to diff View file |
| M | plugins/tracker/include/Tracker/Action/MoveArtifact.php | +10 | −151 | Go to diff View file |
| A | plugins/tracker/include/Tracker/Action/MoveArtifactByDuckTyping.php | +32 | −0 | Go to diff View file |
| M | plugins/tracker/include/Tracker/Admin/ArtifactsDeletion/ArtifactsDeletionConfig.php | +5 | −15 | Go to diff View file |
| M | plugins/tracker/include/Tracker/Artifact/ActionButtons/ArtifactMoveButtonPresenterBuilder.php | +6 | −3 | Go to diff View file |
| M | plugins/tracker/include/Tracker/Artifact/Artifact.php | +6 | −11 | Go to diff View file |
| M | plugins/tracker/include/Tracker/Artifact/ArtifactsDeletion/ArtifactDeletionLimitRetriever.php | +4 | −15 | Go to diff View file |
| M | plugins/tracker/include/Tracker/Artifact/ArtifactsDeletion/AsynchronousArtifactsDeletionActionsRunner.php | +1 | −1 | Go to diff View file |
| A | plugins/tracker/include/Tracker/Artifact/ArtifactsDeletion/RetrieveActionDeletionLimit.php | +34 | −0 | Go to diff View file |
| M | plugins/tracker/include/Tracker/Artifact/Tracker_ArtifactFactory.class.php | +1 | −1 | Go to diff View file |
| M | plugins/tracker/include/Tracker/Artifact/dao/PriorityDao.class.php | +3 | −7 | Go to diff View file |
| M | plugins/tracker/include/Tracker/FormElement/Field/RetrieveUsedFields.php | +2 | −0 | Go to diff View file |
| A | plugins/tracker/include/Tracker/FormElement/RetrieveFieldType.php | +30 | −0 | Go to diff View file |
| M | plugins/tracker/include/Tracker/FormElement/Tracker_FormElementFactory.php | +9 | −24 | Go to diff View file |
| M | plugins/tracker/include/Tracker/FormElement/Tracker_SharedFormElementFactory.php | +11 | −9 | Go to diff View file |
| M | plugins/tracker/include/Tracker/XML/Updater/MoveChangesetXMLUpdater.php | +98 | −1 | Go to diff View file |
| M | plugins/tracker/include/trackerPlugin.php | +2 | −0 | Go to diff View file |
| M | plugins/tracker/tests/unit/Artifact/Changeset/Tracker_Artifact_Changeset_ChangesetDataInitializatorTest.php | +4 | −4 | Go to diff View file |
| M | plugins/tracker/tests/unit/Builders/ArtifactTestBuilder.php | +14 | −1 | Go to diff View file |
| M | plugins/tracker/tests/unit/Builders/TrackerFormElementStringFieldBuilder.php | +8 | −1 | Go to diff View file |
| A | plugins/tracker/tests/unit/Stub/AddPostMoveArtifactFeedbackStub.php | +51 | −0 | Go to diff View file |
| A | plugins/tracker/tests/unit/Stub/CheckBeforeMoveStub.php | +52 | −0 | Go to diff View file |
| A | plugins/tracker/tests/unit/Stub/CheckFieldTypeCompatibilityStub.php | +52 | −0 | Go to diff View file |
| A | plugins/tracker/tests/unit/Stub/CheckMoveArtifactStub.php | +66 | −0 | Go to diff View file |
| A | plugins/tracker/tests/unit/Stub/CollectDryRunTypingFieldStub.php | +57 | −0 | Go to diff View file |
| A | plugins/tracker/tests/unit/Stub/FeedbackFieldCollectorInterfaceStub.php | +118 | −0 | Go to diff View file |
| A | plugins/tracker/tests/unit/Stub/MoveArtifactByDuckTypingStub.php | +55 | −0 | Go to diff View file |
| A | plugins/tracker/tests/unit/Stub/MoveArtifactStub.php | +57 | −0 | Go to diff View file |
| A | plugins/tracker/tests/unit/Stub/MoveDryRunStub.php | +57 | −0 | Go to diff View file |
| A | plugins/tracker/tests/unit/Stub/MoveRestArtifactStub.php | +85 | −0 | Go to diff View file |
| A | plugins/tracker/tests/unit/Stub/RetrieveActionDeletionLimitStub.php | +63 | −0 | Go to diff View file |
| A | plugins/tracker/tests/unit/Stub/RetrieveFieldTypeStub.php | +43 | −0 | Go to diff View file |
| M | plugins/tracker/tests/unit/Stub/RetrieveUsedFieldsStub.php | +11 | −0 | Go to diff View file |
| A | plugins/tracker/tests/unit/Tracker/Action/DryRunDuckTypingFieldCollectorTest.php | +127 | −0 | Go to diff View file |
| A | plugins/tracker/tests/unit/Tracker/Action/FieldTypeCompatibilityCheckerTest.php | +52 | −0 | Go to diff View file |
| A | plugins/tracker/tests/unit/Tracker/Action/MegaMoverArtifactByDuckTypingTest.php | +127 | −0 | Go to diff View file |
| A | plugins/tracker/tests/unit/Tracker/REST/v1/Move/ArtifactMoveTest.php | +123 | −0 | Go to diff View file |
| A | plugins/tracker/tests/unit/Tracker/REST/v1/Move/BeforeMoveCheckerTest.php | +190 | −0 | Go to diff View file |
| A | plugins/tracker/tests/unit/Tracker/REST/v1/Move/DryRunMoveTest.php | +124 | −0 | Go to diff View file |
| A | plugins/tracker/tests/unit/Tracker/REST/v1/Move/MovePatchActionTest.php | +261 | −0 | Go to diff View file |
| M | tests/lib/Builders/ProjectTestBuilder.php | +6 | −0 | Go to diff View file |