You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by dr...@apache.org on 2010/10/06 17:52:22 UTC

svn commit: r1005092 - in /tapestry/tapestry5/trunk/tapestry-beanvalidator/src: main/resources/org/apache/tapestry5/beanvalidator/ test/java/org/apache/tapestry5/beanvalidator/integration/

Author: drobiazko
Date: Wed Oct  6 15:52:22 2010
New Revision: 1005092

URL: http://svn.apache.org/viewvc?rev=1005092&view=rev
Log:
TAP5-1297: Client-side validation of @Pattern is broken

Modified:
    tapestry/tapestry5/trunk/tapestry-beanvalidator/src/main/resources/org/apache/tapestry5/beanvalidator/tapestry-beanvalidator.js
    tapestry/tapestry5/trunk/tapestry-beanvalidator/src/test/java/org/apache/tapestry5/beanvalidator/integration/TapestryBeanValidationIntegrationTests.java

Modified: tapestry/tapestry5/trunk/tapestry-beanvalidator/src/main/resources/org/apache/tapestry5/beanvalidator/tapestry-beanvalidator.js
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-beanvalidator/src/main/resources/org/apache/tapestry5/beanvalidator/tapestry-beanvalidator.js?rev=1005092&r1=1005091&r2=1005092&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-beanvalidator/src/main/resources/org/apache/tapestry5/beanvalidator/tapestry-beanvalidator.js (original)
+++ tapestry/tapestry5/trunk/tapestry-beanvalidator/src/main/resources/org/apache/tapestry5/beanvalidator/tapestry-beanvalidator.js Wed Oct  6 15:52:22 2010
@@ -60,5 +60,5 @@ Tapestry.Validator.size = function(field
 
 Tapestry.Validator.pattern = function(field, message, spec)
 {
-	Tapestry.Validator.regexp(field, message, spec.pattern);
+	Tapestry.Validator.regexp(field, message, spec.regexp);
 };
\ No newline at end of file

Modified: tapestry/tapestry5/trunk/tapestry-beanvalidator/src/test/java/org/apache/tapestry5/beanvalidator/integration/TapestryBeanValidationIntegrationTests.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-beanvalidator/src/test/java/org/apache/tapestry5/beanvalidator/integration/TapestryBeanValidationIntegrationTests.java?rev=1005092&r1=1005091&r2=1005092&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-beanvalidator/src/test/java/org/apache/tapestry5/beanvalidator/integration/TapestryBeanValidationIntegrationTests.java (original)
+++ tapestry/tapestry5/trunk/tapestry-beanvalidator/src/test/java/org/apache/tapestry5/beanvalidator/integration/TapestryBeanValidationIntegrationTests.java Wed Oct  6 15:52:22 2010
@@ -173,6 +173,11 @@ public class TapestryBeanValidationInteg
     	click(SUBMIT);
     	
     	assertBubbleMessage("loginName", "Login Name must match \"[a-zA-Z]+\"");
+    	
+    	type("loginName", "abc");
+    	click(SUBMIT);
+    	
+    	assertBubbleMessage("loginName", "You must provide at least 5 characters for Login Name.");
     }
     
     protected final void assertBubbleMessage(String fieldId, String expected)