Skip to content

Commit a24c411

Browse files
authored
Merge pull request #295 from jglick/getAllParameters
ParametersAction.getAllParameters can now be called directly
2 parents 49c2908 + cfa1c52 commit a24c411

File tree

1 file changed

+1
-8
lines changed

1 file changed

+1
-8
lines changed

src/main/java/org/jenkinsci/plugins/workflow/cps/ParamsVariable.java

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@
3333
import java.io.Serializable;
3434
import java.util.Collections;
3535
import java.util.HashMap;
36-
import java.util.List;
3736
import java.util.Map;
3837
import org.jenkinsci.plugins.workflow.pickles.PickleFactory;
3938

@@ -56,13 +55,7 @@
5655
Map<String,Object> values = new HashMap<>();
5756
ParametersAction action = b.getAction(ParametersAction.class);
5857
if (action != null) {
59-
List<ParameterValue> parameterValues;
60-
try { // TODO 1.651.2+ remove reflection
61-
parameterValues = (List<ParameterValue>) ParametersAction.class.getMethod("getAllParameters").invoke(action);
62-
} catch (NoSuchMethodException x) {
63-
parameterValues = action.getParameters();
64-
}
65-
for (ParameterValue parameterValue : parameterValues) {
58+
for (ParameterValue parameterValue : action.getAllParameters()) {
6659
addValue(values, parameterValue);
6760
}
6861
}

0 commit comments

Comments
 (0)