Closed josegonzalez closed 13 years ago
Lots indeed! Thanks savant. Here's my initial feedback
Some issues:
will change this after merge
will change this after merge
will Not change this after merge since the diffs don't reveil if you also did content changes. i can live with textile for now
will change this after merge
will change this after merge
will apply a6977ea9fd904b882efbfa37cf81fe75d3c4dc0a after the merge
Forseeable but imo acceptable breakage:
->{$action}['extract']
is now ->extract[$action]
we should make a clear warning in the docs that people change their controller code.
Seems right to you?
Sounds good. So long as the code follows CakePHP standards (other than the @access
issue) I'll be happy.
Please pull this in. Commit messages are self-explanatory, although I'm happy to elaborate on my changes.