Jim Lindley's profile

Jim Lindley commented at 7:07 PM
Fixed thanks to Rick and Will - the edit link shows up for me now.
Jim Lindley commented at 5:04 PM
Ryan, I get those edit links on my own Lighthouse projects, but not when I'm logged in here at the Rails project.
Jim Lindley commented at 4:34 PM
Doesn't look like I can edit the original. Wish there was a preview button. Stuck with monospaced.
Jim Lindley commented at 4:02 PM
Aaargh, getting the code formatting wrong. Trying again, with Ryan's example.

Existing behaviour:
```
    cache = ActiveSupport::Cache.lookup_store(:memory_stor...
```
Jim Lindley created the ticket at 4:00 PM
This was a side issue to ticket #785. Splitting out into it's own ticket for further discussion. MemoryStore cache contents can be altered by changing fetched o...

Jim Lindley commented at 4:00 PM
New ticket #799 created to move the MemoryStore issue to, to not detract from the class reloading in development problems.
Jim Lindley commented at 3:47 PM
If the cache contents can be altered by changing fetched objects, then it's not a cache store, it's a fancy global variable to store objects in.

Jim Lindley commented at 11:10 PM
Dup'ing MemoryStore still a good idea in production, though.
Jim Lindley commented at 11:09 PM
Oh, shoot. Yeah, the class gets nuked out from under it still.
Jim Lindley commented at 11:08 PM

One thing your patch is missing (the only thing, everything else looks good) is a rescue TypeError when dup is attempted on inputs like false or nil.

Maybe som...