Ori.livneh has uploaded a new change for review.
https://gerrit.wikimedia.org/r/87493
Change subject: LESS embeddable(): Use lessc::toBool
......................................................................
LESS embeddable(): Use lessc::toBool
With Ib6bc76736 in place, we have access to lessphp's custom function helpers,
which provide some nice encapsulation for interpreter internals like boolean
nodes. This patch makes embeddable() call $less->toBool() rather than construct
its own AST nodes.
Change-Id: I0a015564dff19f5ced764af182558eb351e3d816
---
M includes/resourceloader/ResourceLoaderLESSFunctions.php
1 file changed, 2 insertions(+), 3 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core
refs/changes/93/87493/1
diff --git a/includes/resourceloader/ResourceLoaderLESSFunctions.php
b/includes/resourceloader/ResourceLoaderLESSFunctions.php
index 08d574c..c7570f6 100644
--- a/includes/resourceloader/ResourceLoaderLESSFunctions.php
+++ b/includes/resourceloader/ResourceLoaderLESSFunctions.php
@@ -39,11 +39,10 @@
$base = pathinfo( $less->parser->sourceName, PATHINFO_DIRNAME );
$url = $frame[2][0];
$file = realpath( $base . '/' . $url );
- $embeddable = ( $file
+ return $less->toBool( $file
&& strpos( $url, '//' ) === false
&& filesize( $file ) < CSSMin::EMBED_SIZE_LIMIT
- && CSSMin::getMimeType( $file ) !== false ) ? 'true' :
'false';
- return array( 'keyword', $embeddable );
+ && CSSMin::getMimeType( $file ) !== false );
}
/**
--
To view, visit https://gerrit.wikimedia.org/r/87493
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I0a015564dff19f5ced764af182558eb351e3d816
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Ori.livneh <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
