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