removing acts_as_list

Gemfile

@@ -5,7 +5,6 @@ gem "rails", "3.0.9"
 gem "mysql2"
 gem "redcarpet"
 gem "coderay"
-gem "acts_as_list"
 gem "thinking-sphinx", ">= 2.0.1", :require => "thinking_sphinx"
 gem "whenever", :require => false
 gem "will_paginate", ">= 3.0.pre2"

Gemfile.lock

@@ -42,7 +42,6 @@ GEM
       activemodel (= 3.0.9)
       activesupport (= 3.0.9)
     activesupport (3.0.9)
-    acts_as_list (0.1.2)
     addressable (2.2.5)
     ancestry (1.2.3)
       activerecord (>= 2.2.2)
@@ -232,7 +231,6 @@ PLATFORMS
   ruby
 
 DEPENDENCIES
-  acts_as_list
   ancestry
   cancan!
   capistrano

app/controllers/episodes_controller.rb

@@ -24,6 +24,7 @@ def show
   end
 
   def new
+    @episode.position = Episode.maximum(:position).to_i + 1
   end
 
   def create

app/models/comment.rb

@@ -8,7 +8,6 @@ class Comment < ActiveRecord::Base
 
   has_paper_trail
   has_ancestry
-  acts_as_list :scope => :episode
 
   def request=(request)
     self.user_ip    = request.remote_ip

app/models/episode.rb

@@ -4,7 +4,6 @@ class Episode < ActiveRecord::Base
   has_many :tags, :through => :taggings
 
   has_paper_trail
-  acts_as_list
 
   scope :published, lambda { where('published_at <= ?', Time.now.utc) }
   scope :unpublished, lambda { where('published_at > ?', Time.now.utc) }
@@ -149,6 +148,14 @@ def fetch_file_size(path)
     end
   end
 
+  def previous
+    self.class.where("position < ?", position).order("position desc").first
+  end
+
+  def next
+    self.class.where("position > ?", position).order("position").first
+  end
+
   private
 
   def self.primitive_search_conditions(query, join)

app/views/episodes/_form.html.erb

@@ -2,6 +2,7 @@
   <p><strong>Please follow the <%= link_to "moderator guidelines", moderators_path %>.</strong></p>
   <%= form_for @episode do |f| %>
     <%= f.error_messages %>
+    <%= field f, :position %>
     <%= field f, :name %>
     <%= field f, :tag_names, :label => "Tags" %>
     <%= field f, :description, :type => :text_area, :rows => 6 %>

app/views/episodes/show.html.erb

@@ -43,8 +43,8 @@
     <li class="selected"><%= tab_link "Show Notes", :view => nil %></li>
     <li><%= tab_link pluralize(@episode.comments.size, "Comment"), :view => "comments" %></li>
     <li><%= tab_link "Similar Episodes", :view => "similar" %></li>
-    <%= content_tag :li, link_to("Next Episode >", @episode.lower_item), :class => "next" unless @episode.last_published? || @episode.last? %>
-    <%= content_tag :li, link_to("< Previous Episode", @episode.higher_item), :class => "previous" unless @episode.first? %>
+    <%= content_tag :li, link_to("Next Episode >", @episode.next), :class => "next" if !@episode.last_published? && @episode.next %>
+    <%= content_tag :li, link_to("< Previous Episode", @episode.previous), :class => "previous" if @episode.previous %>
   </ul>
   <div class="nav_section">
     <%= %w[comments similar].include?(params[:view]) ? render(params[:view]) : render("show_notes") %>

spec/models/episode_spec.rb

@@ -17,8 +17,8 @@
 
   it "should sort recent episodes in descending order" do
     Episode.delete_all
-    e1 = Factory(:episode)
-    e2 = Factory(:episode)
+    e1 = Factory(:episode, :position => 1)
+    e2 = Factory(:episode, :position => 2)
     Episode.recent.should == [e2, e1]
   end
 
@@ -116,7 +116,17 @@
 
   it "has a full name which includes position" do
     Episode.delete_all
-    Factory(:episode, :name => "Foo Bar").full_name.should eq('#1 Foo Bar')
+    Factory(:episode, :position => 123, :name => "Foo Bar").full_name.should eq('#123 Foo Bar')
+  end
+
+  it "knows the next and previous episode based on position" do
+    Episode.delete_all
+    e1 = Factory(:episode, :position => 1)
+    e2 = Factory(:episode, :position => 6)
+    e1.previous.should be_nil
+    e1.next.should eq(e2)
+    e2.next.should be_nil
+    e2.previous.should eq(e1)
   end
 
   describe "primitive search" do

spec/requests/episodes_request_spec.rb

@@ -53,10 +53,10 @@
   end
 
   it "contains show notes, comments, and similar episodes" do
-    episode = Factory(:episode, :name => "Blast from the Past", :notes => "Show notes!")
+    episode = Factory(:episode, :name => "Blast from the Past", :notes => "Show notes!", :position => 1)
     Factory(:comment, :content => "Hello world", :episode => episode)
-    Factory(:episode, :name => "Star Wars")
-    Factory(:episode, :name => "Past and Present")
+    Factory(:episode, :name => "Star Wars", :position => 2)
+    Factory(:episode, :name => "Past and Present", :position => 3)
     visit episodes_path
     click_on "Blast from the Past"
     page.should have_content("Blast from the Past")
@@ -88,7 +88,7 @@
     page.should have_content("not authorized")
   end
 
-  it "creates a new episode" do
+  it "creates a new episode with default position" do
     login Factory(:user, :admin => true)
     visit episodes_path
     click_on "New Episode"