Fixed the default value issue

brendon · Aug 11, 2011 · e81eb82 · e81eb82 · MattRogish · Aug 11, 2011
1 parent 8e39132
commit e81eb82
Show file tree

Hide file tree

Showing 2 changed files with 615 additions and 3 deletions.
diff --git a/lib/acts_as_list/active_record/acts/list.rb b/lib/acts_as_list/active_record/acts/list.rb
@@ -180,7 +180,19 @@ def lower_item
 
         # Test if this record is in a list
         def in_list?
-          !send(position_column).nil?
+          !not_in_list?
+        end
+
+        def not_in_list?
+          send(position_column).nil?
+        end
+
+        def default_position
+          acts_as_list_class.columns_hash[position_column.to_s].default
+        end
+
+        def default_position?
+          default_position == send(position_column)
         end
 
         private
@@ -189,7 +201,7 @@ def add_to_list_top
           end
 
           def add_to_list_bottom
-            if self[position_column].nil?
+            if not_in_list? || default_position?
               self[position_column] = bottom_position_in_list.to_i + 1
             else
               increment_positions_on_lower_items(self[position_column])