From 09575f6d84bb3f9f771e54f84efcce0f4f7e4724 Mon Sep 17 00:00:00 2001 From: "C.J. Jameson" Date: Mon, 18 May 2015 14:41:11 -0700 Subject: [PATCH] Catch case where too many capture groups defined --- .../java/se/diabol/jenkins/pipeline/DeliveryPipelineView.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/main/java/se/diabol/jenkins/pipeline/DeliveryPipelineView.java b/src/main/java/se/diabol/jenkins/pipeline/DeliveryPipelineView.java index cefd20fa4..cad595a47 100644 --- 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");