stable

Clone or download

Read-only

refacto: stop repeating ourself when we retrieve field that user can

read Part of story #39685 Order artifact according to a field No functional change expected Change-Id: Iffe298becb300407cab51eea1a13c52b6be6056b

Modified Files

Name
A plugins/crosstracker/include/CrossTracker/Field/ReadableFieldRetriever.php +57 −0 Go to diff View file
M plugins/crosstracker/include/CrossTracker/REST/v1/CrossTrackerReportsResource.php +8 −6 Go to diff View file
M plugins/crosstracker/include/CrossTracker/Report/Query/Advanced/OrderByBuilder/Field/FieldFromOrderBuilder.php +3 −16 Go to diff View file
M plugins/crosstracker/include/CrossTracker/Report/Query/Advanced/QueryValidation/DuckTypedField/DuckTypedFieldChecker.php +4 −26 Go to diff View file
M plugins/crosstracker/include/CrossTracker/Report/Query/Advanced/ResultBuilder/Field/FieldResultBuilder.php +4 −16 Go to diff View file
M plugins/crosstracker/include/crosstrackerPlugin.php +10 −7 Go to diff View file
M plugins/crosstracker/tests/integration/CrossTracker/Tests/Report/ArtifactReportFactoryInstantiator.php +11 −8 Go to diff View file
A plugins/crosstracker/tests/unit/CrossTracker/Field/ReadableFieldRetrieverTest.php +83 −0 Go to diff View file
M plugins/crosstracker/tests/unit/CrossTracker/Report/Query/Advanced/InvalidOrderByBuilderTest.php +5 −1 Go to diff View file
M plugins/crosstracker/tests/unit/CrossTracker/Report/Query/Advanced/InvalidSelectablesCollectionBuilderTest.php +7 −2 Go to diff View file
M plugins/crosstracker/tests/unit/CrossTracker/Report/Query/Advanced/InvalidTermCollectorVisitorTest.php +5 −1 Go to diff View file
M plugins/crosstracker/tests/unit/CrossTracker/Report/Query/Advanced/QueryValidation/DuckTypedField/DuckTypedFieldCheckerTest.php +5 −1 Go to diff View file
M plugins/crosstracker/tests/unit/CrossTracker/Report/Query/Advanced/ResultBuilder/Field/FieldResultBuilderTest.php +9 −5 Go to diff View file