[mod] Exits changed to exceptions

This commit is contained in:
Jose Pabl Domingo Aramburo Sanchez 2018-08-15 00:38:05 -06:00
parent 3ad687f797
commit 477ac28212
2 changed files with 15 additions and 10 deletions

View File

@ -16,14 +16,13 @@ class EmployeeApplication{
$this->cryptographyService = $cryptographyService; $this->cryptographyService = $cryptographyService;
$this->pdo = $mysql; $this->pdo = $mysql;
$this->asserts = $asserts; $this->asserts = $asserts;
$this->databaseSelectQueryErrorMessage = 'There was an error inserting the record.';
} }
/** /**
* A list of the types of employee used in the system * A list of the types of employee used in the system
* *
* @return array * @return array
* @throws Exception
*/ */
function listEmployeeTypes(){ function listEmployeeTypes(){
$stmt = $this->pdo->prepare("SELECT $stmt = $this->pdo->prepare("SELECT
@ -37,7 +36,7 @@ class EmployeeApplication{
$results = $stmt->fetchAll(); $results = $stmt->fetchAll();
if(!$results){ if(!$results){
exit($this->databaseSelectQueryErrorMessage); throw new Exception("The types of employees could not be found..");
} }
$stmt = null; $stmt = null;
@ -190,6 +189,7 @@ class EmployeeApplication{
/** /**
* @param $idEmployee * @param $idEmployee
* @return Integer * @return Integer
* @throws Exception
*/ */
function getIdPersonByIdEmployee($idEmployee){ function getIdPersonByIdEmployee($idEmployee){
$this->asserts->higherThanZero($idEmployee, "idEmployee must be higher than 0"); $this->asserts->higherThanZero($idEmployee, "idEmployee must be higher than 0");
@ -206,7 +206,7 @@ class EmployeeApplication{
$stmt->execute(array(':idEmployee' => $idEmployee)); $stmt->execute(array(':idEmployee' => $idEmployee));
$results = $stmt->fetchAll(); $results = $stmt->fetchAll();
if(!$results){ if(!$results){
exit($this->databaseSelectQueryErrorMessage); throw new Exception("An error occurred while trying to find the person associated with the employee..");
} }
$stmt = null; $stmt = null;
@ -216,6 +216,7 @@ class EmployeeApplication{
/** /**
* @param $code string * @param $code string
* @return integer * @return integer
* @throws Exception
*/ */
function getIdEmployeeTypeByCode($code){ function getIdEmployeeTypeByCode($code){
$this->asserts->isNotEmpty($code, "The code can't be empty."); $this->asserts->isNotEmpty($code, "The code can't be empty.");
@ -232,7 +233,7 @@ class EmployeeApplication{
$stmt->execute(array(':code' => $code)); $stmt->execute(array(':code' => $code));
$results = $stmt->fetchAll(); $results = $stmt->fetchAll();
if(!$results){ if(!$results){
exit($this->databaseSelectQueryErrorMessage); throw new Exception("The employee could not be found.");
} }
$stmt = null; $stmt = null;
@ -242,6 +243,7 @@ class EmployeeApplication{
/** /**
* @param $code string * @param $code string
* @return integer * @return integer
* @throws Exception
*/ */
function getIdEmployeeByCode($code){ function getIdEmployeeByCode($code){
$this->asserts->isNotEmpty($code, "The code can't be empty."); $this->asserts->isNotEmpty($code, "The code can't be empty.");
@ -259,7 +261,7 @@ class EmployeeApplication{
$stmt->execute(array(':code' => $code)); $stmt->execute(array(':code' => $code));
$results = $stmt->fetchAll(); $results = $stmt->fetchAll();
if(!$results){ if(!$results){
exit($this->databaseSelectQueryErrorMessage); throw new Exception("The employee could not be found.");
} }
$stmt = null; $stmt = null;
@ -271,6 +273,7 @@ class EmployeeApplication{
* *
* @param $idEmployee * @param $idEmployee
* @return array * @return array
* @throws Exception
*/ */
function getEmployeeDataById($idEmployee){ function getEmployeeDataById($idEmployee){
$this->asserts->higherThanZero($idEmployee, "idEmployee must be higher than 0"); $this->asserts->higherThanZero($idEmployee, "idEmployee must be higher than 0");
@ -297,7 +300,7 @@ class EmployeeApplication{
$stmt->execute(array(':idEmployee' => $idEmployee)); $stmt->execute(array(':idEmployee' => $idEmployee));
$results = $stmt->fetchAll(); $results = $stmt->fetchAll();
if(!$results){ if(!$results){
exit($this->databaseSelectQueryErrorMessage); throw new Exception("The employee could not be found.");
} }
$stmt = null; $stmt = null;
@ -310,6 +313,7 @@ class EmployeeApplication{
* *
* @param $idEmployee * @param $idEmployee
* @return array * @return array
* @throws Exception
*/ */
function proxyGetEmployeeDataById($idEmployee){ function proxyGetEmployeeDataById($idEmployee){
$this->asserts->higherThanZero($idEmployee, "idEmployee must be higher than 0"); $this->asserts->higherThanZero($idEmployee, "idEmployee must be higher than 0");
@ -520,6 +524,7 @@ class EmployeeApplication{
* currently active in the system * currently active in the system
* *
* @return array * @return array
* @throws Exception
*/ */
function getIdEmployeeFromAllActiveEmployees(){ function getIdEmployeeFromAllActiveEmployees(){
$stmt = $this->pdo->prepare("SELECT $stmt = $this->pdo->prepare("SELECT
@ -533,7 +538,7 @@ class EmployeeApplication{
$results = $stmt->fetchAll(); $results = $stmt->fetchAll();
if(!$results){ if(!$results){
exit($this->databaseSelectQueryErrorMessage); throw new Exception("The employee could not be found.");
} }
$stmt = null; $stmt = null;
@ -545,6 +550,7 @@ class EmployeeApplication{
* all currently active employees * all currently active employees
* *
* @return array * @return array
* @throws Exception
*/ */
function listAllActiveEmployees(){ function listAllActiveEmployees(){
$ids = $this->getIdEmployeeFromAllActiveEmployees(); $ids = $this->getIdEmployeeFromAllActiveEmployees();
@ -571,6 +577,7 @@ class EmployeeApplication{
* *
* @param $partialName string * @param $partialName string
* @return array * @return array
* @throws Exception
*/ */
function findEmployeeByFullName($partialName){ function findEmployeeByFullName($partialName){
$fullList = $this->listAllActiveEmployees(); $fullList = $this->listAllActiveEmployees();

View File

@ -12,8 +12,6 @@ class SessionApplication{
$this->cryptographyService = $cryptographyService; $this->cryptographyService = $cryptographyService;
$this->pdo = $mysql; $this->pdo = $mysql;
$this->asserts = $asserts; $this->asserts = $asserts;
$this->databaseSelectQueryErrorMessage = 'There was an error inserting the record.';
} }
/** /**