• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Component/s: analysis-model
• Labels:

  • help-wanted
  • newbie-friendly
• Environment:

  Jenkins 2.89.2 on Windows
  Warnings Plug-in 4.64

[JENKINS-48720] Warnings Plugin wants "Error" but JCReport apparently writes "error"

Collapse comment: Tobias Redl added a comment - 2019-03-29 22:29

Tobias Redl added a comment - 2019-03-29 22:29

drulli maybe a good way to go would be to use 

issueLevel.toLowerCase()...

then, isn't it?

Or is there a special reason to check just for the capitalized words?

Expand comment: Tobias Redl added a comment - 2019-03-29 22:29

Tobias Redl added a comment - 2019-03-29 22:29 drulli  maybe a good way to go would be to use  issueLevel.toLowerCase()... then, isn't it? Or is there a special reason to check just for the capitalized words?

Collapse comment: Ulli Hafner added a comment - 2019-03-31 10:13

Ulli Hafner added a comment - 2019-03-31 10:13

I think it would be even simpler to use ´Severity.guessFromString´ if possible. That will change the severity of some issues to ˋERRORˋ but that is ok.

Expand comment: Ulli Hafner added a comment - 2019-03-31 10:13

Ulli Hafner added a comment - 2019-03-31 10:13 I think it would be even simpler to use ´Severity.guessFromString´ if possible. That will change the severity of some issues to ˋERRORˋ but that is ok.

Collapse comment: Tobias Redl added a comment - 2019-04-01 12:58

Tobias Redl added a comment - 2019-04-01 12:58

Oh, I just saw, that this was a problem in the old warnings-plugin. As far as I can see this should not be a problem in warnings-ng.

Expand comment: Tobias Redl added a comment - 2019-04-01 12:58

Tobias Redl added a comment - 2019-04-01 12:58 Oh, I just saw, that this was a problem in the old warnings-plugin. As far as I can see this should not be a problem in warnings-ng.

Collapse comment: Ulli Hafner added a comment - 2019-04-01 20:39

Ulli Hafner added a comment - 2019-04-01 20:39

You are right, this already has been fixed quite some time ago: https://github.com/jenkinsci/analysis-model/blob/master/src/main/java/edu/hm/hafner/analysis/parser/jcreport/JcReportParser.java#L39.

Expand comment: Ulli Hafner added a comment - 2019-04-01 20:39

Ulli Hafner added a comment - 2019-04-01 20:39 You are right, this already has been fixed quite some time ago: https://github.com/jenkinsci/analysis-model/blob/master/src/main/java/edu/hm/hafner/analysis/parser/jcreport/JcReportParser.java#L39 .