liszt 0.0.3 → 0.0.4

data/lib/liszt.rb

@@ -44,7 +44,8 @@ module Liszt
     def initialize_list!(obj={}, &block)
       objects = find(:all, :conditions => liszt_query(obj))
 
-      # If the caller provided a block, sort the objects by that block's
+      # If the caller provided a block, or if they passed in a default
+      # with the :sort_by option, sort the objects by that block's
       # output before populating the list with their ids. If not, put
       # the objects in descending order by id.
       ids = if block_given?
@@ -70,15 +71,20 @@ module Liszt
     end
 
     def ordered_list_ids(obj={})
-      return nil unless ordered_list_initialized?(obj)
-      ordered_list(obj).all
+      if ordered_list_initialized?(obj)
+        ordered_list(obj).all
+      else
+        initialize_list!(obj)
+      end
     end
 
     def ordered_list_items(obj={}, double_check=false)
-      return nil unless ordered_list_initialized?(obj)
+      was_initialized = ordered_list_initialized?(obj)
       ids = ordered_list_ids(obj)
 
-      if double_check
+      # If ordered_list_ids just did the initialization, we can trust that
+      # the list of ids is accurate and ignore the double_check flag.
+      if double_check and was_initialized
         objs = find(:all, :conditions => liszt_query(obj))
         real_ids = objs.map(&:id)
         unlisted_ids = real_ids - ids

data/lib/liszt/version.rb

@@ -1,3 +1,3 @@
 module Liszt
-  VERSION = "0.0.3"
+  VERSION = "0.0.4"
 end

data/test/liszt_test.rb

@@ -17,6 +17,11 @@ class LisztTest < ActiveSupport::TestCase
       assert_nothing_raised { people(:nelson).initialize_list! }
       assert people(:nelson).ordered_list_initialized?
     end
+
+    should "auto-initialize successfully" do
+      assert_nothing_raised { people(:nelson).ordered_list_ids }
+      assert people(:nelson).ordered_list_initialized?
+    end
   end
 
   context "after initialization" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: liszt
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2011-09-23 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
-  requirement: &70260000385480 !ruby/object:Gem::Requirement
+  requirement: &70165237093020 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: 2.3.2
   type: :runtime
   prerelease: false
-  version_requirements: *70260000385480
+  version_requirements: *70165237093020
 - !ruby/object:Gem::Dependency
   name: redis
-  requirement: &70260000385060 !ruby/object:Gem::Requirement
+  requirement: &70165237092600 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70260000385060
+  version_requirements: *70165237092600
 - !ruby/object:Gem::Dependency
   name: yard
-  requirement: &70260000384600 !ruby/object:Gem::Requirement
+  requirement: &70165237092140 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70260000384600
+  version_requirements: *70165237092140
 - !ruby/object:Gem::Dependency
   name: rdiscount
-  requirement: &70260000384180 !ruby/object:Gem::Requirement
+  requirement: &70165237091720 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,7 +54,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70260000384180
+  version_requirements: *70165237091720
 description: Liszt is an alternative to acts_as_list and sortable that uses atomic
   Redis operations to maintain scoped ordering information for ActiveRecord objects.
 email: