Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fixed ge/le mnesia db adaptor find sort order: they were reversed. #32

Merged
merged 1 commit into from Sep 3, 2012

Conversation

Projects
None yet
3 participants

LordIvan commented Sep 3, 2012

Hey there - The boss_db mnesia adaptor had a very small bug - the 'le' and 'ge' tests for find are reversed - 'le' returns 'ge'; and 'ge' returns 'le'

eg: the following would return the wrong thing when using the mnesia db adaptor:

boss_db:find(record_type, [{sortfield, ge, 12345}])

This is the first time I've committed/issued a pull request something via git hub; so apologies if I've done something wrong...

Contributor

evanmiller commented Sep 3, 2012

Thanks!

@evanmiller evanmiller added a commit that referenced this pull request Sep 3, 2012

@evanmiller evanmiller Merge pull request #32 from LordIvan/mnesia_adaptor_ge_sort_fix
Fixed ge/le mnesia db adaptor find sort order: they were reversed.
c1cd71d

@evanmiller evanmiller merged commit c1cd71d into ErlyORM:master Sep 3, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment