Ordering projects on project list

app/controllers/projects_controller.rb

@@ -1,7 +1,7 @@
 class ProjectsController < ApplicationController
-  before_filter :locate_project, :only => [:show, :build, :edit, :update, :remove, :destroy]
+  before_filter :locate_project, :only => [:show, :build, :edit, :update, :remove, :destroy, :arrange]
   def index
-    @projects = Project.order("created_at DESC")
+    @projects = Project.order("position ASC")
   end
 
   def show
@@ -42,6 +42,15 @@ def destroy
     redirect_to projects_path
   end
 
+  def arrange
+    if params[:up]
+      @project.move_higher
+    elsif params[:down]
+      @project.move_lower
+    end
+    redirect_to projects_path
+  end
+
   private
   def locate_project
     @project = Project.find(params[:id])

app/models/project.rb

@@ -1,8 +1,11 @@
 class Project < ActiveRecord::Base
   STATUS_NOT_BUILT = "status_not_built"
+
   has_many :builds, :dependent => :destroy
   before_destroy :remove_build_folder
 
+  acts_as_list
+
   def recent_build
     builds.order("created_at DESC").first
   end

app/views/projects/index.html.haml

@@ -11,4 +11,9 @@
           %br
           = ("author: " + link_to(project.recent_build.author, "mailto:#{project.recent_build.email}")).html_safe
           %br
-          = "message: " + truncate(project.recent_build.commit_message, :length => 35)
+          = "message: " + truncate(project.recent_build.commit_message, :length => 55)
+      .info
+        - if ! project.first?
+          = link_to("Up", arrange_project_path(project, :up => true))
+        - if ! project.last?
+          = link_to("Down", arrange_project_path(project, :down => true))

app/views/projects/show.html.haml

@@ -11,7 +11,7 @@
         .info
           = ("author: " + link_to(build.author, "mailto:#{build.email}")).html_safe
           %br
-          = "message: " + truncate(build.commit_message, :length => 35)
+          = "message: " + truncate(build.commit_message, :length => 55)
           %br
           = "scheduled at: " + build.scheduled_at.strftime("%Y-%m-%d %H:%M")
         .info

config/routes.rb

@@ -1,6 +1,6 @@
 BigTuna::Application.routes.draw do
   resources :projects do
-    member { get "build"; get "remove" }
+    member { get "build"; get "remove"; get "arrange" }
   end
   resources :builds
   match "/hooks/:hook_name", :to => "hooks#post_commit"

db/migrate/20101126094722_add_position_to_project.rb

@@ -0,0 +1,9 @@
+class AddPositionToProject < ActiveRecord::Migration
+  def self.up
+    add_column(:projects, :position, :integer)
+  end
+
+  def self.down
+    remove_column(:projects, :position)
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended to check this file into your version control system.
 
-ActiveRecord::Schema.define(:version => 20101125175448) do
+ActiveRecord::Schema.define(:version => 20101126094722) do
 
   create_table "builds", :force => true do |t|
     t.integer  "project_id"
@@ -52,6 +52,7 @@
     t.integer  "max_builds"
     t.string   "hook_name"
     t.text     "steps"
+    t.integer  "position"
   end
 
 end

test/integration/projects_test.rb

@@ -9,6 +9,7 @@ def setup
   def teardown
     FileUtils.rm_rf("test/files/repo")
     FileUtils.rm_rf("builds/valid")
+    FileUtils.rm_rf("builds/valid2")
     FileUtils.rm_rf("builds/invalid")
     super
   end
@@ -48,4 +49,27 @@ def teardown
       click_button "Yes, I'm sure"
     end
   end
+
+  test "user can reorder projects on project list" do
+    project1 = Project.make(:steps => "echo 'ha'", :name => "Valid", :vcs_source => "test/files/repo", :vcs_type => "git")
+    project2 = Project.make(:steps => "echo 'sa'", :name => "Valid2", :vcs_source => "test/files/repo", :vcs_type => "git")
+    visit "/"
+    within("#project_#{project2.id}") do
+      assert page.has_content?("Up")
+      assert ! page.has_content?("Down")
+    end
+    within("#project_#{project1.id}") do
+      assert page.has_content?("Down")
+      assert ! page.has_content?("Up")
+    end
+    click_link "Down"
+    within("#project_#{project1.id}") do
+      assert page.has_content?("Up")
+      assert ! page.has_content?("Down")
+    end
+    within("#project_#{project2.id}") do
+      assert page.has_content?("Down")
+      assert ! page.has_content?("Up")
+    end
+  end
 end

vendor/plugins/acts_as_list/README

@@ -0,0 +1,23 @@
+ActsAsList
+==========
+
+This acts_as extension provides the capabilities for sorting and reordering a number of objects in a list. The class that has this specified needs to have a +position+ column defined as an integer on the mapped database table.
+
+
+Example
+=======
+
+  class TodoList < ActiveRecord::Base
+    has_many :todo_items, :order => "position"
+  end
+
+  class TodoItem < ActiveRecord::Base
+    belongs_to :todo_list
+    acts_as_list :scope => :todo_list
+  end
+
+  todo_list.first.move_to_bottom
+  todo_list.last.move_higher
+
+
+Copyright (c) 2007 David Heinemeier Hansson, released under the MIT license

vendor/plugins/acts_as_list/Rakefile

@@ -0,0 +1,12 @@
+require 'rake'
+require 'rake/testtask'
+
+desc 'Default: run acts_as_list unit tests.'
+task :default => :test
+
+desc 'Test the acts_as_ordered_tree plugin.'
+Rake::TestTask.new(:test) do |t|
+  t.libs << 'lib'
+  t.pattern = 'test/**/*_test.rb'
+  t.verbose = true
+end

vendor/plugins/acts_as_list/init.rb

@@ -0,0 +1,3 @@
+$:.unshift "#{File.dirname(__FILE__)}/lib"
+require 'active_record/acts/list'
+ActiveRecord::Base.class_eval { include ActiveRecord::Acts::List }