Reviewers: rchandia,

Description:
Let the Key for Maps be of type Object.

[JSR 303 TCK Result] 110 of 257 (42.80%) Pass with 20 Failures and 9
Errors.


Please review this at http://gwt-code-reviews.appspot.com/1376803/

Affected files:
  M user/src/com/google/gwt/validation/client/impl/GwtValidationContext.java
M user/src/com/google/gwt/validation/rebind/GwtSpecificValidatorCreator.java M user/test/org/hibernate/jsr303/tck/tests/validation/PropertyPathGwtTest.java


Index: user/src/com/google/gwt/validation/client/impl/GwtValidationContext.java
===================================================================
--- user/src/com/google/gwt/validation/client/impl/GwtValidationContext.java (revision 9825) +++ user/src/com/google/gwt/validation/client/impl/GwtValidationContext.java (working copy)
@@ -102,10 +102,9 @@
   /**
    * Append a keyed node to the path.
    *
-   * @param name
    * @return the new GwtValidationContext.
    */
-  public GwtValidationContext<T> appendKey(String name, String key) {
+  public GwtValidationContext<T> appendKey(String name, Object key) {
     GwtValidationContext<T> temp = new GwtValidationContext<T>(rootBean,
         beanDescriptor, messageInterpolator, validator, validatedObjects);
     temp.path = path.appendKey(name, key);
Index: user/src/com/google/gwt/validation/rebind/GwtSpecificValidatorCreator.java
===================================================================
--- user/src/com/google/gwt/validation/rebind/GwtSpecificValidatorCreator.java (revision 9825) +++ user/src/com/google/gwt/validation/rebind/GwtSpecificValidatorCreator.java (working copy)
@@ -1182,10 +1182,10 @@
     sw.indent();
     sw.indent();

-    // context.appendKey("myProperty",String.valueOf(entry.getKey())),
+    // context.appendKey("myProperty",entry.getKey()),
     sw.print("context.appendKey(\"");
     sw.print(p.getPropertyName());
-    sw.println("\",String.valueOf(entry.getKey())),");
+    sw.println("\",entry.getKey()),");

     // entry.getValue(), groups));
     sw.println("entry.getValue(), groups));");
Index: user/test/org/hibernate/jsr303/tck/tests/validation/PropertyPathGwtTest.java
===================================================================
--- user/test/org/hibernate/jsr303/tck/tests/validation/PropertyPathGwtTest.java (revision 9825) +++ user/test/org/hibernate/jsr303/tck/tests/validation/PropertyPathGwtTest.java (working copy)
@@ -48,7 +48,6 @@
     delegate.testPropertyPathWithList();
   }

-  @Failing(issue = 5803)
   public void testPropertyPathWithMap() {
     delegate.testPropertyPathWithMap();
   }


--
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to