From 9b1ee5cc90e386cae4a71d3c6bbfc73ca8f1d654 Mon Sep 17 00:00:00 2001
From: Andrey Kutejko <andy128k@gmail.com>
Date: Sat, 16 Mar 2019 15:23:54 +0100
Subject: [PATCH] rework error pages

---
 ipf/admin/app.php                             |  6 +--
 ipf/admin/controllers/components.php          |  2 +-
 ipf/admin/controllers/user.php                |  2 +-
 ipf/error/404.html                            |  3 +-
 ipf/error/404.php                             | 16 +++++--
 ipf/error/500.html                            |  3 +-
 ipf/error/500.php                             | 17 +++++--
 .../{500_debug.php => 500_debugPage.php}      | 46 ++++++++++++-------
 ipf/error/page.php                            | 35 --------------
 ipf/error/provider.php                        | 30 ++++++++++++
 ipf/http/response/notfound.php                | 10 ++--
 ipf/http/response/servererror.php             | 10 ----
 ipf/middleware/dispatch.php                   |  4 +-
 ipf/middleware/error.php                      | 20 ++++++--
 ipf/project.php                               |  8 +++-
 ipf/router.php                                | 16 ++++---
 16 files changed, 133 insertions(+), 95 deletions(-)
 rename ipf/error/{500_debug.php => 500_debugPage.php} (79%)
 delete mode 100644 ipf/error/page.php
 create mode 100644 ipf/error/provider.php
 delete mode 100644 ipf/http/response/servererror.php

