From fc38673b7dd7def573521b0085bf3807ea5a1a51 Mon Sep 17 00:00:00 2001
From: Mike Koch <mkoch227@gmail.com>
Date: Wed, 25 Jan 2017 21:55:13 -0500
Subject: [PATCH] Some temporary error handling

---
 api/core/json_error.php |  4 +++-
 api/index.php           | 20 +++++++++++++++++++-
 2 files changed, 22 insertions(+), 2 deletions(-)

diff --git a/api/core/json_error.php b/api/core/json_error.php
index 61b491e6..2280d511 100644
--- a/api/core/json_error.php
+++ b/api/core/json_error.php
@@ -1,11 +1,13 @@
 <?php
 
 function print_error($title, $message) {
+    require_once(__DIR__ . '/output.php');
+
     $error = array();
     $error['type'] = 'ERROR';
     $error['title'] = $title;
     $error['message'] = $message;
 
-    print json_encode($error);
+    print output($error);
     return;
 }
\ No newline at end of file
diff --git a/api/index.php b/api/index.php
index b5f8c6e6..20ae1aa1 100644
--- a/api/index.php
+++ b/api/index.php
@@ -10,13 +10,31 @@ require(__DIR__ . '/../hesk_settings.inc.php');
 require(__DIR__ . '/controllers/CategoryController.php');
 hesk_load_api_database_functions();
 
+// Properly handle error logging, as well as a fatal error workaround
+error_reporting(0); // Override hesk_settings. We're smarter than it
+set_error_handler('errorHandler');
+register_shutdown_function('fatalErrorShutdownHandler');
+
 function handle404() {
     http_response_code(404);
     print json_encode('404 found');
 }
 
 function assertApiIsEnabled() {
-    //-- TODO
+    throw new Exception("Some exception here!", 33);
+}
+
+function errorHandler($errorNumber, $errorMessage, $errorFile, $errorLine) {
+    print_error(sprintf("Uncaught error in %s", $errorFile), $errorMessage);
+    die();
+}
+
+function fatalErrorShutdownHandler() {
+    $last_error = error_get_last();
+    if ($last_error['type'] === E_ERROR) {
+        // fatal error
+        errorHandler(E_ERROR, $last_error['message'], $last_error['file'], $last_error['line']);
+    }
 }
 
 // Must use fully-qualified namespace to controllers