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.

Si

Si

Not a member

Si's latest activity

Friday, May 07 2010
3914 cache_sweeper yields blank output was updated in Ruby on Rails
  • Si
    Si commented at 8:24 AM

    Hi Kevmoo, I updated that patch but the fix is the same in Neeraj's diff. Can you try that?
Saturday, May 01 2010
3914 cache_sweeper yields blank output was updated in Ruby on Rails
  • Si
    Si commented at 8:36 AM

    Now I'm pushing up to staging I see that the before filter will need to return true in all cases. This is because the sweeper filter aspect is really implementa...
Thursday, April 29 2010
4460 Validates_acceptance_of is not respecting database field was updated in Ruby on Rails
  • Si
    Si commented at 1:37 PM

    I can verify that this patch resolves my issue. Thanks Santiago!
Wednesday, April 28 2010
4437 [Patch] XML Mini Parser NoMethodError :getc when application running through ... was updated in Ruby on Rails
  • Si
    Si commented at 1:15 PM

    Ha! I should have used preview :)

    Even something minimal like

    <oops/>

    will do it.
  • Si
    Si commented at 1:12 PM

    Even something minimal like will do it.

    If I use NokogiriSAX as my XmlMini backend, your patch works as hoped however.
Tuesday, April 27 2010
4437 [Patch] XML Mini Parser NoMethodError :getc when application running through ... was updated in Ruby on Rails
  • Si
    Si commented at 4:27 PM

    Hi Simon, I just tried the patch and get the same error:

    PhusionPassenger::Utils::RewindableInput is not a valid input stream.  It must walk like either a Stri...
4460 Validates_acceptance_of is not respecting database field was updated in Ruby on Rails
  • Si
    Si commented at 3:20 PM

    Hi James,

    I'm not a fan of the style employed by AcceptanceValidationTest. The tests are named according to attribute tested (e.g. test_eula), instead of explic...
  • Si
    Si commented at 10:58 AM

    Sure!

    I wasn't set up for tests in the activemodel area before, primarily due to a MySQL dependency. Here's a patch to use database backing for the terms_of_ser...
Thursday, April 22 2010
4460 Validates_acceptance_of is not respecting database field was created in Ruby on Rails
  • Si
    Si created the ticket at 10:27 PM

    In AcceptanceValidator#setup (/lib/active_model/validations/acceptance.rb), the class instance methods are inspected, to see if a writer method exists for each ...
Sunday, April 04 2010
3914 cache_sweeper yields blank output was updated in Ruby on Rails
  • Si
    • Tag changed from cache, cache_sweeper, sweeper to cache, cache_sweeper, patch, sweeper
    by Si at 10:52 PM