diff --git a/ipf/admin/app.php b/ipf/admin/app.php
index 50ac73b..0d08d1f 100644
--- a/ipf/admin/app.php
+++ b/ipf/admin/app.php
@@ -150,7 +150,7 @@ class IPF_Admin_App extends IPF_Application
 
 class IPF_Admin_LoginRequired extends IPF_Router_Shortcut
 {
-    public function response($request)
+    public function response($container, $request)
     {
         return new IPF_HTTP_Response_Redirect(IPF_HTTP_URL::urlForView(array('IPF_Admin_User_Controller', 'login')));
     }
@@ -158,8 +158,8 @@ class IPF_Admin_LoginRequired extends IPF_Router_Shortcut
 
 class IPF_Admin_AccessDenied extends IPF_Router_Shortcut
 {
-    public function response($request)
+    public function response($container, $request)
     {
-        return new IPF_HTTP_Response_NotFound($request);
+        return new IPF_HTTP_Response_NotFound($container, $request);
     }
 }
diff --git a/ipf/admin/controllers/components.php b/ipf/admin/controllers/components.php
index 7786173..b2c4ddd 100644
--- a/ipf/admin/controllers/components.php
+++ b/ipf/admin/controllers/components.php
@@ -43,7 +43,7 @@ class IPF_Admin_Components_Controller extends IPF_Admin_Base_Controller
         $this->checkPermissions('view', 'change');
 
         if (!isset($this->request->POST['ids']) || !is_array($this->request->POST['ids']))
-            return new IPF_HTTP_Response_NotFound($this->request);
+            return new IPF_HTTP_Response_NotFound($this->container, $this->request);
 
         if ($this->component->reorder($this->request->POST['ids']))
             return new IPF_HTTP_Response_Json("Ok");
diff --git a/ipf/admin/controllers/user.php b/ipf/admin/controllers/user.php
index 4430c4d..70ebf99 100644
--- a/ipf/admin/controllers/user.php
+++ b/ipf/admin/controllers/user.php
@@ -57,7 +57,7 @@ class IPF_Admin_User_Controller extends IPF_Admin_Base_Controller
         if (!$this->request->user->isAnonymous() && $this->request->user->is_superuser) {
             $user = $this->authApp()->findUser($this->params[1]);
             if (!$user)
-                return new IPF_HTTP_Response_NotFound($this->request);
+                return new IPF_HTTP_Response_NotFound($this->container, $this->request);
             $this->authApp()->login($this->request, $user);
         }
 
diff --git a/ipf/error/404.html b/ipf/error/404.html
index c62e3a5..4c449d1 100644
--- a/ipf/error/404.html
+++ b/ipf/error/404.html
@@ -2,5 +2,4 @@
 <h1>Error 404</h1>
 <h2>Page not found</h2>
 <p>Sorry, it appears the page you were looking for does not exist anymore or might have been moved.</p>
-<p>Go to <a href="<?php echo IPF::get('app_base') ?>/">home page</a> or go back to <a href="#" onclick="javascript:history.back();">previous page</a>.</p>
-
+<p>Go to <a href="<?php echo $this->app_base ?>/">home page</a> or go back to <a href="#" onclick="javascript:history.back();">previous page</a>.</p>
diff --git a/ipf/error/404.php b/ipf/error/404.php
index 0f88bca..08889ba 100644
--- a/ipf/error/404.php
+++ b/ipf/error/404.php
@@ -1,10 +1,18 @@
 <?php
 
-class IPF_Not_Found_Error_Page extends IPF_Error_Page
+namespace IPF\Error;
+
+use IPF_Settings;
+use PFF\NativeTemplate;
+
+class NotFoundPage extends NativeTemplate
 {
-    protected function templateFile()
+    public static function renderPage(IPF_Settings $settings, $request)
     {
-        return '404.html';
+        $page = new NativeTemplate(array(dirname(__FILE__)));
+        return $page->render('404.html', array(
+            'request' => $request,
+            'app_base' => $settings->get('app_base'),
+        ));
     }
 }
-
diff --git a/ipf/error/500.html b/ipf/error/500.html
index 509e9ea..bcea3a1 100644
--- a/ipf/error/500.html
+++ b/ipf/error/500.html
@@ -2,5 +2,4 @@
 <h1>Error 500</h1>
 <h2>Server error</h2>
 <p>Our apologies&hellip;</p>
-<p>Go to <a href="<?php echo IPF::get('app_base') ?>/">home page</a> or go back to <a href="#" onclick="javascript:history.back();">previous page</a>.</p>
-
+<p>Go to <a href="<?php echo $this->app_base ?>/">home page</a> or go back to <a href="#" onclick="javascript:history.back();">previous page</a>.</p>
diff --git a/ipf/error/500.php b/ipf/error/500.php
index c2e3f95..f13d6b7 100644
--- a/ipf/error/500.php
+++ b/ipf/error/500.php
@@ -1,10 +1,19 @@
 <?php
 
-class IPF_Server_Error_Page extends IPF_Error_Page
+namespace IPF\Error;
+
+use IPF_Settings;
+use PFF\NativeTemplate;
+
+class ServerErrorPage extends NativeTemplate
 {
-    protected function templateFile()
+    public static function renderPage(IPF_Settings $settings, $request, $exception = null)
     {
-        return '500.html';
+        $page = new static(array(dirname(__FILE__)));
+        return $page->render('500.html', array(
+            'request' => $request,
+            'exception' => $exception,
+            'app_base' => $settings->get('app_base'),
+        ));
     }
 }
-
diff --git a/ipf/error/500_debug.php b/ipf/error/500_debugPage.php
similarity index 79%
rename from ipf/error/500_debug.php
rename to ipf/error/500_debugPage.php
index 8a763bf..2b13fde 100644
--- a/ipf/error/500_debug.php
+++ b/ipf/error/500_debugPage.php
@@ -1,13 +1,26 @@
 <?php
 
-class IPF_Server_Error_Page_Debug extends IPF_Error_Page
+namespace IPF\Error;
+
+use Exception;
+use PFF\NativeTemplate;
+use ReflectionException;
+use ReflectionFunction;
+use ReflectionMethod;
+use stdClass;
+
+class ServerErrorDebugPage extends NativeTemplate
 {
-    protected function templateFile()
+    public static function renderPage($request, $exception)
     {
-        return '500_debug.html';
+        $page = new static(array(dirname(__FILE__)));
+        return $page->render('500_debug.html', array(
+            'request' => $request,
+            'exception' => $exception,
+        ));
     }
 
-    function debug_print_r($v, $indent=0)
+    function debug_print_r($v, $indent = 0)
     {
         $result = array();
 
@@ -16,22 +29,22 @@ class IPF_Server_Error_Page_Debug extends IPF_Error_Page
             $result[] = get_class($v) . ' Object';
             $result[] = '{';
             foreach ($props as $key => $val) {
-                $result[] = '    [' . $key . '] => ' . $this->debug_print_r($val, $indent+2);
+                $result[] = '    [' . $key . '] => ' . $this->debug_print_r($val, $indent + 2);
             }
             $result[] = '}';
         } elseif ($v instanceof stdClass) {
             $result[] = 'stdClass';
             $result[] = '{';
-            foreach ($v as $k => $v) {
-                $result[] = '    ' . $k . ' => ' . $this->debug_print_r($v, $indent+2);
+            foreach ($v as $k => $v1) {
+                $result[] = '    ' . $k . ' => ' . $this->debug_print_r($v1, $indent + 2);
             }
             $result[] = '}';
         } elseif (is_array($v)) {
             if ($v) {
                 $result[] = 'Array';
                 $result[] = '{';
-                foreach ($v as $k => $v) {
-                    $result[] = '    [' . $k . '] => ' . $this->debug_print_r($v, $indent+2);
+                foreach ($v as $k => $v1) {
+                    $result[] = '    [' . $k . '] => ' . $this->debug_print_r($v1, $indent + 2);
                 }
                 $result[] = '}';
             } else {
@@ -64,17 +77,17 @@ class IPF_Server_Error_Page_Debug extends IPF_Error_Page
         if ($values) {
             ?><h3><?php echo $name ?></h3>
             <table class="values">
-              <?php foreach ($values as $name => $value): ?>
+            <?php foreach ($values as $name => $value): ?>
                 <tr>
-                  <th><?php $this->e($name) ?></th>
-                  <td><?php $this->e($this->debug_print_r($value)) ?></td>
+                    <th><?php $this->e($name) ?></th>
+                    <td><?php $this->e($this->debug_print_r($value)) ?></td>
                 </tr>
-              <?php endforeach; ?>
+            <?php endforeach; ?>
             </table><?php
         }
     }
 
-    function frames($exception)
+    function frames(Exception $exception)
     {
         $result = array();
 
@@ -85,7 +98,7 @@ class IPF_Server_Error_Page_Debug extends IPF_Error_Page
             if (isset($frame["function"])) {
                 try {
                     if (isset($frame["class"])) {
-                        $r = new ReflectionMethod($frame["class"]."::".$frame["function"]);
+                        $r = new ReflectionMethod($frame["class"] . "::" . $frame["function"]);
                     } else {
                         $r = new ReflectionFunction($frame["function"]);
                     }
@@ -110,7 +123,7 @@ class IPF_Server_Error_Page_Debug extends IPF_Error_Page
                 'class' => @$frame['class'],
                 'type' => @$frame['type'],
                 'function' => @$frame['function'],
-                'func' => @$frame['class'].@$frame['type'].@$frame['function'],
+                'func' => @$frame['class'] . @$frame['type'] . @$frame['function'],
                 'args' => $args,
             );
 
@@ -149,4 +162,3 @@ class IPF_Server_Error_Page_Debug extends IPF_Error_Page
         return $ol;
     }
 }
-
diff --git a/ipf/error/page.php b/ipf/error/page.php
deleted file mode 100644
index 3e9f0dd..0000000
--- a/ipf/error/page.php
+++ /dev/null
@@ -1,35 +0,0 @@
-<?php
-
-abstract class IPF_Error_Page extends \PFF\NativeTemplate
-{
-    public static function render404($request)
-    {
-        if (IPF::get('render404')) {
-            $func = IPF::get('render404');
-            return IPF::callFunction($func, array($request));
-        } else {
-            return IPF_Not_Found_Error_Page::renderPage($request);
-        }
-    }
-
-    public static function render500($request, $exception)
-    {
-        if (IPF::get('debug')) {
-            return IPF_Server_Error_Page_Debug::renderPage($request, $exception);
-        } elseif (IPF::get('render500')) {
-            $func = IPF::get('render500');
-            return IPF::callFunction($func, array($request, $exception));
-        } else {
-            return IPF_Server_Error_Page::renderPage($request, $exception);
-        }
-    }
-
-    protected static function renderPage($request, $exception=null)
-    {
-        $page = new static(array(dirname(__FILE__)));
-        return $page->render($page->templateFile(), array('request' => $request, 'exception' => $exception));
-    }
-
-    abstract protected function templateFile();
-}
-
diff --git a/ipf/error/provider.php b/ipf/error/provider.php
new file mode 100644
index 0000000..959d257
--- /dev/null
+++ b/ipf/error/provider.php
@@ -0,0 +1,30 @@
+<?php
+
+namespace IPF\Error;
+
+use Pimple\Container;
+use Pimple\ServiceProviderInterface;
+
+class ErrorPageProvider implements ServiceProviderInterface
+{
+    public function register(Container $container)
+    {
+        $container['render404'] = function ($c) {
+            $settings = $c['settings'];
+            return function ($request) use ($settings) {
+                return NotFoundPage::renderPage($settings, $request);
+            };
+        };
+        $container['render500'] = function ($c) {
+            $settings = $c['settings'];
+            return function ($request, $exception) use ($settings) {
+                $isDebug = $settings->get('debug');
+                if ($isDebug) {
+                    return ServerErrorDebugPage::renderPage($request, $exception);
+                } else {
+                    return ServerErrorPage::renderPage($settings, $request, $exception);
+                }
+            };
+        };
+    }
+}
diff --git a/ipf/http/response/notfound.php b/ipf/http/response/notfound.php
index 2a551aa..4dfa439 100644
--- a/ipf/http/response/notfound.php
+++ b/ipf/http/response/notfound.php
@@ -1,11 +1,15 @@
 <?php
 
+use Pimple\Container;
+
 class IPF_HTTP_Response_NotFound extends IPF_HTTP_Response
 {
-    function __construct($request=null)
+    function __construct(Container $container, IPF_HTTP_Request $request)
     {
-        parent::__construct(IPF_Error_Page::render404($request));
+        $render404 = $container['render404'];
+        $content = call_user_func($render404, $request);
+
+        parent::__construct($content);
         $this->status_code = 404;
     }
 }
-
diff --git a/ipf/http/response/servererror.php b/ipf/http/response/servererror.php
deleted file mode 100644
index 9b4e86d..0000000
--- a/ipf/http/response/servererror.php
+++ /dev/null
@@ -1,10 +0,0 @@
-<?php
-
-class IPF_HTTP_Response_ServerError extends IPF_HTTP_Response
-{
-    function __construct($request, $exception, $mimetype=null)
-    {
-        parent::__construct(IPF_Error_Page::render500($request, $exception), $mimetype);
-        $this->status_code = 500;
-    }
-}
diff --git a/ipf/middleware/dispatch.php b/ipf/middleware/dispatch.php
index 0d8e1ab..007617b 100644
--- a/ipf/middleware/dispatch.php
+++ b/ipf/middleware/dispatch.php
@@ -6,7 +6,7 @@ class IPF_Dispatch_Middleware extends IPF_Middleware
     {
         $m = $this->getRouter()->match($request);
         if (!$m) {
-            return new IPF_HTTP_Response_NotFound($request);
+            return new IPF_HTTP_Response_NotFound($this->container, $request);
         }
 
         list($route, $match) = $m;
@@ -18,7 +18,7 @@ class IPF_Dispatch_Middleware extends IPF_Middleware
             $controller = new $controllerClass($this->container);
             return $controller->process($route->action(), $request, $match);
         } catch (IPF_Router_Shortcut $e) {
-            return $e->response($request);
+            return $e->response($this->container, $request);
         }
     }
 
diff --git a/ipf/middleware/error.php b/ipf/middleware/error.php
index 01b3832..7cec390 100644
--- a/ipf/middleware/error.php
+++ b/ipf/middleware/error.php
@@ -18,7 +18,7 @@ class IPF_Error_Middleware extends IPF_Middleware
                 throw new \Exception('Response is not a IPF_HTTP_Response');
         } catch (\Exception $exception) {
             error_log($exception);
-            $response = new IPF_HTTP_Response_ServerError($request, $exception);
+            $response = $this->getServerErrorResponse($request, $exception);
         }
         if ($this->previous_error_handler)
             set_error_handler($this->previous_error_handler);
@@ -41,9 +41,23 @@ class IPF_Error_Middleware extends IPF_Middleware
         $error = error_get_last();
         if ($error) {
             $exception = new \ErrorException($error['message'], 0, $error['type'], $error['file'], $error['line']);
-            $response = new IPF_HTTP_Response_ServerError($request, $exception);
+            $response = $this->getServerErrorResponse($request, $exception);
             $response->render();
         }
     }
-}
 
+    /**
+     * @param IPF_HTTP_Request $request
+     * @param Exception $exception
+     * @return IPF_HTTP_Response
+     */
+    private function getServerErrorResponse(IPF_HTTP_Request $request, Exception $exception)
+    {
+        $render500 = $this->container['render500'];
+        $content = call_user_func($render500, $request, $exception);
+
+        $response = new IPF_HTTP_Response($content, null);
+        $response->status_code = 500;
+        return $response;
+    }
+}
diff --git a/ipf/project.php b/ipf/project.php
index ddad1f6..d4b9b6f 100644
--- a/ipf/project.php
+++ b/ipf/project.php
@@ -1,6 +1,7 @@
 <?php
 
 use IPF\CoreServicesProvider;
