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.

There was a problem

You must be a member of this account.

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...
```