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.

Simon Jefford

Simon Jefford

Developer · Not a member

Simon Jefford's latest activity

Sunday, May 16 2010
4602 Make 's' reserved word in 'rails ...' command was updated in Ruby on Rails
  • Simon Jefford
    Simon Jefford commented at 5:46 PM

    Added a test to the patch - authorship has been retained.

    +1 verified
4575 Erubis template parsing error on invalid match for variable name containing "do" was updated in Ruby on Rails
  • Simon Jefford
    Simon Jefford commented at 2:29 AM

    Patch with test
  • Simon Jefford
    Simon Jefford commented at 1:31 AM

    I've attached a patch

    It doesn't have any tests at the moment - hoping to enlist some help on that front...
Saturday, May 15 2010
3074 Remove sqlite 2 from CI setup instructions was updated in Ruby on Rails
  • Simon Jefford
    • Tag changed from bugmash, master, patch to bugmash, bugmash-review, master, patch
    by Simon Jefford at 11:56 PM
  • Simon Jefford
    Simon Jefford commented at 11:55 PM

    +1 agreed - this ticket should be closed.
1367 [PATCH] Add a NullStore to ActiveSupport::Cache was updated in Ruby on Rails
  • Simon Jefford
    Simon Jefford commented at 11:45 PM

    Updated patch - more test coverage + remove unnecessary override
  • Simon Jefford
    • Tag changed from activesupport, bugmash, cache, patch to activesupport, bugmash, bugmash-review, cache, patch
    by Simon Jefford at 10:22 PM

    I've rebased the above patch, cleaned up some whitespace issues and updated it slightly so overrides the correct methods (they seem to have changed slightly fro...
2919 Activerecord::Base#clone does not set dirty bits was updated in Ruby on Rails
Thursday, August 13 2009
3044 Change scaffolding to use respond_with was updated in Ruby on Rails
  • Simon Jefford
    Simon Jefford commented at 10:50 AM

    Here's a patch without inline assignment but with flash messages!
Wednesday, August 12 2009
3044 Change scaffolding to use respond_with was updated in Ruby on Rails
  • Simon Jefford
    Simon Jefford commented at 10:54 PM

    José,

    Yes, you're absolutely right of course. Guess I got carried away with deleting code and missed that I was taking the flash messages out.

    I'll get an updat...