Use new placeholder layout features of Saft

splattater · Jun 2, 2013 · 13b8b43 · 13b8b43
1 parent 3bd2420
commit 13b8b43
Show file tree

Hide file tree

Showing 16 changed files with 35 additions and 54 deletions.
diff --git a/classes/Xodx/Application.php b/classes/Xodx/Application.php
@@ -19,7 +19,7 @@ public function run() {
             /**
              * Prepare Template
              */
-            $this->_layout->setLayout('templates/layout.phtml');
+            $this->_layout->setLayout('layout.phtml');
 
             $request = $bootstrap->getResource('request');
 
@@ -47,18 +47,23 @@ public function run() {
 
             $this->_layout->username = $user->getName();
             $this->_layout->notifications = $userController->getNotifications($user->getUri());
-            $this->_layout->addDebug($bootstrap->getResource('logger')->getLastLog());
 
             $config = $bootstrap->getResource('config');
-            if (isset($config['debug']) && $config['debug'] == false) {
-                $this->_layout->disableDebug();
+            if (!isset($config['debug']) || $config['debug'] == true) {
+                $debug = new Saft_Template(
+                    'debug.phtml', array('log' => $bootstrap->getResource('logger')->getLastLog())
+                );
+                $this->_layout->setPlaceholder('debug', $debug);
             }
         } catch (Exception $e) {
-            $this->_layout->setLayout('templates/errorlayout.phtml');
-            $this->_layout->errorType = 'Application Error';
-            $this->_layout->exception = $e;
-            $this->_layout->back = $this->getBaseUri();
-            $this->_layout->addContent('templates/error.phtml');
+            $this->_layout->setLayout('errorlayout.phtml');
+            $options = array(
+                'errorType' => 'Application Error',
+                'exception' => $e,
+                'back' => $this->getBaseUri()
+            );
+            $errorPlaceholder = new Saft_Template('error.phtml', $options);
+            $this->_layout->setPlaceholder('error', $errorPlaceholder);
         }
         $this->_layout->render();
     }

diff --git a/classes/Xodx/FeedController.php b/classes/Xodx/FeedController.php
@@ -45,7 +45,7 @@ public function getFeedAction($template, $uri = null, $format = null)
 
             $nameHelper = new Xodx_NameHelper($this->_app);
 
-            $template->setLayout('templates/feed.phtml');
+            $template->setLayout('feed.phtml');
             $template->updated = $updated;
             $template->uri = $uri;
             $template->feedUri = $feedUri;

diff --git a/libraries/Saft b/libraries/Saft
diff --git a/templates/activity/addactivity.phtml b/templates/activity/addactivity.phtml
@@ -0,0 +1,2 @@
+<h2>New Activity</h2>
+A new activity has been created
diff --git a/templates/reply.phtml → templates/activity/reply.phtml b/templates/reply.phtml → templates/activity/reply.phtml
diff --git a/templates/login.phtml → templates/application/login.phtml b/templates/login.phtml → templates/application/login.phtml
diff --git a/templates/newuser.phtml → templates/application/newuser.phtml b/templates/newuser.phtml → templates/application/newuser.phtml
diff --git a/templates/debug.phtml b/templates/debug.phtml
@@ -1,5 +1,8 @@
-<?php
-    if (isset($this->debugText)) {
-        echo $this->debugText;
-    }
-?>
+<?php if (!empty($this->log)) : ?>
+<div class="debug" id="debugConsole">
+    <h3>Debuglog:</h3>
+    <pre>
+    <?= $this->log ?>
+    </pre>
+</div>
+<?php endif; ?>
diff --git a/templates/errorlayout.phtml b/templates/errorlayout.phtml
@@ -34,22 +34,11 @@
     </div>
 
     <div class="container">
-        <?php
-        foreach ($this->_contentFiles as $contentFile) {
-            include $contentFile;
-        }
-    ?>
-    </div>
-
-    <?php if ($this->_debug && !empty($this->_debugLog)) : ?>
-
-    <div class="debug" id="debugConsole">
-        <h3>Debuglog:</h3>
-        <pre>
-<?= $this->_debugLog ?>
-        </pre>
+        <?php $this->placeholder('error'); ?>
     </div>
 
+    <?php if ($this->has('debug')) : ?>
+    <?php $this->placeholder('debug'); ?>
     <?php endif; ?>
 </body>
 </html>
diff --git a/templates/layout.phtml b/templates/layout.phtml
@@ -31,7 +31,7 @@
                 <a class="brand"><img src="resources/img/xodx_20.png" alt="Xodx" title="[ˈɛksodʊs]" /></a>
                 <?php
                     $this->partial(
-                        'templates/menu.phtml',
+                        'menu.phtml',
                         array('username' => $this->username, 'notifications' => $this->notifications)
                     );
                 ?>
@@ -40,22 +40,11 @@
     </div>
 
     <div class="container">
-        <?php
-        foreach ($this->_contentFiles as $contentFile) {
-            include $contentFile;
-        }
-    ?>
-    </div>
-
-    <?php if ($this->_debug && !empty($this->_debugLog)) : ?>
-
-    <div class="debug" id="debugConsole">
-        <h3>Debuglog:</h3>
-        <pre>
-<?= $this->_debugLog ?>
-        </pre>
+        <?php $this->placeholder('main.content'); ?>
     </div>
 
+    <?php if ($this->has('debug')) : ?>
+    <?php $this->placeholder('debug'); ?>
     <?php endif; ?>
 </body>
 </html>
diff --git a/templates/newactivity.phtml b/templates/newactivity.phtml
diff --git a/templates/menu.phtml → templates/partials/menu.phtml b/templates/menu.phtml → templates/partials/menu.phtml
diff --git a/templates/profileshow.phtml → templates/person/show.phtml b/templates/profileshow.phtml → templates/person/show.phtml
@@ -51,7 +51,7 @@
         <hr />
         <?php
             $this->partial(
-                'templates/partials/friendlist.phtml',
+                'friendlist.phtml',
                 array('personUri' => $this->profileshowPersonUri, 'knows' => $this->profileshowKnows, 'addFriend' => false)
             );
         ?>

diff --git a/templates/profilelist.phtml → templates/profile/list.phtml b/templates/profilelist.phtml → templates/profile/list.phtml
diff --git a/templates/resourceshow.phtml → templates/resource/show.phtml b/templates/resourceshow.phtml → templates/resource/show.phtml
diff --git a/templates/usershow.phtml → templates/user/home.phtml b/templates/usershow.phtml → templates/user/home.phtml
@@ -93,7 +93,7 @@
         <hr />
         <?php
             $this->partial(
-                'templates/partials/friendlist.phtml',
+                'friendlist.phtml',
                 array('personUri' => $this->profileshowPersonUri, 'knows' => $this->profileshowKnows, 'addFriend' => true)
             );
         ?>
+1 −1		Application.php
+18 −17		Layout.php
+2 −2		Template.php
+25 −0		Template/Layout.php
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		<h2>New Activity</h2>
		A new activity has been created