search

sxross / MotionModel

Simple Model and Validation Mixins for RubyMotion
MIT License
192 stars 67 forks source link

Adds #array method as alias of #all #138

Closed pixlwave closed 9 years ago

pixlwave commented 9 years ago

Just a small syntax addition that I got used to with CDQ, and makes more sense in my head. Clearly indicates that it's casting from a MM object type to an array, and reads better (to me) with queries e.g. Task.where(:importance).gt(2).all becomes Task.where(:importance).gt(2).array

sxross commented 9 years ago

Travis is failing on this change. I have no idea why. Pursuing it now.

sxross commented 9 years ago

Please catch up to master and your Travis build should pass. Then I can merge.

pixlwave commented 9 years ago

All done :grinning:

sxross commented 9 years ago

Thanks!

pixlwave commented 9 years ago

👍

pixlwave commented 9 years ago

Hold on, has this been closed and not merged? (can't tell properly as I'm on mobile).