[Rails-core] Please review my Pull Request

2013-04-30 Thread James Pinto
Hi everyone, I really liked the new rails 4 generated scaffold, I was working on making my code more readable for a project and I came up with this solution. Please review it, I'd like to know your opinion. https://github.com/rails/rails/pull/10392 -- You received this message because you ar

[Rails-core] Re: Adding #with to ActiveRecord::Relation

2013-06-08 Thread James Pinto
could you gist that code for me please, I'd like to take a look at it. On Thursday, May 9, 2013 5:52:42 PM UTC-4, Joshua Wood wrote: > > Postgres has it: > http://www.postgresql.org/docs/9.2/static/queries-with.html > > Arel has it: > http://www.ruby-doc.org/gems/docs/a/arel-3.0.2/Arel/SelectMan

[Rails-core] Re: false.present? is false. Is it desired behaviour?

2013-06-25 Thread James Pinto
I understand where you're coming from, and I completely disagree false is blank, but it's not nil, however, in some cases, falseness should not be validated "validates_presence_of :aggrees_with_contract"* is correct, requires the user to check the contract* "validates_presence_of :displays_photo