Joel Chippindale's profile

Joel Chippindale commented at 12:57 PM
This is still an issue with master, here is a new version of the patch which applies cleanly to current master.
Joel Chippindale commented at 12:38 PM
Piotr,

The patch for master does not use /;/, rather they both use ';'

If you look in the patch for master and for 2.3 the line is
```
[Mime::Type.lookup(accept_he...
```

- State changed from stale to open
by Joel Chippindale at 12:26 PM
[state:open] This patch still applies cleanly, and without it the version of Memoizable on rails/master still fails to memoize private and protected methods cor...

Joel Chippindale commented at 12:08 PM
And patch for 2-3-stable
- Tag changed from http accepts, actionpack to http accepts, actionpack, patch
by Joel Chippindale at 12:07 PM
Attached patch for master
Joel Chippindale created the ticket at 11:54 AM

Mime::Type#parse doesn't split out the accept params from the media range when there is a single media range.

I.e. it incorrectly parses the valid HTTP_ACCEPTS ...

- Tag changed from activesupport, memoizable to activesupport, memoizable, patch
by Joel Chippindale at 11:23 AM
Patch to fix the bug (with additional tests) attached
Joel Chippindale created the ticket at 11:02 AM

Memoizable#prime_cache ignores private and protected methods.

This means that memoization of private and protected methods does not work on objects that are fro...