The php_jsmin extension, being pre-compiled code, is likely to be significantly faster than the PHP-based JSMin even when using APC. Attached is a patch to enable its use.

CommentFileSizeAuthor
#4 0001-Add-support-for-ypass.net-s-conversion-of-jsmin-to-a.patch5.03 KBdougp
javascript_aggregator.module.php_jsmin.patch2.3 KBgreenreaper

Comments

AntiNSA’s picture

AntiNSA commented

Anyone using this?

AntiNSA’s picture

AntiNSA commented
Priority: Normal » Major

So, do you check the box for:

GZip JavaScript

Use php_jsmin instead of JSMin

Use JSMin+ instead of JSMin

aLL THREE, OR JUST ONE OF THREE , OR JUST GZ ZIP and php_jsmin?

Thanks for the help. Maybe good to put in readme

mikeytown2’s picture

mikeytown2 commented

subscribe/hijack... #1109978: Add support for php_jsmin extension

Committed this code

dougp’s picture

dougp commented
StatusFileSize
new 0001-Add-support-for-ypass.net-s-conversion-of-jsmin-to-a.patch5.03 KB

I made my own version of this without realising that someone had already done it. I like my version better, it replaces the "javascript_aggregator_jsminplus" variable with a "javascript_aggregator_minifier" setting which allows you to select only one minifier.

  • jsmin
  • jsmin+
  • jsmin.so php extension

I think ideally though, the module would be extensible with some sort of hook to implement other minifiers/javascript processors, there are quite a few other ones out there.