Jason Gignac's profile - rails

Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

Jason Gignac's latest activity

Thursday, June 24 2010

3774 Gem generators are not found by script/generate was updated in Ruby on Rails

Jason Gignac commented at 5:29 PM
TMorgan - I guess, I would argue that generation isn't something that one thinks of as being specific to a particular environment - I want rails to make some ne...

3251 Dirty attributes use inconsistent accessors for before/after was updated in Ruby on Rails

Jason Gignac commented at 5:18 PM
As far as I'm concerned, this is expected behaviour? When I access the current value, by calling, say 'this_field', there is an implicit understanding that if I...

4955 reduce usage of Pathname to make things faster was updated in Ruby on Rails

Jason Gignac commented at 4:51 PM
I am only curious, but why is pathname slower than the alternative implementation that is in the original commit and that you suggest in your diff here?

4951 Rails Console Executes Where Queries without lazy loading was updated in Ruby on Rails

Jason Gignac commented at 4:48 PM
-1 I would submit there are two conflicting desires here: on the one hand, the behaviour of inspect is what you would expect in this situation - if in any other...

Monday, May 31 2010

4739 core_ext/numeric/bytes don't respect IEEE 1541 standard was updated in Ruby on Rails

Jason Gignac commented at 9:45 PM
-1 I would argue that while this may be an official standard, the de facto standard is 1024? Most programmers I'd ask, if you said 'how many are in a kilobyte' ...

4725 [PATCH] Allow ActiveModel attribute names with spaces and other characters was updated in Ruby on Rails

Jason Gignac commented at 9:42 PM
-1 Though I can see the argument for this, I think it would end up adding confusion for what is a somewhat slender use case - the number of situations in which ...

4740 Improve Rails README was updated in Ruby on Rails

Jason Gignac commented at 9:35 PM

A few comments:

Rather than:
+By default, Rails will try to use Mongrel if it's installed when started with +rails server, otherwise Rails will use WEBrick, the...

Saturday, May 08 2010

~~511~~ ~~link_to_remote_if and link_to_remote_unless~~ was updated in Ruby on Rails

Jason Gignac commented at 1:44 PM
-1 Maybe I'm wrong, but wouldn't it be just as easy to just use link_to_remote(params) if(condition)? This form seems to make the syntax a little needlessly com...

3143 url_for should remember params[:format] was updated in Ruby on Rails

Jason Gignac commented at 1:37 PM
-1 Honestly, to me it makes more sense NOT to make the behaviour of url_for specific to the context in which it's executed. But, YMMV.

Sunday, March 15 2009

2228 ActiveRecord::Base#saved? was updated in Ruby on Rails

Jason Gignac commented at 5:43 PM

I second Mr. Atzen -

saved? could mean "Did my last saved work how I expected it to?" Whereas what you're actually asking is "Is this object already in the data...