Jim Wilson'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.

Jim Wilson's latest activity

Sunday, November 07 2010

5611 active_support/cache/file_store.rb is this correct? was updated in Ruby on Rails

Jim Wilson commented at 9:17 PM
I don't need any credit. Just take simpliest approach to fix it. However if you need me to do a patch I can... I am not a very active submitter...

Wednesday, September 15 2010

5634 reset_session in action_dispatch/http/request.rb failing in test mode was created in Ruby on Rails

Jim Wilson created the ticket at 5:38 PM
I get the below when running one of my tests that does a reset_session in a ActionController::TestCase
```
NoMethodError: undefined method `destroy' for #<Hash:0xc...
```

Tuesday, September 14 2010

5611 active_support/cache/file_store.rb is this correct? was updated in Ruby on Rails

Jim Wilson commented at 3:48 PM
Lighthouse is messing up the formatting. I am not real familiar with Lighthouse...
Here it is attached.
Jim Wilson commented at 3:43 PM

Added a test that fails without the patch and passes with it. git diff below

git diff -p
diff --git a/activesupport/lib/active_support/cache/file_store.rb b/act...

Sunday, September 12 2010

5611 active_support/cache/file_store.rb is this correct? was updated in Ruby on Rails

Jim Wilson commented at 3:07 PM
Ok, it is working correctly with above patch. I had an incorrect expire_fragment regexp.

I do see that cache fragments are stored in an encoded fashion:
```
jwilso...
```

Saturday, September 11 2010

5611 active_support/cache/file_store.rb is this correct? was updated in Ruby on Rails

Jim Wilson commented at 10:55 PM
Wait... Its still not quite right. Let me debug some more.
Jim Wilson commented at 8:37 PM
OK, I dug into this a bit deeper and I think I have both a test case showing failure and a fix. My prior suggested change just prevents a whiny nil dump, but do...

Friday, September 10 2010

5611 active_support/cache/file_store.rb is this correct? was created in Ruby on Rails

Jim Wilson created the ticket at 11:41 PM
This is crashing on me when I try and run a functional test of a cache sweeper in the test env.

I wonder if this shouldn't be "match.ord.to_s(16)"??
```
    # Tran...
```