Skip to content

DataMapper adapter improvements #355

merged 3 commits into from May 2, 2011

2 participants


I'm sure there's more to be done, but this gets the pending spec passing, at least. Also eliminates the round-trip to the database to evaluate conditions on a single resource (DataMapperAdapter.matches_conditions_hash?).

At the moment I don't see how to associate this pull request with ryanb/cancan#245 but it is meant as a fix for that issue.

ryanb commented Apr 29, 2011

Awesome thanks! I'll get this pulled in soon.

@ryanb ryanb merged commit ff13a82 into ryanb:master May 2, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.