@@ -34,82 +34,82 @@ |
||
| 34 | 34 | */ |
| 35 | 35 | class PageLayoutRenderer |
| 36 | 36 | { |
| 37 | - /** |
|
| 38 | - * @var bool Whether to force (or not force) embedding the content in the layout |
|
| 39 | - */ |
|
| 40 | - protected $use_layout; |
|
| 37 | + /** |
|
| 38 | + * @var bool Whether to force (or not force) embedding the content in the layout |
|
| 39 | + */ |
|
| 40 | + protected $use_layout; |
|
| 41 | 41 | |
| 42 | - /** |
|
| 43 | - * @var HTMLRenderer |
|
| 44 | - */ |
|
| 45 | - protected $view_renderer; |
|
| 42 | + /** |
|
| 43 | + * @var HTMLRenderer |
|
| 44 | + */ |
|
| 45 | + protected $view_renderer; |
|
| 46 | 46 | |
| 47 | - /** |
|
| 48 | - * @var \Request |
|
| 49 | - */ |
|
| 50 | - protected $current_request; |
|
| 47 | + /** |
|
| 48 | + * @var \Request |
|
| 49 | + */ |
|
| 50 | + protected $current_request; |
|
| 51 | 51 | |
| 52 | - public function __construct(Renderer $view_renderer, \Request $current_request = NULL) |
|
| 53 | - { |
|
| 54 | - $this->view_renderer = $view_renderer; |
|
| 55 | - $this->current_request = $current_request; |
|
| 56 | - } |
|
| 52 | + public function __construct(Renderer $view_renderer, \Request $current_request = NULL) |
|
| 53 | + { |
|
| 54 | + $this->view_renderer = $view_renderer; |
|
| 55 | + $this->current_request = $current_request; |
|
| 56 | + } |
|
| 57 | 57 | |
| 58 | - /** |
|
| 59 | - * @param PageContentView $content_view |
|
| 60 | - * |
|
| 61 | - * @return string |
|
| 62 | - */ |
|
| 63 | - public function render(PageContentView $content_view) |
|
| 64 | - { |
|
| 65 | - $content = $this->view_renderer->render($content_view); |
|
| 66 | - if ($this->shouldUseLayout()) { |
|
| 67 | - return $this->renderInLayout($content_view->var_page(), $content); |
|
| 68 | - } else { |
|
| 69 | - return $content; |
|
| 70 | - } |
|
| 71 | - } |
|
| 58 | + /** |
|
| 59 | + * @param PageContentView $content_view |
|
| 60 | + * |
|
| 61 | + * @return string |
|
| 62 | + */ |
|
| 63 | + public function render(PageContentView $content_view) |
|
| 64 | + { |
|
| 65 | + $content = $this->view_renderer->render($content_view); |
|
| 66 | + if ($this->shouldUseLayout()) { |
|
| 67 | + return $this->renderInLayout($content_view->var_page(), $content); |
|
| 68 | + } else { |
|
| 69 | + return $content; |
|
| 70 | + } |
|
| 71 | + } |
|
| 72 | 72 | |
| 73 | - /** |
|
| 74 | - * @return bool |
|
| 75 | - */ |
|
| 76 | - protected function shouldUseLayout() |
|
| 77 | - { |
|
| 78 | - if ($this->use_layout !== NULL) { |
|
| 79 | - return $this->use_layout; |
|
| 80 | - } |
|
| 73 | + /** |
|
| 74 | + * @return bool |
|
| 75 | + */ |
|
| 76 | + protected function shouldUseLayout() |
|
| 77 | + { |
|
| 78 | + if ($this->use_layout !== NULL) { |
|
| 79 | + return $this->use_layout; |
|
| 80 | + } |
|
| 81 | 81 | |
| 82 | - if ($this->current_request AND $this->current_request->is_ajax()) { |
|
| 83 | - return FALSE; |
|
| 84 | - } else { |
|
| 85 | - return TRUE; |
|
| 86 | - } |
|
| 87 | - } |
|
| 82 | + if ($this->current_request AND $this->current_request->is_ajax()) { |
|
| 83 | + return FALSE; |
|
| 84 | + } else { |
|
| 85 | + return TRUE; |
|
| 86 | + } |
|
| 87 | + } |
|
| 88 | 88 | |
| 89 | - /** |
|
| 90 | - * @param PageLayoutView $layout |
|
| 91 | - * @param string $content |
|
| 92 | - * |
|
| 93 | - * @return string |
|
| 94 | - */ |
|
| 95 | - protected function renderInLayout(PageLayoutView $layout, $content) |
|
| 96 | - { |
|
| 97 | - $layout->setBodyHTML($content); |
|
| 89 | + /** |
|
| 90 | + * @param PageLayoutView $layout |
|
| 91 | + * @param string $content |
|
| 92 | + * |
|
| 93 | + * @return string |
|
| 94 | + */ |
|
| 95 | + protected function renderInLayout(PageLayoutView $layout, $content) |
|
| 96 | + { |
|
| 97 | + $layout->setBodyHTML($content); |
|
| 98 | 98 | |
| 99 | - return $this->view_renderer->render($layout); |
|
| 100 | - } |
|
| 99 | + return $this->view_renderer->render($layout); |
|
| 100 | + } |
|
| 101 | 101 | |
| 102 | - /** |
|
| 103 | - * Configure whether to always wrap the content in the layout (TRUE), never (FALSE) or automatically for |
|
| 104 | - * non-AJAX requests (NULL) |
|
| 105 | - * |
|
| 106 | - * @param bool $use_layout |
|
| 107 | - * |
|
| 108 | - * @return void |
|
| 109 | - */ |
|
| 110 | - public function setUseLayout($use_layout) |
|
| 111 | - { |
|
| 112 | - $this->use_layout = $use_layout; |
|
| 113 | - } |
|
| 102 | + /** |
|
| 103 | + * Configure whether to always wrap the content in the layout (TRUE), never (FALSE) or automatically for |
|
| 104 | + * non-AJAX requests (NULL) |
|
| 105 | + * |
|
| 106 | + * @param bool $use_layout |
|
| 107 | + * |
|
| 108 | + * @return void |
|
| 109 | + */ |
|
| 110 | + public function setUseLayout($use_layout) |
|
| 111 | + { |
|
| 112 | + $this->use_layout = $use_layout; |
|
| 113 | + } |
|
| 114 | 114 | |
| 115 | 115 | } |
@@ -21,10 +21,10 @@ |
||
| 21 | 21 | interface PageContentView extends ViewModel |
| 22 | 22 | { |
| 23 | 23 | |
| 24 | - /** |
|
| 25 | - * The page layout that this content view will be rendered into |
|
| 26 | - * |
|
| 27 | - * @return PageLayoutView |
|
| 28 | - */ |
|
| 29 | - public function var_page(); |
|
| 24 | + /** |
|
| 25 | + * The page layout that this content view will be rendered into |
|
| 26 | + * |
|
| 27 | + * @return PageLayoutView |
|
| 28 | + */ |
|
| 29 | + public function var_page(); |
|
| 30 | 30 | } |
@@ -1,9 +1,9 @@ discard block |
||
| 1 | 1 | <?php |
| 2 | 2 | /** |
| 3 | - * @author Andrew Coulton <[email protected]> |
|
| 4 | - * @copyright 2015 inGenerator Ltd |
|
| 5 | - * @license http://kohanaframework.org/license |
|
| 6 | - */ |
|
| 3 | + * @author Andrew Coulton <[email protected]> |
|
| 4 | + * @copyright 2015 inGenerator Ltd |
|
| 5 | + * @license http://kohanaframework.org/license |
|
| 6 | + */ |
|
| 7 | 7 | |
| 8 | 8 | namespace Ingenerator\KohanaView\ViewModel\PageLayout; |
| 9 | 9 | |
@@ -27,23 +27,23 @@ discard block |
||
| 27 | 27 | abstract class AbstractPageContentView extends AbstractViewModel implements PageContentView |
| 28 | 28 | { |
| 29 | 29 | |
| 30 | - /** |
|
| 31 | - * @var PageLayoutView |
|
| 32 | - */ |
|
| 33 | - protected $page_view; |
|
| 34 | - |
|
| 35 | - public function __construct(PageLayoutView $page) |
|
| 36 | - { |
|
| 37 | - $this->page_view = $page; |
|
| 38 | - parent::__construct(); |
|
| 39 | - } |
|
| 40 | - |
|
| 41 | - /** |
|
| 42 | - * @return PageLayoutView |
|
| 43 | - */ |
|
| 44 | - public function var_page() |
|
| 45 | - { |
|
| 46 | - return $this->page_view; |
|
| 47 | - } |
|
| 30 | + /** |
|
| 31 | + * @var PageLayoutView |
|
| 32 | + */ |
|
| 33 | + protected $page_view; |
|
| 34 | + |
|
| 35 | + public function __construct(PageLayoutView $page) |
|
| 36 | + { |
|
| 37 | + $this->page_view = $page; |
|
| 38 | + parent::__construct(); |
|
| 39 | + } |
|
| 40 | + |
|
| 41 | + /** |
|
| 42 | + * @return PageLayoutView |
|
| 43 | + */ |
|
| 44 | + public function var_page() |
|
| 45 | + { |
|
| 46 | + return $this->page_view; |
|
| 47 | + } |
|
| 48 | 48 | |
| 49 | 49 | } |
@@ -26,32 +26,32 @@ |
||
| 26 | 26 | abstract class AbstractPageLayoutView extends AbstractViewModel implements ViewModel\PageLayoutView |
| 27 | 27 | { |
| 28 | 28 | |
| 29 | - /** |
|
| 30 | - * @var array |
|
| 31 | - */ |
|
| 32 | - protected $variables = [ |
|
| 33 | - 'body_html' => NULL, |
|
| 34 | - 'title' => NULL, |
|
| 35 | - ]; |
|
| 36 | - |
|
| 37 | - /** |
|
| 38 | - * @param string $title |
|
| 39 | - * |
|
| 40 | - * @return void |
|
| 41 | - */ |
|
| 42 | - public function setTitle($title) |
|
| 43 | - { |
|
| 44 | - $this->variables['title'] = $title; |
|
| 45 | - } |
|
| 46 | - |
|
| 47 | - /** |
|
| 48 | - * @param string $html |
|
| 49 | - * |
|
| 50 | - * @return void |
|
| 51 | - */ |
|
| 52 | - public function setBodyHTML($html) |
|
| 53 | - { |
|
| 54 | - $this->variables['body_html'] = $html; |
|
| 55 | - } |
|
| 29 | + /** |
|
| 30 | + * @var array |
|
| 31 | + */ |
|
| 32 | + protected $variables = [ |
|
| 33 | + 'body_html' => NULL, |
|
| 34 | + 'title' => NULL, |
|
| 35 | + ]; |
|
| 36 | + |
|
| 37 | + /** |
|
| 38 | + * @param string $title |
|
| 39 | + * |
|
| 40 | + * @return void |
|
| 41 | + */ |
|
| 42 | + public function setTitle($title) |
|
| 43 | + { |
|
| 44 | + $this->variables['title'] = $title; |
|
| 45 | + } |
|
| 46 | + |
|
| 47 | + /** |
|
| 48 | + * @param string $html |
|
| 49 | + * |
|
| 50 | + * @return void |
|
| 51 | + */ |
|
| 52 | + public function setBodyHTML($html) |
|
| 53 | + { |
|
| 54 | + $this->variables['body_html'] = $html; |
|
| 55 | + } |
|
| 56 | 56 | |
| 57 | 57 | } |
@@ -21,11 +21,11 @@ |
||
| 21 | 21 | interface PageLayoutView extends ViewModel |
| 22 | 22 | { |
| 23 | 23 | |
| 24 | - /** |
|
| 25 | - * @param string $html |
|
| 26 | - * |
|
| 27 | - * @return void |
|
| 28 | - */ |
|
| 29 | - public function setBodyHTML($html); |
|
| 24 | + /** |
|
| 25 | + * @param string $html |
|
| 26 | + * |
|
| 27 | + * @return void |
|
| 28 | + */ |
|
| 29 | + public function setBodyHTML($html); |
|
| 30 | 30 | |
| 31 | 31 | } |