fix determine order column name bug

spatie · Oct 14, 2014 · a8ede61 · a8ede61
1 parent bf577dc
commit a8ede61
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/Spatie/EloquentSortable/Sortable.php b/src/Spatie/EloquentSortable/Sortable.php
@@ -23,11 +23,11 @@ protected function determineOrderColumnName()
     {
         if (! isset($this->sortable['order_column_name']) OR $this->sortable['order_column_name'] == '')
         {
-            return 'order_column';
+            $orderColumnName =  'order_column';
         }
         else
         {
-            return $this->sortable['order_column_name'];
+            $orderColumnName = $this->sortable['order_column_name'];
         }
 
         return $orderColumnName;
@@ -40,7 +40,7 @@ protected function determineOrderColumnName()
      */
     public function getHighestOrderNumber()
     {
-         return ((int) self::max($this->sortable['order_column_name'])) + 1;
+         return ((int) self::max($this->determineOrderColumnName())) + 1;
     }
 
     /**
@@ -51,7 +51,7 @@ public function getHighestOrderNumber()
      */
     public function scopeOrdered($query)
     {
-        return $query->orderBy($this->sortable['order_column_name']);
+        return $query->orderBy($this->determineOrderColumnName());
     }
 
     /**