afrittella / back-project

src/app/Repositories/Base.php

Spacing +5 added lines, -5 removed lines

@@ -48,7 +48,7 @@ 
         return $this->model->create($data);
     }
 
-    public function update(array $data, $id, $attribute="id")
+    public function update(array $data, $id, $attribute = "id")
     {
         $model_data = $this->model->where($attribute, '=', $id)->first();
 
@@ -123,7 +123,7 @@ 
         return $this;
     }
 
-    public function skipCriteria($status = true){
+    public function skipCriteria($status = true) {
         $this->skipCriteria = $status;
         return $this;
     }
@@ -143,11 +143,11 @@ 
     }
 
     public function  applyCriteria() {
-        if($this->skipCriteria === true)
+        if ($this->skipCriteria === true)
             return $this;
 
-        foreach($this->getCriteria() as $criteria) {
-            if($criteria instanceof Criteria)
+        foreach ($this->getCriteria() as $criteria) {
+            if ($criteria instanceof Criteria)
                 $this->model = $criteria->apply($this->model, $this);
         }
 

Doc Comments +6 added lines

@@ -29,6 +29,9 @@ 
         $this->makeModel();
     }
 
+    /**
+     * @return string
+     */
     abstract public function model();
 
     public function all($columns = ['*'])
@@ -96,6 +99,9 @@ 
         return $this->model->findOrCreate($data);
     }
 
+    /**
+     * @param string $attribute
+     */
     public function findBy($attribute, $value, $columns = array('*'))
     {
         $this->applyCriteria();

Braces +9 added lines, -6 removed lines

@@ -107,8 +107,9 @@ 
         $model = $this->app->make($this->model());
         //$model = app()->make($this->model());
 
-        if (!$model instanceof Model)
-            throw new RepositoryException("Class {$this->model()} must be an instance of Illuminate\\Database\\Eloquent\\Model");
+        if (!$model instanceof Model) {
+                    throw new RepositoryException("Class {$this->model()} must be an instance of Illuminate\\Database\\Eloquent\\Model");
+        }
 
         return $this->model = $model;
     }
@@ -143,12 +144,14 @@ 
     }
 
     public function  applyCriteria() {
-        if($this->skipCriteria === true)
-            return $this;
+        if($this->skipCriteria === true) {
+                    return $this;
+        }
 
         foreach($this->getCriteria() as $criteria) {
-            if($criteria instanceof Criteria)
-                $this->model = $criteria->apply($this->model, $this);
+            if($criteria instanceof Criteria) {
+                            $this->model = $criteria->apply($this->model, $this);
+            }
         }
 
         return $this;