Joakim Repomaa
Joakim Repomaa
this is "by design". It's a bit confusing though. the query in `Repo.get` is only used for preloads. If you want to use `where`s you will need to use `Repo.get_by`....
Returning nothing or only the primary key value could be a real performance boost.
Same applies of course to other adapters that use SELECT with LAST_INSERTED_ID()
well, `last_insert_id` is undefined up there but basically it would be the same as leaving the guard clause out. :/ In this case the method just freezes the program.
To recap: This PR is on hold, because the specs for the query methods introduced to `LiveTransaction` rely on having the the inserted record (with it's id set) returned. This...
I now use compiler flags to determine which db the specs are run against and depending on those enable/disable some specs. Nested transactions aren't supported by sqlite so i exclude...
What's a bit of a mystery to me is the currently failing spec for delete_all in a transaction for sqlite. I can't reproduce this on my local machine (arch linux...
Anything standing in the way of a merge?
merge #52 first
I'm using crecto for my latest project at work so i might be able to sneek in some features/fixes from time to time.