SherifElfadaly / Laravel-Api-Skeleton

src/Modules/Core/BaseClasses/BaseRepository.php

Spacing +25 added lines, -25 removed lines

@@ -120,7 +120,7 @@ 
         $model     = new Model();
         $relations = [];
 
-        DB::transaction(function () use (&$model, &$relations, $data) {
+        DB::transaction(function() use (&$model, &$relations, $data) {
 
             $model = $this->prepareModel($data);
             $model->save();
@@ -157,8 +157,8 @@ 
      */
     public function delete(string $value, string $attribute = 'id'): bool
     {
-        DB::transaction(function () use ($value, $attribute) {
-            $this->model->where($attribute, '=', $value)->lockForUpdate()->get()->each(function ($model) {
+        DB::transaction(function() use ($value, $attribute) {
+            $this->model->where($attribute, '=', $value)->lockForUpdate()->get()->each(function($model) {
                 $model->delete();
             });
         });
@@ -247,8 +247,8 @@ 
     {
         $model = $this->model->onlyTrashed()->find($id);
 
-        if (!$model) {
-            Errors::notFound(class_basename($this->model) . ' with id : ' . $id);
+        if ( ! $model) {
+            Errors::notFound(class_basename($this->model).' with id : '.$id);
         }
 
         $model->restore();
@@ -267,8 +267,8 @@ 
         $modelClass = $this->model;
         $model = Arr::has($data, 'id') ? $modelClass->lockForUpdate()->find($data['id']) : new $modelClass;
 
-        if (!$model) {
-            Errors::notFound(class_basename($modelClass) . ' with id : ' . $data['id']);
+        if ( ! $model) {
+            Errors::notFound(class_basename($modelClass).' with id : '.$data['id']);
         }
 
         foreach ($data as $key => $value) {
@@ -297,7 +297,7 @@ 
                 Core::$relation() &&
                 in_array(class_basename($model->$key()), ['MorphToMany', 'BelongsToMany'])
             ) {
-                if (!$relatedModels || !count($relatedModels)) {
+                if ( ! $relatedModels || ! count($relatedModels)) {
                     /**
                      * Mark the relation to be deleted.
                      */
@@ -324,14 +324,14 @@ 
      */
     protected function prepareRelatedModel(mixed $relatedModelData, Model $relationBaseModel): Model
     {
-        if (!is_array($relatedModelData)) { // if the relation is integer id
+        if ( ! is_array($relatedModelData)) { // if the relation is integer id
             $relatedModel = $relationBaseModel->lockForUpdate()->find($relatedModelData);
         } else {
             $relatedModel = Arr::has($relatedModelData, 'id') ? $relationBaseModel->lockForUpdate()->find($relatedModelData['id']) : new $relationBaseModel;
         }
 
-        if (!$relatedModel) {
-            Errors::notFound(class_basename($relationBaseModel) . ' with id : ' . $relatedModelData['id']);
+        if ( ! $relatedModel) {
+            Errors::notFound(class_basename($relationBaseModel).' with id : '.$relatedModelData['id']);
         }
 
         if (is_array($relatedModelData)) {
@@ -401,11 +401,11 @@ 
 
             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']);
             } elseif ($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 {
                 if (is_array($value)) {
@@ -421,38 +421,38 @@ 
                 }
 
                 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()->withoutGlobalScopes()->has($key)->toSql();
                     $bindings         = $model->withTrashed()->withoutGlobalScopes()->has($key)->getBindings();
                     if ($value) {
                         $conditions       = $this->constructConditions($value, $model->$key()->getRelated());
-                        $conditionString .= substr(substr($sql, strpos($sql, 'exists')), 0, -1) . ' and ' . $conditions['conditionString'] . ') {op} ';
+                        $conditionString .= substr(substr($sql, strpos($sql, 'exists')), 0, -1).' and '.$conditions['conditionString'].') {op} ';
                         $conditionValues  = array_merge($conditionValues, $bindings);
                         $conditionValues  = array_merge($conditionValues, $conditions['conditionValues']);
                     } else {
-                        $conditionString .= substr(substr($sql, strpos($sql, 'exists')), 0, -1) . ') {op} ';
+                        $conditionString .= substr(substr($sql, strpos($sql, 'exists')), 0, -1).') {op} ';
                         $conditionValues  = array_merge($conditionValues, $bindings);
                     }
                 } else {
-                    $conditionString  .= $key . ' ' . $operator . ' ? {op} ';
+                    $conditionString  .= $key.' '.$operator.' ? {op} ';
                     $conditionValues[] = $value;
                 }
             }
         }
-        $conditionString = '(' . rtrim($conditionString, '{op} ') . ')';
+        $conditionString = '('.rtrim($conditionString, '{op} ').')';
         return ['conditionString' => $conditionString, 'conditionValues' => $conditionValues];
     }
 
@@ -468,10 +468,10 @@ 
         $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) {
-            return '"' . $part . '"';
+        $result     = sprintf('%s->\'$.%s\'', $field, collect($path)->map(function($part) {
+            return '"'.$part.'"';
         })->implode('.'));
 
-        return $removeLast === false ? $result : $result . ')';
+        return $removeLast === false ? $result : $result.')';
     }
 }