Branch: refs/heads/master
  Home:   https://github.com/jenkinsci/performance-plugin
  Commit: b9a709fc1ebe1bfddfb1a8f2501500186b687f49
      
https://github.com/jenkinsci/performance-plugin/commit/b9a709fc1ebe1bfddfb1a8f2501500186b687f49
  Author: hfufu <[email protected]>
  Date:   2016-04-15 (Fri, 15 Apr 2016)

  Changed paths:
    M src/main/java/hudson/plugins/performance/PerformanceReport.java

  Log Message:
  -----------
  Compatible with flexible publisher 

The original parser lookup in ifSummarizerParsedUsed is not comptaible with 
flexible publisher, because the performance publisher is not directly in 
buildAction.getBuild().getProject().getPublishersList(), but wrapped inside 
publisher list inside flexible publisher. It will result in NPE because 
buildAction.getBuild().getProject().getPublishersList().get(PerformancePublisher.class)
 return null.

We can just simply look up with buildAction.getParserByDisplayName directly 
instead of looping the project publisher list to check for JmeterSummarizer and 
Iago parser. The parser list is already there in buildAction. This way the code 
is simpler, and we can avoid complicated lookup when this publisher is under 
flexible publisher.


  Commit: d47e404ae0b74f89a38636d21ff57bbd97a64bf9
      
https://github.com/jenkinsci/performance-plugin/commit/d47e404ae0b74f89a38636d21ff57bbd97a64bf9
  Author: Andrey Pokhilko <[email protected]>
  Date:   2016-04-15 (Fri, 15 Apr 2016)

  Changed paths:
    M src/main/java/hudson/plugins/performance/PerformanceReport.java

  Log Message:
  -----------
  Merge pull request #59 from hfufu/master

Compatible with flexible publisher


Compare: 
https://github.com/jenkinsci/performance-plugin/compare/75327e7680d3...d47e404ae0b7

-- 
You received this message because you are subscribed to the Google Groups 
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to