Merge pull request #63 from CPSB/patch-61

Convert $id variables $primaryKey
Misfits-BE · Nov 26, 2017 · bf0643f · bf0643f
2 parents 8ae7a13 + 07a23a3
commit bf0643f
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/ActivismeBE/Repositories/Eloquent/Repository.php b/src/ActivismeBE/Repositories/Eloquent/Repository.php
@@ -214,7 +214,7 @@ public function saveModel(array $data)
      */
     public function update(array $data, $primaryKey, $attribute = "id")
     {
-        return $this->model->where($attribute, '=', $id)->update($data);
+        return $this->model->where($attribute, '=', $primaryKey)->update($data);
     }
 
     /**
@@ -227,7 +227,7 @@ public function update(array $data, $primaryKey, $attribute = "id")
      */
     public function updateRich(array $data, $primaryKey)
     {
-        if (! ($model = $this->model->find($id))) {
+        if (! ($model = $this->model->find($primaryKey))) {
             return false;
         }
 
@@ -291,7 +291,7 @@ public function find($primaryKey, $columns = array('*'))
      */
     public function findOrFail($primaryKey) 
     {
-        return $this->model->findOrFail($id);
+        return $this->model->findOrFail($primaryKey);
     }
 
     /**