Skip to content

Commit

Permalink
Merge pull request #722 from hazendaz/master
Browse files Browse the repository at this point in the history
[cleanup] Cleanup location of properties or remove unused
  • Loading branch information
hazendaz authored Dec 25, 2023
2 parents 0e6f4d6 + 4e99998 commit b5b0fd1
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -57,8 +57,6 @@ trait SpotBugsPluginsTrait {
String getSpotbugsPlugins() {
ResourceHelper resourceHelper = new ResourceHelper(log, spotbugsXmlOutputDirectory, resourceManager)

URL[] pluginURL

String urlPlugins = ""

if (pluginList) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -438,15 +438,13 @@ class SpotbugsReportGenerator implements SpotBugsInfo {
*
*/
protected String assembleJxrHyperlink(GPathResult line) {
String hyperlink
String prefix

log.debug("Inside assembleJxrHyperlink")
log.debug("line is " + line.text())
log.debug("outputDirectory is " + outputDirectory.getAbsolutePath())
log.debug("xrefLocation is " + xrefLocation.getAbsolutePath())
log.debug("xrefTestLocation is " + xrefTestLocation.getAbsolutePath())

String prefix
compileSourceRoots.each { compileSourceRoot ->
if (!new File(compileSourceRoot + File.separator + line.@sourcepath.text()).exists()) {
return
Expand All @@ -468,6 +466,7 @@ class SpotbugsReportGenerator implements SpotBugsInfo {
String path = prefix + line.@classname.text().replaceAll("[.]", "/").replaceAll("[\$].*", "")
String lineNumber = valueForLine(line)

String hyperlink
if (lineNumber != bundle.getString(NOLINE_KEY)) {
hyperlink = "<a href=\"" + path + ".html#L" + line.@start.text() + "\">" + lineNumber + "</a>"
} else {
Expand Down

0 comments on commit b5b0fd1

Please sign in to comment.