Preferred way to transform a query result back to an entity

[sebelga]

Preferred way to transform a query result back to an entity

[jfbenckhuijsen]

Ok, been experimenting a bit. So far I've come up with this:

return entities.map((entity) => { delete entity.id; return new MyModelClass(entity, entity[gstore.KEY]); });

The delete entity.id seems needed as the query result includes this value, whereas the schema doesn't define this as a valid property. I am seeing some weird behaviour though, where saving the returned entity seems to result in new entities being added, instead of the existing one being updated.

[sebelga]

That might be something nice to add as an option parameter to query.
Until then you guessed right that you need to delete the id first. Then, try something like this in your map (untested):

return entities.map((entity) => {
    delete entity.id;
    return MyModelClass.__model(entity, null, null, null, entity[gstore.KEY]);
});

Tell me if it worked.

[jfbenckhuijsen]

Doesn't seem to give the desired effect. When debugging, in model.js:717, var entity is constructed. In the first case (i.e. after construction of the initial object), this.entityKey contains four fields:

• kind: User
• namespace: undefined
• path: undefined
• name : "[some uuid generated string]"

After executing a query, I call save again on the object constructed in the way described above. When I reenter model.js:717, this.entityKey does not have the name property.

[jfbenckhuijsen]

Been debugging this on the fork I created, have the feeling the fromDatastore() function in datastore.js somehow does not copy the key.

[sebelga]

Sorry I made a mistake. Inside the map function, its gstore.ds.KEY to access the Symbol for the entityKey. So try with:

return entities.map((entity) => {
    delete entity.id;
    return MyModelClass.__model(entity, null, null, null, entity[gstore.ds.KEY]);
});

It should work.

[sebelga]

Did you have a chance to try the solution?

[jfbenckhuijsen]

Yes. It worked. After I finished with the virtual properties pull request, I'll craft up a feature to have this in the library itself.

Verstuurd vanaf mijn iPhone

Op 8 nov. 2016 om 18:20 heeft Sébastien Loix [email protected] het volgende geschreven:

Did you have a chance to try the solution?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.