Ad some checkstyle fixes

pbalint-ge · pbalint-ge · commit 430fb93eeb2b · 2019-12-18T08:33:15.000+01:00
diff --git a/src/main/java/com/cloudbees/jenkins/GitHubPushTrigger.java b/src/main/java/com/cloudbees/jenkins/GitHubPushTrigger.java
@@ -119,7 +119,10 @@ public void onPost(final GitHubTriggerEvent event) {
                 }
             }
 
-            String lowercaseTriggeredByUser = event.getTriggeredByUser() != null? event.getTriggeredByUser().toLowerCase(): null;
+            String lowercaseTriggeredByUser = null;
+            if (event.getTriggeredByUser() != null) {
+                lowercaseTriggeredByUser = event.getTriggeredByUser().toLowerCase();
+            }
             if (lowercaseExcludedUsers != null && lowercaseExcludedUsers.contains(lowercaseTriggeredByUser)) {
                 return; // user is excluded from triggering build
             }

Original file line number	Diff line number	Diff line change
`@@ -119,7 +119,10 @@ public void onPost(final GitHubTriggerEvent event) {`
`119`	`119`	`}`
`120`	`120`	`}`
`121`	`121`
`122`		`- String lowercaseTriggeredByUser = event.getTriggeredByUser() != null? event.getTriggeredByUser().toLowerCase(): null;`
	`122`	`+ String lowercaseTriggeredByUser = null;`
	`123`	`+ if (event.getTriggeredByUser() != null) {`
	`124`	`+ lowercaseTriggeredByUser = event.getTriggeredByUser().toLowerCase();`
	`125`	`+ }`
`123`	`126`	`if (lowercaseExcludedUsers != null && lowercaseExcludedUsers.contains(lowercaseTriggeredByUser)) {`
`124`	`127`	`return; // user is excluded from triggering build`
`125`	`128`	`}`