From 4942fdbfda4e2acfa094e48a3eeb6739f23aac46 Mon Sep 17 00:00:00 2001
From: Mike Koch <mkoch227@gmail.com>
Date: Sat, 14 Jan 2017 22:00:27 -0500
Subject: [PATCH] Use __DIR__ to make importing easier and only when required

---
 api/businesslogic/ValidationException.php     | 18 +++++++++++++++
 ...el.php => CreateTicketByCustomerModel.php} |  2 +-
 api/businesslogic/ticket/ticket_creator.php   | 22 ++++++++++++++-----
 3 files changed, 36 insertions(+), 6 deletions(-)
 create mode 100644 api/businesslogic/ValidationException.php
 rename api/businesslogic/ticket/{CreateTicketForCustomerModel.php => CreateTicketByCustomerModel.php} (92%)

diff --git a/api/businesslogic/ValidationException.php b/api/businesslogic/ValidationException.php
new file mode 100644
index 00000000..3f174212
--- /dev/null
+++ b/api/businesslogic/ValidationException.php
@@ -0,0 +1,18 @@
+<?php
+
+class ValidationException extends Exception {
+    public $validationModel;
+
+    /**
+     * ValidationException constructor.
+     * @param ValidationModel $validationModel The validatio model
+     * @throws Exception If the validationModel's errorKeys is empty
+     */
+    function __construct($validationModel) {
+        if (count($validationModel->errorKeys) === 0) {
+            throw new Exception('Tried to throw a ValidationException, but the validation model was valid!');
+        }
+
+        $this->validationModel = $validationModel;
+    }
+}
\ No newline at end of file
diff --git a/api/businesslogic/ticket/CreateTicketForCustomerModel.php b/api/businesslogic/ticket/CreateTicketByCustomerModel.php
similarity index 92%
rename from api/businesslogic/ticket/CreateTicketForCustomerModel.php
rename to api/businesslogic/ticket/CreateTicketByCustomerModel.php
index da4c233c..46840751 100644
--- a/api/businesslogic/ticket/CreateTicketForCustomerModel.php
+++ b/api/businesslogic/ticket/CreateTicketByCustomerModel.php
@@ -1,6 +1,6 @@
 <?php
 
-class CreateTicketForCustomerModel {
+class CreateTicketByCustomerModel {
     // Metadata
     /**
      * @var string
diff --git a/api/businesslogic/ticket/ticket_creator.php b/api/businesslogic/ticket/ticket_creator.php
index d4eb6482..b94bb0ce 100644
--- a/api/businesslogic/ticket/ticket_creator.php
+++ b/api/businesslogic/ticket/ticket_creator.php
@@ -1,19 +1,31 @@
 <?php
 
 /**
- * @param $ticket_request CreateTicketForCustomerModel
+ * @param $ticket_request CreateTicketByCustomerModel
+ * @param $hesk_settings array HESK settings
+ * @param $modsForHesk_settings array Mods for HESK settings
+ * @throws ValidationException When a required field in $ticket_request is missing
  */
-function create_ticket_for_customer($ticket_request, $hesk_settings, $modsForHesk_settings) {
-    validate($ticket_request, false, $hesk_settings, $modsForHesk_settings);
+function createTicketByCustomer($ticket_request, $hesk_settings, $modsForHesk_settings) {
+    $validationModel = validate($ticket_request, false, $hesk_settings, $modsForHesk_settings);
+
+    if (count($validationModel->errorKeys) > 0) {
+        require_once(__DIR__ . '/../ValidationException.php');
+
+        // Validation failed
+        throw new ValidationException($validationModel);
+    }
+
+    // Create the ticket
 }
 
 /**
- * @param $ticket_request CreateTicketForCustomerModel
+ * @param $ticket_request CreateTicketByCustomerModel
  * @param $staff bool
  * @return ValidationModel If errorKeys is empty, validation successful. Otherwise invalid ticket
  */
 function validate($ticket_request, $staff, $hesk_settings, $modsForHesk_settings) {
-    require_once('../email_validators.php');
+    require_once(__DIR__ . '/../email_validators.php');
     //require_once('../category/retriever.php');
     //require_once('../bans/retriever.php');