Catch case where too many capture groups defined

Diabol · May 18, 2015 · 09575f6 · 09575f6
1 parent 9713b00
commit 09575f6
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/main/java/se/diabol/jenkins/pipeline/DeliveryPipelineView.java b/src/main/java/se/diabol/jenkins/pipeline/DeliveryPipelineView.java
@@ -517,8 +517,10 @@ public FormValidation doCheckRegexp(@QueryParameter String value) {
                         Pattern pattern = Pattern.compile(value);
                         if (pattern.matcher("").groupCount() == 1) {
                             return FormValidation.ok();
-                        } else {
+                        } else if (pattern.matcher("").groupCount() == 0) {
                             return FormValidation.error("No capture group defined");
+                        } else {
+                            return FormValidation.error("Too many capture groups defined");
                         }
                     } catch (PatternSyntaxException e) {
                         return FormValidation.error(e, "Syntax error in regular-expression pattern");