framework/bedrock
Template
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Memoize asset_path function

Browse Source 
Previously roots would do a file operation each time asset_path is
called. Now it will only parse the json manifest one time per program
execution.

Extract json reading to class that can be used to read bower.json
This commit is contained in:
Austin Pray
2015-01-30 23:13:45 -06:00
parent 3f7c8459f7
commit f93a1f2c8c
1 changed files with 22 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

27
lib/assets.php
View File

@@ -17,17 +17,34 @@ namespace Roots\Sage\Assets;
* - An ID has been defined in config.php
* - You're not logged in as an administrator
*/
class JsonManifest {
private $manifest;
public function __construct($manifest_path) {
if (file_exists($manifest_path)) {
$this->manifest = json_decode(file_get_contents($manifest_path), true);
} else {
$this->manifest = [];
}
}
public function get() {
return $this->manifest;
}
}
function asset_path($filename) {
$dist_path = get_template_directory_uri() . '/dist/';
$directory = dirname($filename) . '/';
$file = basename($filename);
$manifest_path = get_template_directory() . '/dist/assets.json';
static $manifest;
if (file_exists($manifest_path)) {
$manifest = json_decode(file_get_contents($manifest_path), true);
} else {
$manifest = [];
if (empty($manifest)) {
$manifest_path = get_template_directory() . '/dist/assets.json';
$manifest = new JsonManifest($manifest_path);
$manifest = $manifest->get();
}
if (WP_ENV !== 'development' && array_key_exists($file, $manifest)) {