diff --git a/laravel/blade.php b/laravel/blade.php index 31dfde1d..349234c8 100644 --- a/laravel/blade.php +++ b/laravel/blade.php @@ -2,35 +2,13 @@ class Blade { - /** - * The cache key for the extension tree. - * - * @var string - */ - const cache = 'laravel.blade.extensions'; - - /** - * An array containing the template extension tree. - * - * @var array - */ - public static $extensions; - - /** - * The original extension tree loaded from the cache. - * - * @var array - */ - public static $original; - /** * All of the compiler functions used by Blade. * * @var array */ protected static $compilers = array( - 'extends', - 'includes', + 'layouts', 'echos', 'forelse', 'empty', @@ -38,6 +16,9 @@ class Blade { 'structure_openings', 'structure_closings', 'else', + 'includes', + 'yields', + 'yield_sections', 'section_start', 'section_end', ); @@ -49,8 +30,6 @@ class Blade { */ public static function sharpen() { - static::extensions(); - Event::listen(View::engine, function($view) { // The Blade view engine should only handle the rendering of views which @@ -80,42 +59,6 @@ public static function sharpen() }); } - /** - * Load the extension tree so we can correctly invalidate caches. - * - * @return void - */ - protected static function extensions() - { - // The entire view extension tree is cached so we can check for expired - // views anywhere in the tree. This allows us to recompile a child - // view if any of its parent views change throughout the tree. - static::$extensions = Cache::get(Blade::cache); - - static::$original = static::$extensions; - - // If no extension tree was present, we need to invalidate every cache - // since we have no way of knowing which views needs to be compiled - // since we don't know any of their parent views. - if (is_null(static::$extensions)) - { - static::flush(); - - static::$extensions = array(); - } - - // We'll hook into the "done" event of Laravel and write out the tree - // of extensions if it was changed during the course of the request. - // The tree would change if new templates were rendered, etc. - Event::listen('laravel.done', function() - { - if (Blade::$extensions !== Blade::$original) - { - Cache::forever(Blade::cache, Blade::$extensions); - } - }); - } - /** * Determine if a view is "expired" and needs to be re-compiled. * @@ -128,39 +71,7 @@ public static function expired($view, $path) { $compiled = static::compiled($path); - return filemtime($path) > filemtime($compiled) or static::expired_parent($view); - } - - /** - * Determine if the given view has an expired parent view. - * - * @param string $view - * @return bool - */ - protected static function expired_parent($view) - { - // If the view is extending another view, we need to recursively check - // whether any of the extended views have expired, all the way up to - // the top most parent view of the extension chain. - if (isset(static::$extensions[$view])) - { - $e = static::$extensions[$view]; - - return static::expired($e['view'], $e['path']); - } - - return false; - } - - /** - * Get the fully qualified path for a compiled view. - * - * @param string $view - * @return string - */ - public static function compiled($path) - { - return path('storage').'views/'.md5($path); + return filemtime($path) > filemtime(static::compiled($path)); } /** @@ -193,101 +104,44 @@ public static function compile_string($value, $view = null) return $value; } - /** - * Rewrites Blade extended templates into valid PHP. - * - * @param string $value - * @param View $view - * @return string - */ - protected static function compile_extends($value, $view) + protected static function compile_layouts($value) { - // If the view doesn't begin with @extends, we don't need to do anything - // and can simply return the view to be parsed by the rest of Blade's - // compilers like any other normal Blade view would be compiled. - if (is_null($view) or ! starts_with($value, '@extends')) + // If the Blade template is not using "layouts", we'll just return it + // it unchanged since there is nothing to do with layouts and we'll + // just let the other Blade compilers handle it. + if ( ! starts_with($value, '@layout')) { return $value; } - // First we need to parse the parent template from the extends keyword - // so we know which parent to render. We will remove the extends - // from the template after we have extracted the parent. - $template = static::extract_template($value); + // First we'll split out the lines of the template so we can get the + // the layout from the top of the template. By convention it must + // be located on the first line of the template contents. + $lines = preg_split("/(\r?\n)/", $value); - $path = static::store_extended($value, $view); + $layout = static::extract($lines[0], '@layout'); - // Once we have stored a copy of the view without the "extends" clause - // we can load up that stored view and render it. The extending view - // should only be using "sections", so we don't need the output. - View::make("path: {$path}", $view->data())->render(); + // We will add a "render" statement to the end of the templates and + // and then slice off the @layout shortcut from the start so the + // sections register before the parent template renders. + $lines[] = ""; - $parent = View::make($template); - - // Finally we will make and return the parent view as the output of - // the compilation. We'll touch the parent to force it to compile - // when it is rendered so we can make sure we're all fresh. - touch($parent->path); - - static::log_extension($view, $parent); - - return $parent->render(); + return implode(CRLF, array_slice($lines, 1)); } /** - * Extract the parent template name from an extending view. + * Extract a variable value out of a Blade expression. * * @param string $value * @return string */ - protected static function extract_template($value) + protected static function extract($value, $expression) { - preg_match('/@extends(\s*\(.*\))(\s*)/', $value, $matches); + preg_match('/'.$expression.'(\s*\(.*\))(\s*)/', $value, $matches); return str_replace(array("('", "')"), '', $matches[1]); } - /** - * Store an extended view in the view storage. - * - * @param string $value - * @param View $view - * @return array - */ - protected static function store_extended($value, $view) - { - $value = preg_replace('/@extends(\s*\(.*\))(\s*)/', '', $value); - - file_put_contents($path = static::compiled($view->path.'_extended').BLADE_EXT, $value); - - return $path; - } - - /** - * Log a view extension for a given view in the extension tree. - * - * @param View $view - * @param View $parent - * @return void - */ - protected static function log_extension($view, $parent) - { - static::$extensions[$view->view] = array('view' => $parent->view, 'path' => $parent->path); - } - - /** - * Rewrites Blade "include" statements to valid PHP. - * - * @param string $value - * @return string - */ - protected static function compile_includes($value) - { - $pattern = '/\{\{(\s*)include(\s*\(.*\))(\s*)\}\}/'; - - return preg_replace($pattern, 'with(get_defined_vars()); ?>', $value); - } - /** * Rewrites Blade echo statements into PHP echo statements. * @@ -309,31 +163,25 @@ protected static function compile_forelse($value) { preg_match_all('/(\s*)@forelse(\s*\(.*\))(\s*)/', $value, $matches); - // First we'll loop through all of the "@forelse" lines. We need to - // wrap each loop in an if/else statement that checks the count - // of the variable that is being iterated by the loop. - if (isset($matches[0])) + foreach ($matches[0] as $forelse) { - foreach ($matches[0] as $forelse) - { - preg_match('/\$[^\s]*/', $forelse, $variable); + preg_match('/\$[^\s]*/', $forelse, $variable); - // Once we have extracted the variable being looped against, we can - // prepend an "if" statmeent to the start of the loop that checks - // that the count of the variable is greater than zero. - $if = " 0): ?>"; + // Once we have extracted the variable being looped against, we can add + // an if statmeent to the start of the loop that checks if the count + // of the variable being looped against is greater than zero. + $if = " 0): ?>"; - $search = '/(\s*)@forelse(\s*\(.*\))/'; + $search = '/(\s*)@forelse(\s*\(.*\))/'; - $replace = '$1'.$if.''; + $replace = '$1'.$if.''; - $blade = preg_replace($search, $replace, $forelse); + $blade = preg_replace($search, $replace, $forelse); - // Finally, once we have the check prepended to the loop, we will - // replace all instances of this "forelse" syntax in the view - // content of the view being compiled to Blade syntax. - $value = str_replace($forelse, $blade, $value); - } + // Finally, once we have the check prepended to the loop we'll replace + // all instances of this "forelse" syntax in the view content of the + // view being compiled to Blade syntax with real syntax. + $value = str_replace($forelse, $blade, $value); } return $value; @@ -398,6 +246,46 @@ protected static function compile_else($value) return preg_replace('/(\s*)@(else)(\s*)/', '$1$3', $value); } + /** + * Rewrites Blade @include statements into valid PHP. + * + * @param string $value + * @return string + */ + protected static function compile_includes($value) + { + $pattern = static::matcher('include'); + + return preg_replace($pattern, '$1with(get_defined_vars()); ?>', $value); + } + + /** + * Rewrites Blade @yield statements into Section statements. + * + * The Blade @yield statement is a shortcut to the Section::yield method. + * + * @param string $value + * @return string + */ + protected static function compile_yields($value) + { + $pattern = static::matcher('yield'); + + return preg_replace($pattern, '$1', $value); + } + + /** + * Rewrites Blade yield section statements into valid PHP. + * + * @return string + */ + protected static function compile_yield_sections($value) + { + $replace = ''; + + return str_replace('@yield_section', $replace, $value); + } + /** * Rewrites Blade @section statements into Section statements. * @@ -438,21 +326,14 @@ protected static function matcher($function) } /** - * Remove all of the cached views from storage. + * Get the fully qualified path for a compiled view. * - * @return void + * @param string $view + * @return string */ - protected static function flush() + public static function compiled($path) { - $items = new fIterator(path('storage').'views'); - - foreach ($items as $item) - { - if ($item->isFile() and $item->getBasename() !== '.gitignore') - { - @unlink($item->getRealPath()); - } - } + return path('storage').'views/'.md5($path); } } \ No newline at end of file diff --git a/laravel/section.php b/laravel/section.php index d913a12e..ff12889f 100644 --- a/laravel/section.php +++ b/laravel/section.php @@ -14,7 +14,7 @@ class Section { * * @var array */ - protected static $last = array(); + public static $last = array(); /** * Start injecting content into a section. @@ -33,9 +33,14 @@ class Section { */ public static function start($section, $content = '') { - if ($content === '') ob_start() and static::$last[] = $section; - - static::append($section, $content); + if ($content === '') + { + ob_start() and static::$last[] = $section; + } + else + { + static::append($section, $content); + } } /** @@ -57,14 +62,26 @@ public static function inject($section, $content) static::start($section, $content); } + /** + * Stop injecting content into a section and return its contents. + * + * @return string + */ + public static function yield_section() + { + return static::yield(static::stop()); + } + /** * Stop injecting content into a section. * - * @return void + * @return string */ public static function stop() { - static::append(array_pop(static::$last), ob_get_clean()); + static::append($last = array_pop(static::$last), ob_get_clean()); + + return $last; } /** @@ -78,10 +95,12 @@ protected static function append($section, $content) { if (isset(static::$sections[$section])) { - $content = static::$sections[$section].PHP_EOL.$content; + static::$sections[$section] = str_replace('@parent', $content, static::$sections[$section]); + } + else + { + static::$sections[$section] = $content; } - - static::$sections[$section] = $content; } /** diff --git a/laravel/view.php b/laravel/view.php index 1fc8823d..4b649ceb 100644 --- a/laravel/view.php +++ b/laravel/view.php @@ -349,11 +349,11 @@ public function data() // All nested views and responses are evaluated before the main view. // This allows the assets used by nested views to be added to the // asset container before the main view is evaluated. - foreach ($data as &$value) + foreach ($data as $key => $value) { if ($value instanceof View or $value instanceof Response) { - $value = $value->render(); + $data[$key] = $value->render(); } }