Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-14225

Bring back BugInstance Reviews (Classifications and Comments) from Cloud in Details

      Showing the Reviews (Classification and Comment by Users) from the configured FindBugs Cloud was obviously achieved by sending a request to the "bug details url" (http://any.findbugs.cloud/<issueHash>?embed). The code for getting this URL template from the Cloud definition is now commented out as the FindBugs API has changed:

      In FindBugsParser#setCloudInformation
      // FIXME: This method has been removed in findbugs 2.0.0
      // bug.setDetailsUrlTemplate(cloud.getBugDetailsUrlTemplate());
      

      As this is a major feature of connecting to a FindBugs Cloud in the first place, I think this should be fixed to work again by addressing the current FindBugs API.

      Possible solutions I can think of:

      1. try to get the "details url template" by calling some other function.
        • Map<String, String> edu.umd.cs.findbugs.BugCollection.getXmlCloudDetails() is a candidate.
      2. Extend the model adding Collection<BugDesignation> to Bug and populate with the most-recent Designation of each Reviewer by calling appropriate functions of Cloud.

          [JENKINS-14225] Bring back BugInstance Reviews (Classifications and Comments) from Cloud in Details

          Vasil Dilchev created issue -
          Vasil Dilchev made changes -
          Attachment New: findbugs-plugin.patch [ 22048 ]
          Vasil Dilchev made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          SCM/JIRA link daemon made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Progress [ 3 ] New: Resolved [ 5 ]
          Vasil Dilchev made changes -
          Resolution Original: Fixed [ 1 ]
          Status Original: Resolved [ 5 ] New: Reopened [ 4 ]
          Ulli Hafner made changes -
          Status Original: Reopened [ 4 ] New: In Progress [ 3 ]
          SCM/JIRA link daemon made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Progress [ 3 ] New: Resolved [ 5 ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 144857 ] New: JNJira + In-Review [ 191220 ]

            drulli Ulli Hafner
            erase_your_mind Vasil Dilchev
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: