Fix wrong subscribable reference

Closes #30
gerardojbaez · Feb 23, 2018 · 8a23b42 · 8a23b42
1 parent 8ee1e25
commit 8a23b42
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -9,6 +9,9 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/) and this p
 - Support for Laravel 5.4
 - `SubscriptionDeleted` event.
 
+### Fixed
+- Renamed `user_id` to `subscribable_id`, fixes #30
+
 ## [2.1.0] - 2017-11-27
 
 ### Added

diff --git a/src/Laraplans/Models/PlanSubscription.php b/src/Laraplans/Models/PlanSubscription.php
@@ -306,15 +306,15 @@ public function ability()
     }
 
     /**
-     * Find by user id.
+     * Find by subscribable id.
      *
      * @param  \Illuminate\Database\Eloquent\Builder
-     * @param  int $user_id
+     * @param  int $subscribable
      * @return \Illuminate\Database\Eloquent\Builder
      */
-    public function scopeByUser($query, $user_id)
+    public function scopeByUser($query, $subscribable)
     {
-        return $query->where('user_id', $user_id);
+        return $query->where('subscribable_id', $subscribable);
     }
 
     /**