geoext/geoext2 — pull request #151 by adube (2012-05-18 13:07:50+00:00)
Title: fix data.feature.Reader - record internalId set instead of 'id' data

If I understand correctly, in 1.1 that line created a new record instance using the feature.id if it was set, or 'undefined' if state was INSERT.

In 2.0, the convertedValues is the record data property, so setting 'id' here doesn't seem okay. Plus, in my case, I have an 'id' field and its value is always overriden by the feature.id value.

I'm not sure if setting the record.internalId is the right way to go, so please review.