[JIRA] (JENKINS-35352) unclassified field org.jenkinsci.plugins.workflow.support.steps.build.RunWrapper result when trying to call setResult on wrong type

[r.fuere...@xortex.com (JIRA)]

Title: Message Title

Reinhold Füreder commented on JENKINS-35352   Re: unclassified field org.jenkinsci.plugins.workflow.support.steps.build.RunWrapper result when trying to call setResult on wrong type Sergey Kovin and Jesse Glick: I think there are two "problems": The line currentBuild.result = hudson.model.Result.FAILURE actually fails, because currently (unfortunately) the result can only be set based on String. See org.jenkinsci.plugins.workflow.support.steps.build.RunWrapper#setResult(...): @Whitelisted public void setResult(String result) throws AbortException { if(!this.currentBuild) { throw new SecurityException("can only set the result property on the current build"); } else { this.build().setResult(Result.fromString(result)); } } => So you could use currentBuild.result = hudson.model.Result.FAILURE.toString(), I guess. => Jesse Glick would it be possible to also allow using hudson.model.Result (directly), which would be type/typo-safe? (For backwards compatibility of course the old String based approach needs to be kept, I assume.) After this change, however, I think the second problem should pop up: namely that hudson.model.Result.FAILURE is not whitelisted either. => Jesse Glick Would it be possible to whitelist all static result states from hudson.model.Result, e.g.: staticField hudson.model.Result FAILURE? Add Comment   This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c) -- You received this message because you are subscribed to the Google Groups "Jenkins Issues" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.