Tweaks

[DouweM]

I did some refactoring, plus I made the following significant changes:

• Wrap cache read/write calls in notification instrumentation for fragment read/write logging.
• Use correct template when templates are derived using item.to_partial_path.

[n8]

@DouweM just pulled in some pull requests, and now your work conflicts. Would you mind please updating? Sorry about the late attention to this.

[DouweM]

Yup, I'll rebase tonight.

[DouweM]

Updated!

[n8]

Is the custom key test failing for you?

1) MultiFetchFragments works for passing in a custom key
     Failure/Error: view.render(:partial => "views/customer", :collection => [ customer ], :cache => Proc.new{ |item| [item, 'key']}).should == "Hello: david\n"
       expected: "Hello: david\n"
            got: "Hello" (using ==)
       Diff:
       @@ -1,2 +1,2 @@
       -Hello: david
       +Hello

[DouweM]

Edit: Disregard this.

It was failing for me before when it read .should == "Hello"; it works with the current .should == "Hello: david\n". I'm not sure where Hello comes from any way, seeing how spec/views/_customer.html.erb reads Hello: [name].

[DouweM]

Any idea when you can review this?

[DouweM]

@n8 Ping!