Revision: 8608
Author: [email protected]
Date: Fri Aug 20 10:37:07 2010
Log: Ignoring __gwt_ObjectId in AbstractRequest#processRelated. The __gwt_ObjectId attribute is added by Chrome dev mode to all objects passed between JSNI and Java.

Review at http://gwt-code-reviews.appspot.com/760803

http://code.google.com/p/google-web-toolkit/source/detail?r=8608

Modified:
/trunk/user/src/com/google/gwt/requestfactory/client/impl/AbstractRequest.java

=======================================
--- /trunk/user/src/com/google/gwt/requestfactory/client/impl/AbstractRequest.java Thu Aug 19 13:24:10 2010 +++ /trunk/user/src/com/google/gwt/requestfactory/client/impl/AbstractRequest.java Fri Aug 20 10:37:07 2010
@@ -79,10 +79,10 @@
   /**
    * @deprecated use {...@link #with(String...)} instead.
    * @param properties
-   * @return
    */
+  @Deprecated
   public R forProperties(Collection<Property<?>> properties) {
-    for (Property p : properties) {
+    for (Property<?> p : properties) {
       with(p.getName());
     }
     return getThis();
@@ -134,6 +134,8 @@

   protected native void processRelated(JavaScriptObject related) /*-{
     for(var recordKey in related) {
+      // Workaround for __gwt_ObjectId appearing in Chrome dev mode.
+      if (!related.hasOwnProperty(recordKey)) continue;
       var schemaAndId = recordKey.split(/-/, 2);
       var jso = related[recordKey];
[email protected]::setSchemaAndRecord(Ljava/lang/String;Lcom/google/gwt/requestfactory/client/impl/RecordJsoImpl;)(schemaAndId[0], jso);

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

Reply via email to