change error to warn on reregister btn

MerkushevKirill · MerkushevKirill · commit a88fc10ee77e · 2015-07-11T15:10:26.000+03:00
diff --git a/src/main/java/com/cloudbees/jenkins/GitHubPushTrigger.java b/src/main/java/com/cloudbees/jenkins/GitHubPushTrigger.java
@@ -325,9 +325,10 @@ public FormValidation doCheckHookUrl(@QueryParameter String value) {
 
         }
 
+        @SuppressWarnings("unused")
         public FormValidation doReRegister() {
             if (!manageHook) {
-                return FormValidation.error("Works only when Jenkins manages hooks");
+                return FormValidation.warning("Works only when Jenkins manages hooks");
             }
 
             List<GitHubPushTrigger> registered = from(getJenkinsInstance().getAllItems(AbstractProject.class))
diff --git a/src/main/java/org/jenkinsci/plugins/github/util/JobInfoHelpers.java b/src/main/java/org/jenkinsci/plugins/github/util/JobInfoHelpers.java
@@ -63,7 +63,6 @@ public Collection<GitHubRepositoryName> apply(AbstractProject job) {
         };
     }
 
-
     /**
      * If any of event subscriber interested in hook for job, then return true
      * By default, push hook subscriber is interested in job with gh-push-trigger
@@ -78,5 +77,5 @@ public boolean apply(AbstractProject job) {
             }
         };
     }
-
 }
+

Original file line number	Diff line number	Diff line change
`@@ -325,9 +325,10 @@ public FormValidation doCheckHookUrl(@QueryParameter String value) {`
`325`	`325`
`326`	`326`	`}`
`327`	`327`
	`328`	`+ @SuppressWarnings("unused")`
`328`	`329`	`public FormValidation doReRegister() {`
`329`	`330`	`if (!manageHook) {`
`330`		`- return FormValidation.error("Works only when Jenkins manages hooks");`
	`331`	`+ return FormValidation.warning("Works only when Jenkins manages hooks");`
`331`	`332`	`}`
`332`	`333`
`333`	`334`	`List<GitHubPushTrigger> registered = from(getJenkinsInstance().getAllItems(AbstractProject.class))`
Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,6 @@ public Collection<GitHubRepositoryName> apply(AbstractProject job) {`
`63`	`63`	`};`
`64`	`64`	`}`
`65`	`65`
`66`		`-`
`67`	`66`	`/**`
`68`	`67`	`* If any of event subscriber interested in hook for job, then return true`
`69`	`68`	`* By default, push hook subscriber is interested in job with gh-push-trigger`
`@@ -78,5 +77,5 @@ public boolean apply(AbstractProject job) {`
`78`	`77`	`}`
`79`	`78`	`};`
`80`	`79`	`}`
`81`		`-`
`82`	`80`	`}`
	`81`	`+`