MDL-53695 minify: switch to MatthiasMullie\Minify
The previously used minification libraries were: a) Unmaintained b) Had some silly licensing issues (MDL-36457) The new library is smaller, maintained, active and GPL-compatible.
This commit is contained in:
@@ -64,12 +64,13 @@ color: #281f18;
|
||||
|
||||
$files = array($testfile1, $testfile2);
|
||||
|
||||
$this->assertSame("body{background:#fff;margin:0;padding:0;color:#281f18}\nbody{}", core_minify::css_files($files));
|
||||
$this->assertSame("body{background:#fff;margin:0;padding:0;color:#281f18}\n", core_minify::css_files($files));
|
||||
|
||||
|
||||
$files = array($testfile1, $testfile2, $testfile3);
|
||||
|
||||
$this->assertStringStartsWith("body{background:#fff;margin:0;padding:0;color:#281f18}\nbody{}\n\n\n/* Cannot read CSS file ", @core_minify::css_files($files));
|
||||
$this->assertStringStartsWith("body{background:#fff;margin:0;padding:0;color:#281f18}\n\n\n\n/* Cannot read CSS file ",
|
||||
@core_minify::css_files($files));
|
||||
|
||||
unlink($testfile1);
|
||||
unlink($testfile2);
|
||||
@@ -82,11 +83,10 @@ function hm()
|
||||
}
|
||||
";
|
||||
|
||||
$this->assertSame("function hm(){}", core_minify::js($js));
|
||||
$this->assertSame("function hm()\n{}", core_minify::js($js));
|
||||
|
||||
$js = "function hm{}";
|
||||
$result = core_minify::js($js);
|
||||
$this->assertStringStartsWith("\ntry {console.log('Error: Minimisation of JavaScript failed!');} catch (e) {}", $result);
|
||||
$this->assertContains($js, $result);
|
||||
}
|
||||
|
||||
@@ -110,11 +110,12 @@ function hm()
|
||||
|
||||
$files = array($testfile1, $testfile2);
|
||||
|
||||
$this->assertSame("function hm(){};\nfunction oh(){}", core_minify::js_files($files));
|
||||
$this->assertSame("function hm()\n{};\nfunction oh(){}", core_minify::js_files($files));
|
||||
|
||||
$files = array($testfile1, $testfile2, $testfile3);
|
||||
|
||||
$this->assertStringStartsWith("function hm(){};\nfunction oh(){};\n\n\n// Cannot read JS file ", @core_minify::js_files($files));
|
||||
$this->assertStringStartsWith("function hm()\n{};\nfunction oh(){};\n\n\n// Cannot read JS file ",
|
||||
@core_minify::js_files($files));
|
||||
|
||||
unlink($testfile1);
|
||||
unlink($testfile2);
|
||||
|
||||
Reference in New Issue
Block a user