RFR: 257: Proper setting of custom fields in Jira issues

Erik Helin ehelin at openjdk.java.net
Thu Feb 6 09:53:32 UTC 2020


On Tue, 4 Feb 2020 12:05:33 GMT, Robin Westberg <rwestberg at openjdk.org> wrote:

> Hi all,
> 
> Please review this change that properly sets custom fields when creating Jira issues.
> 
> Best regards,
> Robin

Looks good! Just a minor nit: since this patch was created the `JSON` API has gotten the possibility to query `JSONValue` objects for their type using `is*` methods (e.g. `isObject()`). If you merge master into this PR then you will be able to use these new methods ��

issuetracker/src/main/java/org/openjdk/skara/issuetracker/jira/JiraProject.java line 204:

> 203:         }
> 204:         if (unfiltered instanceof JSONObject) {
> 205:             if (unfiltered.asObject().contains("id")) {

Suggestion:

        if (unfiltered.isObject()) {

-------------

Marked as reviewed by ehelin (Reviewer).

PR: https://git.openjdk.java.net/skara/pull/392


More information about the skara-dev mailing list