+use IPF\Error\ErrorPageProvider;
 use IPF\Template\TemplateProvider;
 use Pimple\Container;
 
@@ -38,7 +39,11 @@ final class IPF_Project
         }
         $this->container['apps'] = $apps;
         $this->container['router'] = function ($c) {
-            return new IPF_Router;
+            $settings = $c['settings'];
+            return new IPF_Router(
+                $settings->get('urls'),
+                $settings->get('app_base')
+            );
         };
         $this->router = $this->container['router'];
     }
@@ -72,6 +77,7 @@ final class IPF_Project
     {
         $this->container->register(new CoreServicesProvider());
         $this->container->register(new TemplateProvider());
+        $this->container->register(new ErrorPageProvider());
 
         $this->container['databaseConnection'] = function ($c) {
             return IPF_Database::connect();
diff --git a/ipf/router.php b/ipf/router.php
index 0572b6a..02f13ca 100644
--- a/ipf/router.php
+++ b/ipf/router.php
@@ -4,10 +4,13 @@ class IPF_Router
 {
     /** @var IPF_Route[] */
     private $routes = array();
+    /** @var string */
+    private $app_base;
 
-    public function __construct()
+    public function __construct(array $urls, $app_base)
     {
-        $this->flattenRoutes(IPF::get('urls'));
+        $this->app_base = $app_base;
+        $this->flattenRoutes($urls);
     }
 
     private function flattenRoutes($routes, $prefix='', $common=array())
@@ -62,7 +65,7 @@ class IPF_Router
     {
         foreach ($this->routes as $route)
             if ($route->routesTo($view))
-                return IPF::get('app_base') . $route->matcher()->reverse($params);
+                return $this->app_base . $route->matcher()->reverse($params);
         throw new IPF_Exception('Error, the view: '.print_r($view, true).' has not been found.');
     }
 }
@@ -187,14 +190,13 @@ class IPF_Route
 
 abstract class IPF_Router_Shortcut extends Exception
 {
-    public abstract function response($request);
+    public abstract function response($container, $request);
 }
 
 class IPF_HTTP_Error404 extends IPF_Router_Shortcut
 {
-    public function response($request)
+    public function response($container, $request)
     {
-        return new IPF_HTTP_Response_NotFound($request);
+        return new IPF_HTTP_Response_NotFound($container, $request);
     }
 }
-
-- 
2.49.0