flourishlib/flourish-classes — pull request #70 by mattsah (2012-05-11 07:52:43+00:00)
Title: Changes to Flourish

Hey Will, this is a single commit which patched a full clean slate fork of your flourish-classes repository with all the .inK changes. Unfortunately, this commit does contain whitespace changes. I assume with a pull request you need to select pretty much all or nothing, but perhaps you can do more selective merging. I've never actually done one, so I'm not sure, but either way, aside from all trailing whitespace being stripped here's the changes as have been noted in multiple places:

- Added fJSONSerializable non-native implementation support
- Made CSS/JS compression in fTemplating use PHP to include (i.e. you can put PHP in CSS and JS if it's compressed)

One of the more recent changes is to fix issue #69 on the issues list for your repo. I looked through the code and realized it was just a matter of trimming the content type of whitespace to ensure RegEx matches worked properly.

- Fixed bug in fRequest so that types are trimmed for proper RegEx matching

Take what you want, none at all, or just copy the bugfix manually for all I care, I'll deal with the merging from upstream on my fork down the line if you don't want to pull the full commit.