Updated based on jat's comments.

http://gwt-code-reviews.appspot.com/760803/diff/1/2
File
user/src/com/google/gwt/requestfactory/client/impl/AbstractRequest.java
(right):

http://gwt-code-reviews.appspot.com/760803/diff/1/2#newcode139
user/src/com/google/gwt/requestfactory/client/impl/AbstractRequest.java:139:
if (schemaAndId == '__gwt_ObjectId') continue;
On 2010/08/20 19:58:52, jat wrote:
Instead you should add at the top of the loop:

if (!related.hasOwnProperty(recordKey)) continue;

We patch in a check for __gwt_ObjectId on Chrome in DevMode, and it is
recommended practice to do this anyway for any JS value used as a map
to avoid
picking up spurious keys.

See our Dictionary.addKeys for an example.

Done.

http://gwt-code-reviews.appspot.com/760803/show

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

Reply via email to