Errors and exceptions handling, #16

git-svn-id: svn+ssh://code.netmonsters.ru/svn/majestic/branches/evo@122 4cb57b5f-5bbd-dd11-951b-001d605cbbc5

Load.php

@@ -53,7 +53,7 @@ class Load
             trigger_error('Can\'t create directory: "' . $dir . '"', E_USER_ERROR);
         }
         
-        $scan = array(PATH . '/lib', PATH . '/' . APP . '/src');
+        $scan = array(PATH . '/' . APP . '/src', PATH . '/lib');
         
         require_once(PATH . '/lib/core/util/AutoloadBuilder.php');
         

app/Action.php

@@ -20,15 +20,12 @@ abstract class Action
     
     protected function extractParams()
     {
-        $params = FrontController::getInstance()->getRouter()->getRoute()->getParams();
-        if ($params) {
-            foreach ($params as $name => $value) {
+        foreach (Env::getParam() as $name => $value) {
             if (is_string($name)) {
                 $this->$name = $value;
             }
         }
     }
-    }
     
     abstract protected function execute();
     

app/ErrorAction.php

@@ -20,8 +20,7 @@ class ErrorAction extends ViewAction
     public function __construct($exception)
     {
         $this->exception = $exception;
-        $this->view = FrontController::getInstance()->getView();
-        $this->execute();
+        parent::__construct();
     }
     
     protected function execute()

app/FrontController.php

@@ -100,12 +100,18 @@ class FrontController
             $request = Env::getRequestUri();
             $route = $this->getRouter()->route($request);
             if (!$route) {
-                throw new Error404Exception('Route "' . $request . '" not found');
+                throw new Error404Exception('Route for "' . $request . '" not found');
             }
             
             $action_class = $route->getAction();
+            if (!class_exists($action_class)) {
+                throw new GeneralException('Action class "' . $action_class . '" not found.');
+            }
             $action = new $action_class();
             $layout_class = $route->getLayout();
+            if (!class_exists($layout_class)) {
+                throw new GeneralException('Layout class "' . $layout_class . '" not found.');
+            }
             $layout = new $layout_class();
             return $layout->fetch($action);
             
@@ -113,7 +119,7 @@ class FrontController
             if (DEBUG == true) {
                 return ErrorHandler::showDebug($e);
             }
-            $layout = new DefaultLayout();
+            $layout = new ErrorLayout();
             return $layout->fetch(new ErrorAction($e));
         }
     }

app/router/Router.php

@@ -37,8 +37,9 @@ class Router
         
         foreach ($this->routes as $name => $route) {
             if ($route->match($req)) {
-                $this->route_name = $route;
+                $this->route_name = $name;
                 $this->route = $route;
+                Env::setParams($route->getParams());
                 return $this->route;
             }
         }

classes/Env.class.php

@@ -16,6 +16,8 @@ class Env
     
     static protected $request = null;
     
+    static protected $params = array();
+    
     static public function getRequestUri()
     {
         if (self::$request === null) {
@@ -100,4 +102,23 @@ class Env
         $res = isset($_FILES[$name]) ? $_FILES[$name] : $default;
         return $param ? $res[$param] : $res;
     }
+    
+    static public function getParam($key = null, $default = false)
+    {
+        if ($key === null) {
+            return self::$params;
+        }
+        return (isset(self::$params[$key])) ? self::$params[$key] : $default;
+    }
+
+    static public function setParam($key, $value)
+    {
+        self::$params[$key] = $value;
+    }
+
+    static public function setParams($params = array())
+    {
+        self::$params = self::$params + $params;
+    }
+    
 }

exception/ErrorHandler.php

@@ -19,6 +19,7 @@ class ErrorHandler
     
     static public function error_handler($errno, $errstr, $errfile, $errline )
     {
+        ob_clean();
         throw new ErrorException($errstr, 0, $errno, $errfile, $errline);
         return false;
     }
@@ -31,9 +32,9 @@ class ErrorHandler
             $i++;
             if($i >= $hiline - 10 && $i <= $hiline + 10) {
                 if ($i == $hiline) {
-                    $code[] = '<tr class="error"><th>' . $i . '</th><td><span class="specific">' . $line . '</span></td></tr>';
+                    $code[] = '<tr class="error"><th>' . $i . '</th><td><span class="specific">' . htmlentities($line, ENT_QUOTES, 'UTF-8') . '</span></td></tr>';
                 }else{
-                    $code[] = '<tr><th>' . $i . '</th><td>' . $line . '</td></tr>';
+                    $code[] = '<tr><th>' . $i . '</th><td>' . htmlentities($line, ENT_QUOTES, 'UTF-8') . '</td></tr>';
                 }
             }
             if($i > $hiline + 10) {
@@ -50,6 +51,7 @@ class ErrorHandler
         }
         $text = '<table class="req"><thead><tr><th>Variable</th><th>Value</th></tr></thead><tbody>';
         foreach ($array as $key => $value) {
+            $value = ($value) ? htmlentities($value, ENT_QUOTES, 'UTF-8') : '&nbsp;';
             $text .= '<tr><td>' . $key . '</td><td class="code"><div>' . $value . '</div></td></tr>';
         }
         $text .= '</tbody></table>';
@@ -139,7 +141,6 @@ class ErrorHandler
       <pre class="exception_value">{$exception->getMessage()}</pre>
       <table class="meta">
         <tbody>
-            <tbody>
             <tr><th>Request Method:</th><td>{$method}</td></tr>
             <tr><th>Exception Type:</th><td>{$class}</td></tr>
             <tr><th>Exception Message:</th><td><pre>{$exception->getMessage()}</pre></td></tr>

layout/Error.layout.php

@@ -0,0 +1,15 @@
+<?php
+/**
+ * @copyright NetMonsters <team@netmonsters.ru>
+ * @link http://netmonsters.ru
+ * @package Majestic
+ * @subpackage Layout
+ * @since 2010-02-25
+ * @version SVN: $Id$
+ * @filesource $URL$
+ */
+
+class ErrorLayout extends Layout
+{
+    protected function execute(){}
+}

view/PHPView.php

@@ -20,7 +20,7 @@ class PHPView implements iView
     public function __construct($config)
     {
         if (!isset($config['path'])) {
-            throw new Exception('Configuration must have a "host".');
+            throw new Exception('Configuration must have a "path".');
         }
         $this->setPath($config['path']);
     }