Merge pull request #10 from rayarub/master

[JENKINS-17807] Fix NPE caused by configuration being changed while a build is happening and current build not having a result yet.
jenkinsci · Mar 19, 2014 · 8ce67f5 · 8ce67f5
2 parents 7473c4c + 742e201
commit 8ce67f5
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/src/main/java/hockeyapp/HockeyappRecorder.java b/src/main/java/hockeyapp/HockeyappRecorder.java
@@ -261,12 +261,16 @@ public Collection<? extends Action> getProjectActions(
 			AbstractProject<?, ?> project) {
 		ArrayList<HockeyappBuildAction> actions = new ArrayList<HockeyappBuildAction>();
 		RunList<? extends AbstractBuild<?, ?>> builds = project.getBuilds();
-
+		
 		@SuppressWarnings("unchecked")
 		Collection<AbstractBuild<?, ?>> predicated = CollectionUtils.select(builds, new Predicate() {
 			public boolean evaluate(Object o) {
-				return ((AbstractBuild<?, ?>) o).getResult().isBetterOrEqualTo(
-						Result.SUCCESS);
+				if(o == null || ((AbstractBuild<?, ?>) o).getResult() == null) {
+					return false;
+				} else {
+					return ((AbstractBuild<?, ?>) o).getResult().isBetterOrEqualTo(
+							Result.SUCCESS);
+				}
 			}
 		});