search

chicks / sugarcrm

A ruby based REST Client for SugarCRM
MIT License
90 stars 64 forks source link

Return `nil` (instead of false) if no results are found #12

Closed davidsulc closed 13 years ago

davidsulc commented 13 years ago

Makes more sense to me, and is more consistent with commit https://github.com/chicks/sugarcrm/commit/515068985267077b4058c5120b6e5fcc3dbdb28e

Should be backwards compatible for boolean tests: result = SugarCRM::Contact.find_by_first_name('blah') process(result) if result # should work the same as it did when false was returned

chicks commented 13 years ago

Merged pull request. Will cut another gem at the end of the day, as I expect more changes! :)

chicks commented 13 years ago

Just re-opening to cheat and see how to pull your branch :)