Entity#persisted? should not go by id presence

Question

Closed this issue 11 years ago · 2 comments

There are use cases for assigning an id to an unpersisted entity.

Currently, Minimapper entities have a persisted? that just checks for an id, which will lie if you assign an id to an unpersisted entity.

Instead, it could be the mapper's responsibility to do something like entity.persisted = true whenever you pull from the DB, save successfully etc.

Answer 1 · 2013-07-02T16:07:41.000Z

See commit. How about it? Can make a pull request if you'd like.

Answer 2 · 2013-07-02T16:18:27.000Z

This commit is relevant also: barsoom@cb4b63d