SherifElfadaly / Laravel-Api-Skeleton

src/Modules/Acl/Repositories/UserRepository.php

Doc Comments +3 added lines, -3 removed lines

@@ -61,7 +61,7 @@ 
 	/**
 	 * Check if the logged in user has the given group.
 	 * 
-	 * @param  array $groups
+	 * @param  string[] $groups
 	 * @param  mixed $user
 	 * @return boolean
 	 */
@@ -155,7 +155,7 @@ 
 	 * 
 	 * @param  array   $credentials
 	 * @param  boolean $skipConfirmEmail
-	 * @return array
+	 * @return boolean
 	 */
 	public function register($credentials, $skipConfirmEmail = false)
 	{
@@ -246,7 +246,7 @@ 
 	 * Reset the given user's password.
 	 *
 	 * @param  array  $credentials
-	 * @return array
+	 * @return string|null
 	 */
 	public function resetPassword($credentials)
 	{

Spacing +4 added lines, -4 removed lines

@@ -28,7 +28,7 @@ 
 		$user        = \Core::users()->find(\Auth::id(), $relations);
 		foreach ($user->groups()->get() as $group)
 		{
-			$group->permissions->each(function ($permission) use (&$permissions){
+			$group->permissions->each(function($permission) use (&$permissions){
 				$permissions[$permission->model][$permission->id] = $permission->name;
 			});
 		}
@@ -51,7 +51,7 @@ 
 		$user        = $user ?: $this->find(\Auth::id(), ['groups.permissions']);
 		$permissions = [];
 
-		$user->groups->pluck('permissions')->each(function ($permission) use (&$permissions, $model){
+		$user->groups->pluck('permissions')->each(function($permission) use (&$permissions, $model){
 			$permissions = array_merge($permissions, $permission->where('model', $model)->pluck('name')->toArray()); 
 		});
         
@@ -80,7 +80,7 @@ 
 	 */
 	public function assignGroups($userId, $groupIds)
 	{
-		\DB::transaction(function () use ($userId, $groupIds) {
+		\DB::transaction(function() use ($userId, $groupIds) {
 			$user = $this->find($userId);
 			$user->groups()->detach();
 			$user->groups()->attach($groupIds);
@@ -250,7 +250,7 @@ 
 	 */
 	public function resetPassword($credentials)
 	{
-		$response = \Password::reset($credentials, function ($user, $password) {
+		$response = \Password::reset($credentials, function($user, $password) {
 			$user->password = $password;
 			$user->save();
 		});

Braces +7 added lines, -14 removed lines

@@ -102,20 +102,16 @@ 
 		if ( ! $user = $this->first(['email' => $credentials['email']])) 
 		{
 			\ErrorHandler::loginFailed();
-		}
-		else if ($adminLogin && ! $user->groups->whereIn('name', ['Admin'])->count()) 
+		} else if ($adminLogin && ! $user->groups->whereIn('name', ['Admin'])->count()) 
 		{
 			\ErrorHandler::loginFailed();
-		}
-		else if ( ! $adminLogin && $user->groups->whereIn('name', ['Admin'])->count()) 
+		} else if ( ! $adminLogin && $user->groups->whereIn('name', ['Admin'])->count()) 
 		{
 			\ErrorHandler::loginFailed();
-		}
-		else if ($user->blocked)
+		} else if ($user->blocked)
 		{
 			\ErrorHandler::userIsBlocked();
-		}
-		else if ( ! config('skeleton.disable_confirm_email') && ! $user->confirmed)
+		} else if ( ! config('skeleton.disable_confirm_email') && ! $user->confirmed)
 		{
 			\ErrorHandler::emailNotConfirmed();
 		}
@@ -165,8 +161,7 @@ 
 		{
 			$user->confirmed = 1;
 			$user->save();
-		}
-		else if ( ! config('skeleton.disable_confirm_email'))  
+		} else if ( ! config('skeleton.disable_confirm_email'))  
 		{
 			$this->sendConfirmationEmail($user->email);
 		}
@@ -189,12 +184,10 @@ 
 		if ( ! $this->hasGroup(['Admin']))
 		{
 			\ErrorHandler::noPermissions();
-		}
-		else if (\Auth::id() == $userId)
+		} else if (\Auth::id() == $userId)
 		{
 			\ErrorHandler::noPermissions();
-		}
-		else if ($user->groups->pluck('name')->search('Admin', true) !== false) 
+		} else if ($user->groups->pluck('name')->search('Admin', true) !== false) 
 		{
 			\ErrorHandler::noPermissions();
 		}

src/Modules/Core/AbstractRepositories/AbstractRepository.php

Doc Comments +8 added lines, -8 removed lines

@@ -32,7 +32,7 @@ 
 	 *
 	 * @param  array   $relations
 	 * @param  string  $sortBy
-	 * @param  boolean $desc
+	 * @param  integer $desc
 	 * @param  array   $columns
 	 * @return collection
 	 */
@@ -50,7 +50,7 @@ 
 	 * @param  integer $perPage
 	 * @param  array   $relations
 	 * @param  string  $sortBy
-	 * @param  boolean $desc
+	 * @param  integer $desc
 	 * @param  array   $columns
 	 * @return collection
 	 */
@@ -164,7 +164,7 @@ 
 	 * @param  integer $perPage
 	 * @param  array   $relations
 	 * @param  string  $sortBy
-	 * @param  boolean $desc
+	 * @param  integer $desc
 	 * @param  array   $columns
 	 * @return collection
 	 */
@@ -182,7 +182,7 @@ 
 	 * @param  integer $perPage
 	 * @param  array   $relations
 	 * @param  string  $sortBy
-	 * @param  boolean $desc
+	 * @param  integer $desc
 	 * @param  array   $columns
 	 * @return collection
 	 */
@@ -198,7 +198,7 @@ 
 	 * Save the given model to the storage.
 	 * 
 	 * @param  array $data
-	 * @return mixed
+	 * @return boolean
 	 */
 	public function save(array $data)
 	{
@@ -517,7 +517,7 @@ 
 	 * id.
 	 * 
 	 * @param  integer $id
-	 * @param  array   $relations
+	 * @param  string[]   $relations
 	 * @param  array   $columns
 	 * @return object
 	 */
@@ -533,7 +533,7 @@ 
 	 * @param  array   $conditions array of conditions
 	 * @param  array   $relations
 	 * @param  string  $sortBy
-	 * @param  boolean $desc
+	 * @param  integer $desc
 	 * @param  array   $columns
 	 * @return collection
 	 */
@@ -565,7 +565,7 @@ 
 	 * @param  array   $conditions array of conditions
 	 * @param  integer $perPage
 	 * @param  string  $sortBy
-	 * @param  boolean $desc
+	 * @param  integer $desc
 	 * @param  array   $columns
 	 * @return collection
 	 */

Spacing +36 added lines, -36 removed lines

@@ -63,11 +63,11 @@ 
 		/**
 		 * Construct the select conditions for the model.
 		 */
-		$model->where(function ($q) use ($query, $conditionColumns, $relations){
+		$model->where(function($q) use ($query, $conditionColumns, $relations){
 
 			if (count($conditionColumns)) 
 			{
-				$column = 'LOWER(' . array_shift($conditionColumns) . ')';
+				$column = 'LOWER('.array_shift($conditionColumns).')';
 				if (str_contains($column, '->')) 
 				{
 					$column = $this->wrapJsonSelector($column);
@@ -76,7 +76,7 @@ 
 				/**
 				 * Use the first element in the model columns to construct the first condition.
 				 */
-				$q->where(\DB::raw($column), 'LIKE', '%' . strtolower($query) . '%');
+				$q->where(\DB::raw($column), 'LIKE', '%'.strtolower($query).'%');
 			}
 
 			/**
@@ -84,13 +84,13 @@ 
 			 */
 			foreach ($conditionColumns as $column) 
 			{
-				$column = 'LOWER(' . $column . ')';
+				$column = 'LOWER('.$column.')';
 				if (str_contains($column, '->')) 
 				{
 					$column = $this->wrapJsonSelector($column);
 				}
 
-				$q->orWhere(\DB::raw($column), 'LIKE', '%' . strtolower($query) . '%');
+				$q->orWhere(\DB::raw($column), 'LIKE', '%'.strtolower($query).'%');
 			}
 
 			/**
@@ -111,9 +111,9 @@ 
 					/**
 					 * Construct the relation condition.
 					 */
-					$q->orWhereHas($relation, function ($subModel) use ($query, $relation){
+					$q->orWhereHas($relation, function($subModel) use ($query, $relation){
 
-						$subModel->where(function ($q) use ($query, $relation){
+						$subModel->where(function($q) use ($query, $relation){
 
 							/**
 							 * Get columns of the relation.
@@ -123,7 +123,7 @@ 
 							if (count($subConditionColumns)) 
 							{
 
-								$column = 'LOWER(' . array_shift($subConditionColumns) . ')';
+								$column = 'LOWER('.array_shift($subConditionColumns).')';
 								if (str_contains($column, '->')) 
 								{
 									$column = $this->wrapJsonSelector($column);
@@ -132,7 +132,7 @@ 
 								/**
 								 * Use the first element in the relation model columns to construct the first condition.
 								 */
-								$q->where(\DB::raw($column), 'LIKE', '%' . strtolower($query) . '%');
+								$q->where(\DB::raw($column), 'LIKE', '%'.strtolower($query).'%');
 							}
 
 							/**
@@ -140,13 +140,13 @@ 
 							 */
 							foreach ($subConditionColumns as $subConditionColumn)
 							{
-								$column = 'LOWER(' . $subConditionColumn . ')';
+								$column = 'LOWER('.$subConditionColumn.')';
 								if (str_contains($column, '->')) 
 								{
 									$column = $this->wrapJsonSelector($column);
 								}
                                 
-								$q->orWhere(\DB::raw($column), 'LIKE', '%' . strtolower($query) . '%');
+								$q->orWhere(\DB::raw($column), 'LIKE', '%'.strtolower($query).'%');
 							} 
 						});
 
@@ -207,7 +207,7 @@ 
 		$modelClass = $this->model;
 		$relations  = [];
 
-		\DB::transaction(function () use (&$model, &$relations, $data, $modelClass) {
+		\DB::transaction(function() use (&$model, &$relations, $data, $modelClass) {
 			/**
 			 * If the id is present in the data then select the model for updating,
 			 * else create new model.
@@ -216,7 +216,7 @@ 
 			$model = array_key_exists('id', $data) ? $modelClass->lockForUpdate()->find($data['id']) : new $modelClass;
 			if ( ! $model) 
 			{
-				\ErrorHandler::notFound(class_basename($modelClass) . ' with id : ' . $data['id']);
+				\ErrorHandler::notFound(class_basename($modelClass).' with id : '.$data['id']);
 			}
 
 			/**
@@ -274,7 +274,7 @@ 
 								 */
 								if ( ! $relationModel) 
 								{
-									\ErrorHandler::notFound(class_basename($relationBaseModel) . ' with id : ' . $val['id']);
+									\ErrorHandler::notFound(class_basename($relationBaseModel).' with id : '.$val['id']);
 								}
 
 								/**
@@ -285,7 +285,7 @@ 
 									/**
 									 * Prevent the sub relations or attributes not in the fillable.
 									 */
-									if (gettype($val) !== 'object' && gettype($val) !== 'array' &&  array_search($attr, $relationModel->getFillable(), true) !== false)
+									if (gettype($val) !== 'object' && gettype($val) !== 'array' && array_search($attr, $relationModel->getFillable(), true) !== false)
 									{
 										$relationModel->$attr = $val;
 									}
@@ -315,7 +315,7 @@ 
 									 */
 									if ( ! $relationModel) 
 									{
-										\ErrorHandler::notFound(class_basename($relationBaseModel) . ' with id : ' . $value['id']);
+										\ErrorHandler::notFound(class_basename($relationBaseModel).' with id : '.$value['id']);
 									}
 
 									foreach ($value as $relationAttribute => $relationValue) 
@@ -474,7 +474,7 @@ 
 		}
 		else
 		{
-			call_user_func_array("{$this->getModel()}::where", array($attribute, '=', $value))->lockForUpdate()->get()->each(function ($model) use ($data){
+			call_user_func_array("{$this->getModel()}::where", array($attribute, '=', $value))->lockForUpdate()->get()->each(function($model) use ($data){
 				$model->update($data);
 			});
 		}
@@ -492,11 +492,11 @@ 
 	{
 		if ($attribute == 'id') 
 		{
-			\DB::transaction(function () use ($value, $attribute, &$result) {
+			\DB::transaction(function() use ($value, $attribute, &$result) {
 				$model = $this->model->lockForUpdate()->find($value);
 				if ( ! $model) 
 				{
-					\ErrorHandler::notFound(class_basename($this->model) . ' with id : ' . $value);
+					\ErrorHandler::notFound(class_basename($this->model).' with id : '.$value);
 				}
                 
 				$model->delete();
@@ -504,8 +504,8 @@ 
 		}
 		else
 		{
-			\DB::transaction(function () use ($value, $attribute, &$result) {
-				call_user_func_array("{$this->getModel()}::where", array($attribute, '=', $value))->lockForUpdate()->get()->each(function ($model){
+			\DB::transaction(function() use ($value, $attribute, &$result) {
+				call_user_func_array("{$this->getModel()}::where", array($attribute, '=', $value))->lockForUpdate()->get()->each(function($model) {
 					$model->delete();
 				});
 			});   
@@ -541,7 +541,7 @@ 
 	{
 		$conditions = $this->constructConditions($conditions, $this->model);
 		$sort       = $desc ? 'desc' : 'asc';
-		return call_user_func_array("{$this->getModel()}::with",  array($relations))->whereRaw($conditions['conditionString'], $conditions['conditionValues'])->orderBy($sortBy, $sort)->get($columns);
+		return call_user_func_array("{$this->getModel()}::with", array($relations))->whereRaw($conditions['conditionString'], $conditions['conditionValues'])->orderBy($sortBy, $sort)->get($columns);
 	}
 
 	/**
@@ -581,7 +581,7 @@ 
 			$model->whereRaw($conditions['conditionString'], $conditions['conditionValues']);
 		}
 
-		return $model->orderBy($sortBy, $sort)->paginate($perPage, $columns);;
+		return $model->orderBy($sortBy, $sort)->paginate($perPage, $columns); ;
 	}
 
 	/**
@@ -596,7 +596,7 @@ 
 
 		if ( ! $model) 
 		{
-			\ErrorHandler::notFound(class_basename($this->model) . ' with id : ' . $id);
+			\ErrorHandler::notFound(class_basename($this->model).' with id : '.$id);
 		}
 
 		$model->restore();
@@ -621,13 +621,13 @@ 
 			if ($key == 'and') 
 			{
 				$conditions       = $this->constructConditions($value, $model);
-				$conditionString .= str_replace('{op}', 'and', $conditions['conditionString']) . ' {op} ';
+				$conditionString .= str_replace('{op}', 'and', $conditions['conditionString']).' {op} ';
 				$conditionValues  = array_merge($conditionValues, $conditions['conditionValues']);
 			}
 			else if ($key == 'or')
 			{
 				$conditions       = $this->constructConditions($value, $model);
-				$conditionString .= str_replace('{op}', 'or', $conditions['conditionString']) . ' {op} ';
+				$conditionString .= str_replace('{op}', 'or', $conditions['conditionString']).' {op} ';
 				$conditionValues  = array_merge($conditionValues, $conditions['conditionValues']);
 			}
 			else
@@ -652,41 +652,41 @@ 
                 
 				if (strtolower($operator) == 'between') 
 				{
-					$conditionString  .= $key . ' >= ? and ';
+					$conditionString  .= $key.' >= ? and ';
 					$conditionValues[] = $value1;
 
-					$conditionString  .= $key . ' <= ? {op} ';
+					$conditionString  .= $key.' <= ? {op} ';
 					$conditionValues[] = $value2;
 				}
 				elseif (strtolower($operator) == 'in') 
 				{
 					$conditionValues  = array_merge($conditionValues, $value);
 					$inBindingsString = rtrim(str_repeat('?,', count($value)), ',');
-					$conditionString .= $key . ' in (' . rtrim($inBindingsString, ',') . ') {op} ';
+					$conditionString .= $key.' in ('.rtrim($inBindingsString, ',').') {op} ';
 				}
 				elseif (strtolower($operator) == 'null') 
 				{
-					$conditionString .= $key . ' is null {op} ';
+					$conditionString .= $key.' is null {op} ';
 				}
 				elseif (strtolower($operator) == 'not null') 
 				{
-					$conditionString .= $key . ' is not null {op} ';
+					$conditionString .= $key.' is not null {op} ';
 				}
 				elseif (strtolower($operator) == 'has') 
 				{
 					$sql              = $model->withTrashed()->has($key)->toSql();
 					$conditions       = $this->constructConditions($value, $model->$key()->getRelated());
-					$conditionString .= rtrim(substr($sql, strpos($sql, 'exists')), ')') . ' and ' . $conditions['conditionString'] . ') {op} ';
+					$conditionString .= rtrim(substr($sql, strpos($sql, 'exists')), ')').' and '.$conditions['conditionString'].') {op} ';
 					$conditionValues  = array_merge($conditionValues, $conditions['conditionValues']);
 				}
 				else
 				{
-					$conditionString  .= $key . ' ' . $operator . ' ? {op} ';
+					$conditionString  .= $key.' '.$operator.' ? {op} ';
 					$conditionValues[] = $value;
 				}
 			}
 		}
-		$conditionString = '(' . rtrim($conditionString, '{op} ') . ')';
+		$conditionString = '('.rtrim($conditionString, '{op} ').')';
 		return ['conditionString' => $conditionString, 'conditionValues' => $conditionValues];
 	}
 
@@ -702,11 +702,11 @@ 
 		$value      = $removeLast === false ? $value : substr($value, 0, $removeLast);
 		$path       = explode('->', $value);
 		$field      = array_shift($path);
-		$result     = sprintf('%s->\'$.%s\'', $field, collect($path)->map(function ($part) {
+		$result     = sprintf('%s->\'$.%s\'', $field, collect($path)->map(function($part) {
 			return '"'.$part.'"';
 		})->implode('.'));
         
-		return $removeLast === false ? $result : $result . ')';
+		return $removeLast === false ? $result : $result.')';
 	}
 
 	/**

Braces +11 added lines, -22 removed lines

@@ -471,8 +471,7 @@ 
 		{
 			$model = $this->model->lockForUpdate()->find($value);
 			$model ? $model->update($data) : 0;
-		}
-		else
+		} else
 		{
 			call_user_func_array("{$this->getModel()}::where", array($attribute, '=', $value))->lockForUpdate()->get()->each(function ($model) use ($data){
 				$model->update($data);
@@ -501,8 +500,7 @@ 
                 
 				$model->delete();
 			});
-		}
-		else
+		} else
 		{
 			\DB::transaction(function () use ($value, $attribute, &$result) {
 				call_user_func_array("{$this->getModel()}::where", array($attribute, '=', $value))->lockForUpdate()->get()->each(function ($model){
@@ -623,14 +621,12 @@ 
 				$conditions       = $this->constructConditions($value, $model);
 				$conditionString .= str_replace('{op}', 'and', $conditions['conditionString']) . ' {op} ';
 				$conditionValues  = array_merge($conditionValues, $conditions['conditionValues']);
-			}
-			else if ($key == 'or')
+			} else if ($key == 'or')
 			{
 				$conditions       = $this->constructConditions($value, $model);
 				$conditionString .= str_replace('{op}', 'or', $conditions['conditionString']) . ' {op} ';
 				$conditionValues  = array_merge($conditionValues, $conditions['conditionValues']);
-			}
-			else
+			} else
 			{
 				if (is_array($value)) 
 				{
@@ -639,13 +635,11 @@ 
 					{
 						$value1 = $value['val1'];
 						$value2 = $value['val2'];
-					}
-					else
+					} else
 					{
 						$value = array_key_exists('val', $value) ? $value['val'] : '';
 					}
-				}
-				else
+				} else
 				{
 					$operator = '=';
 				}
@@ -657,29 +651,24 @@ 
 
 					$conditionString  .= $key . ' <= ? {op} ';
 					$conditionValues[] = $value2;
-				}
-				elseif (strtolower($operator) == 'in') 
+				} elseif (strtolower($operator) == 'in') 
 				{
 					$conditionValues  = array_merge($conditionValues, $value);
 					$inBindingsString = rtrim(str_repeat('?,', count($value)), ',');
 					$conditionString .= $key . ' in (' . rtrim($inBindingsString, ',') . ') {op} ';
-				}
-				elseif (strtolower($operator) == 'null') 
+				} elseif (strtolower($operator) == 'null') 
 				{
 					$conditionString .= $key . ' is null {op} ';
-				}
-				elseif (strtolower($operator) == 'not null') 
+				} elseif (strtolower($operator) == 'not null') 
 				{
 					$conditionString .= $key . ' is not null {op} ';
-				}
-				elseif (strtolower($operator) == 'has') 
+				} elseif (strtolower($operator) == 'has') 
 				{
 					$sql              = $model->withTrashed()->has($key)->toSql();
 					$conditions       = $this->constructConditions($value, $model->$key()->getRelated());
 					$conditionString .= rtrim(substr($sql, strpos($sql, 'exists')), ')') . ' and ' . $conditions['conditionString'] . ') {op} ';
 					$conditionValues  = array_merge($conditionValues, $conditions['conditionValues']);
-				}
-				else
+				} else
 				{
 					$conditionString  .= $key . ' ' . $operator . ' ? {op} ';
 					$conditionValues[] = $value;

src/Modules/Core/Console/Commands/GenerateDoc.php

Doc Comments +2 added lines, -2 removed lines

@@ -124,7 +124,7 @@ 
 	 * based on the docblock.
 	 * 
 	 * @param  array  &$route
-	 * @param  object $reflectionMethod
+	 * @param  \ReflectionMethod $reflectionMethod
 	 * @return void
 	 */
 	protected function processDocBlock(&$route, $reflectionMethod)
@@ -148,7 +148,7 @@ 
 	 * Generate post body for the given route.
 	 * 
 	 * @param  array  &$route
-	 * @param  object $reflectionMethod
+	 * @param  \ReflectionMethod $reflectionMethod
 	 * @param  array  $validationRules
 	 * @return void
 	 */

Spacing +9 added lines, -9 removed lines

@@ -43,7 +43,7 @@ 
 			if ($route) 
 			{
 				$actoinArray = explode('@', $route['action']);
-				if(array_get($actoinArray, 1, false))
+				if (array_get($actoinArray, 1, false))
 				{
 					$controller       = $actoinArray[0];
 					$method           = $actoinArray[1];
@@ -62,7 +62,7 @@ 
 					$route['response'] = $this->getResponseObject($classProperties['model'], $route['name'], $route['returnDocBlock']);
 
 					preg_match('/api\/([^#]+)\//iU', $route['uri'], $module);
-					$docData['modules'][$module[1]][substr($route['prefix'], strlen('/api/' . $module[1] . '/') - 1)][] = $route;
+					$docData['modules'][$module[1]][substr($route['prefix'], strlen('/api/'.$module[1].'/') - 1)][] = $route;
 
 					$this->getModels($classProperties['model'], $docData);   
 				}
@@ -81,7 +81,7 @@ 
 	 */
 	protected function getRoutes()
 	{
-		return collect(\Route::getRoutes())->map(function ($route) {
+		return collect(\Route::getRoutes())->map(function($route) {
 			if (strpos($route->uri(), 'api') !== false) 
 			{
 				return [
@@ -113,7 +113,7 @@ 
 		];
 
 
-		if (! $skipLoginCheck || ! in_array($method, $skipLoginCheck)) 
+		if ( ! $skipLoginCheck || ! in_array($method, $skipLoginCheck)) 
 		{
 			$route['headers']['Authorization'] = 'Bearer {token}';
 		}
@@ -167,16 +167,16 @@ 
 				}
 				else
 				{
-					$route['body'] = eval('return ' . str_replace(',\'.$request->get(\'id\')', ',{id}\'', $match[1]) . ';');
+					$route['body'] = eval('return '.str_replace(',\'.$request->get(\'id\')', ',{id}\'', $match[1]).';');
 				}
 
 				foreach ($route['body'] as &$rule) 
 				{
-					if(strpos($rule, 'unique'))
+					if (strpos($rule, 'unique'))
 					{
 						$rule = substr($rule, 0, strpos($rule, 'unique') + 6);
 					}
-					elseif(strpos($rule, 'exists'))
+					elseif (strpos($rule, 'exists'))
 					{
 						$rule = substr($rule, 0, strpos($rule, 'exists') - 1);
 					}
@@ -249,10 +249,10 @@ 
 			$model      = factory($modelClass)->make();
 			$modelArr   = $model->toArray();
 
-			if ( $model->trans && ! $model->trans->count()) 
+			if ($model->trans && ! $model->trans->count()) 
 			{
 				$modelArr['trans'] = [
-					'en' => factory($modelClass . 'Translation')->make()->toArray()
+					'en' => factory($modelClass.'Translation')->make()->toArray()
 				];
 			}
 

Braces +3 added lines, -6 removed lines

@@ -164,8 +164,7 @@ 
 				if ($match[1] == '$this->validationRules')
 				{
 					$route['body'] = $validationRules;
-				}
-				else
+				} else
 				{
 					$route['body'] = eval('return ' . str_replace(',\'.$request->get(\'id\')', ',{id}\'', $match[1]) . ';');
 				}
@@ -175,14 +174,12 @@ 
 					if(strpos($rule, 'unique'))
 					{
 						$rule = substr($rule, 0, strpos($rule, 'unique') + 6);
-					}
-					elseif(strpos($rule, 'exists'))
+					} elseif(strpos($rule, 'exists'))
 					{
 						$rule = substr($rule, 0, strpos($rule, 'exists') - 1);
 					}
 				}
-			}
-			else
+			} else
 			{
 				$route['body'] = 'conditions';
 			}

src/Modules/Notifications/Notifications/ConfirmEmail.php

Doc Comments +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 	 * Get the notification's delivery channels.
 	 *
 	 * @param  mixed  $notifiable
-	 * @return array
+	 * @return string[]
 	 */
 	public function via($notifiable)
 	{

Spacing +1 added lines, -1 removed lines

@@ -43,6 +43,6 @@
 			->subject('Email verification')
 			->line('Email verification')
 			->line('To validate your email click on the button below')
-			->action('Verify your email', config('skeleton.confrim_email_url') . '/' . $notifiable->confirmation_code);
+			->action('Verify your email', config('skeleton.confrim_email_url').'/'.$notifiable->confirmation_code);
 	}
 }
\ No newline at end of file

files/Handler.php

Braces +6 added lines, -12 removed lines

@@ -57,29 +57,23 @@
 			if ($exception instanceof \Illuminate\Database\QueryException) 
 			{
 				\ErrorHandler::dbQueryError();
-			}
-			else if ($exception instanceof \predis\connection\connectionexception) 
+			} else if ($exception instanceof \predis\connection\connectionexception) 
 			{
 				\ErrorHandler::redisNotRunning();
-			}
-			else if ($exception instanceof \GuzzleHttp\Exception\ClientException) 
+			} else if ($exception instanceof \GuzzleHttp\Exception\ClientException) 
 			{
 				\ErrorHandler::connectionError();
-			}
-			else if ($exception instanceof \Symfony\Component\HttpKernel\Exception\HttpException) 
+			} else if ($exception instanceof \Symfony\Component\HttpKernel\Exception\HttpException) 
 			{
 				return \Response::json($exception->getMessage(), $exception->getStatusCode());   
-			}
-			else if ($exception instanceof \Illuminate\Validation\ValidationException) 
+			} else if ($exception instanceof \Illuminate\Validation\ValidationException) 
 			{
 				return \Response::json($exception->errors(), 422);   
-			}
-			else if ( ! $exception instanceof \Symfony\Component\Debug\Exception\FatalErrorException)
+			} else if ( ! $exception instanceof \Symfony\Component\Debug\Exception\FatalErrorException)
 			{
 				return parent::render($request, $exception);
 			}
-		}
-		else
+		} else
 		{
 			return parent::render($request, $exception);
 		}

files/AuthServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 	{
 		$this->registerPolicies();
 
-		Passport::routes(function ($router) {
+		Passport::routes(function($router) {
 			$router->forAuthorization();
 			$router->forAccessTokens();
 			$router->forPersonalAccessTokens();

src/Modules/Reporting/Providers/RouteServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -54,7 +54,7 @@ 
 		Route::group([
 			'middleware' => 'web',
 			'namespace'  => $this->namespace,
-		], function ($router) {
+		], function($router) {
 			require module_path('reporting', 'Routes/web.php');
 		});
 	}
@@ -72,7 +72,7 @@ 
 			'middleware' => 'api',
 			'namespace'  => $this->namespace,
 			'prefix'     => 'api',
-		], function ($router) {
+		], function($router) {
 			require module_path('reporting', 'Routes/api.php');
 		});
 	}

src/Modules/Reporting/Report.php

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Eloquent\SoftDeletes;
 
-class Report extends Model{
+class Report extends Model {
 
 	use SoftDeletes;
 	protected $table    = 'reports';

src/Modules/Reporting/Repositories/ReportRepository.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
 		{
 			\ErrorHandler::notFound('report');
 		}
-		else if (! $skipPermission && ! \Core::users()->can($report->view_name, 'reports'))
+		else if ( ! $skipPermission && ! \Core::users()->can($report->view_name, 'reports'))
 		{
 			\ErrorHandler::noPermissions();
 		}

Braces +2 added lines, -4 removed lines

@@ -39,8 +39,7 @@ 
 		if ( ! $report) 
 		{
 			\ErrorHandler::notFound('report');
-		}
-		else if (! $skipPermission && ! \Core::users()->can($report->view_name, 'reports'))
+		} else if (! $skipPermission && ! \Core::users()->can($report->view_name, 'reports'))
 		{
 			\ErrorHandler::noPermissions();
 		}
@@ -61,8 +60,7 @@ 
 		if ($perPage) 
 		{
 			return $report->paginate($perPage);
-		}
-		else
+		} else
 		{
 			return $report->get();  
 		}