backlog 0.6.6 → 0.7.0

data/History.txt

@@ -1,3 +1,14 @@
+== 0.7.0 2007-08-14
+
+* Back in business after halting to avoid spamming people.
+* Fixed work flow when moving task to other period using drag&drop in side bar
+* Fixed bug in moving tasks to other periods
+* Optimized slightly by omitting generating layout data for no_layout action
+* Added more validation of "position" fields
+* Added completed tasks to backlog edit view
+* Removed unique index for periods(party_id, position) since it broke list updates in PostgreSQL
+* Added better assertion of position sequences in tests
+
 == 0.6.6 2007-08-07
 
 * Improved task lists in backlog edit view.

data/app/controllers/application.rb

@@ -4,6 +4,7 @@ require 'user_system'
 class ApplicationController < ActionController::Base
   include Localization
   include UserSystem
+  include ApplicationHelper
   include ActionView::Helpers::TagHelper
   include ActionView::Helpers::JavaScriptHelper
   include ActionView::Helpers::PrototypeHelper
@@ -138,7 +139,7 @@ class ApplicationController < ActionController::Base
         content << "</ul>\n"
         content << drop_receiving_element("sidebar_#{period.id}",
         :update => "spotlight", 
-        :url => { :controller => 'tasks', :action => "move_to_period", :period_id => period.id},
+        :url => with_detour({:controller => 'tasks', :action => "move_to_period", :period_id => period.id, :layout => false}),
         :accept => "tasks", :loading => "", :complete => "",
         :hoverclass => 'highlight')
         

data/app/controllers/periods_controller.rb

@@ -1,5 +1,5 @@
 class PeriodsController < ApplicationController
-  skip_before_filter :populate_layout, :only => [:index, :show_nolayout, :destroy, :burn_down_chart_thumbnail, :burn_down_chart]
+  skip_before_filter :populate_layout, :only => [:index, :show_no_layout, :destroy, :burn_down_chart_thumbnail, :burn_down_chart]
   skip_before_filter :authenticate_user, :only => [:burn_down_chart, :burn_down_chart_thumbnail, :burn_down_chart_large]
   
   def index

data/app/controllers/tasks_controller.rb

@@ -59,18 +59,24 @@ class TasksController < ApplicationController
       back_or_redirect_to :controller => 'periods', :action => :show, :id => @task.period, :task => @task.id
       return
     end
+
+    # TODO (uwe): This should be moved to the Task model: Task#period=
     if params[:task] && params[:task][:period_id]
       if params[:task][:period_id] != ''
         if params[:task][:period_id].to_i != @task.period_id
+          @task.remove_from_list
           params[:task][:position] = Period.find(params[:task][:period_id]).tasks.size
         end
       else
         if @task.period_id != nil
+          @task.remove_from_list
           @task.period = nil
-          params[:task][:position] = @task.backlog.tasks.count(:conditions => 'period_id IS NULL') + 1
+          new_pos = @task.backlog.tasks.count(:conditions => 'period_id IS NULL') + 1
+          params[:task][:position] = new_pos
         end
       end
     end
+
     if params[:task] && @task.update_attributes(params[:task])
       flash[:notice] = 'Task was successfully updated.'
       if params[:task][:todo] && params[:task][:todo].to_i == 0
@@ -137,11 +143,11 @@ class TasksController < ApplicationController
     end
     task.insert_at params[:position]
     task.save!
-    back_or_redirect_to :controller => 'periods', :action => :show_nolayout, :id => task.period, :task => task.id
+    back_or_redirect_to :controller => 'periods', :action => :show_no_layout, :id => task.period, :task => task.id
   end
   
   def move_to_period(with_layout = false)
-    task = Task.find(params[:id])
+    task = Task.find(params[:id][5..-1])
     if request.post?
       period = params[:period_id] && Period.find(params[:period_id])
       if period && period.active_or_future?
@@ -150,7 +156,7 @@ class TasksController < ApplicationController
         if with_layout
           back_or_redirect_to :controller => 'periods', :action => :show, :id => next_task.period, :task_id => next_task.id
         else
-          back_or_redirect_to :controller => 'periods', :action => :show_nolayout, :id => next_task.period, :task_id => next_task.id
+          back_or_redirect_to :controller => 'periods', :action => :show_no_layout, :id => next_task.period, :task_id => next_task.id
         end
       else
         detour_to :controller => 'periods', :action => :new, :period => {:party_id => task.period.party_id}, :layout => with_layout && determine_layout
@@ -164,6 +170,7 @@ class TasksController < ApplicationController
     task = Task.find(params[:id])
     if task.period
       params[:period_id] = task.period.lower_item && task.period.lower_item.id
+      params[:id] = 'task_' + task.id.to_s
       move_to_period(true)
     else
       redirect_to :action => :edit, :id => task

data/app/helpers/application_helper.rb

@@ -11,10 +11,10 @@ module ApplicationHelper
   end
   
   def detour_to(title, options, html_options = nil)
-    link_to title, options_for_detour(options), html_options
+    link_to title, with_detour(options), html_options
   end
   
-  def options_for_detour(options)
+  def with_detour(options)
     detour_options = {:detour => params.reject {|k, v| [:detour, :return_from_detour].include? k.to_sym}}.update(options)
     if options[:layout]== false
       if params[:action] !~ /_no_layout$/

data/app/models/backlog.rb

@@ -10,6 +10,7 @@ class Backlog < ActiveRecord::Base
   validates_length_of :invoice_code, :allow_nil => true, :maximum => 255
 
   has_many :tasks, :order => 'period_id, position', :dependent => :destroy
+  has_many :unplanned_tasks, :class_name => 'Task', :conditions => 'period_id IS NULL', :order => 'position'
 
   def active_tasks
     tasks.find(:all, :conditions => "finished_at IS NULL", :order => "position")

data/app/models/group.rb

@@ -1,4 +1,3 @@
-#class Group < ActiveRecord::Base
 class Group < Party
   class_table_inheritance
 

data/app/models/task.rb

@@ -16,11 +16,11 @@ class Task < ActiveRecord::Base
   
   validates_presence_of :backlog_id, :if => Proc.new { |task| task.parent_id.nil? }
   validates_presence_of :parent_id, :if => Proc.new { |task| task.backlog_id.nil? }
-  #validates_absence_of :position, :if => :finished_at
-  #validates_absence_of :finished_at, :if => :position
+  validates_presence_of :position, :if => Proc.new { |task| task.finished_at.nil? }
+  validates_presence_of :finished_at, :if => Proc.new { |task| task.position.nil? }
   validates_presence_of :resolution, :if => :finished_at
   validates_presence_of :description, :if => :backlog_id
-
+  
   validates_size_of :description, :maximum => 80, :if => :description
   validates_size_of :customer, :maximum => 64, :if => :customer
   
@@ -121,7 +121,7 @@ class Task < ActiveRecord::Base
       parent.open if parent
     end
   end
-
+  
   def reopen
     if period.passed?
       flash[:notice] = "You cannot reopen a task in a period that is passed."
@@ -132,23 +132,46 @@ class Task < ActiveRecord::Base
     end
   end
   
-  def move_to_period(period)
-    raise "Period missing" unless period
+  def move_to_period(new_period)
+    raise "Period missing" unless new_period
     raise "Cannot move finished tasks" unless active?
+    return if new_period == period
     
-    ancestor_id = self.previous_task_id || self.id
-    new_task =   Task.find_by_period_id_and_id(period.id, ancestor_id)
-    new_task ||= Task.find_by_period_id_and_previous_task_id(period.id, ancestor_id)
-    new_task ||= Task.new
-    new_task.open
-    new_task.previous_task_id = self.previous_task_id || self.id
-    new_task.backlog = root_task.backlog
-    new_task.period = period
-    new_task.description = self.description
-    new_task.save!
-    new_task.estimate(self.todo)
-    new_task.move_to_top
-    self.finish(new_task.period.party == self.period.party ? Task::POSTPONED : Task::MOVED, true)
+    self.remove_from_list
+    self.position = nil
+    
+    if self.period
+      self.finish(new_period.party == self.period.party ? Task::POSTPONED : Task::MOVED, true)
+      ancestor_id = self.previous_task_id || self.id
+      existing_task =   Task.find_by_period_id_and_id(new_period.id, ancestor_id)
+      if existing_task ||= Task.find_by_period_id_and_previous_task_id(new_period.id, ancestor_id)
+        raise "mismatch" unless existing_task.backlog == root_task.backlog
+        raise "Mismatch" unless existing_task.period == new_period
+        existing_task.open
+        existing_task.previous_task_id = self.previous_task_id || self.id
+        existing_task.description = self.description
+        existing_task.save!
+        existing_task.estimate(self.todo)
+        existing_task.move_to_top
+        existing_task.save!
+      else
+        new_task = Task.new
+        new_task.previous_task_id = self.previous_task_id || self.id
+        new_task.backlog = root_task.backlog
+        new_task.period = new_period
+        new_task.description = self.description
+        new_task.insert_at 1
+        new_task.save!
+        new_task.estimate(self.todo)
+      end
+    else
+      self.period_id = new_period.id
+      self.period = new_period
+      self.insert_at(1)
+      self.position = 1
+      self.move_to_bottom
+      self.save!
+    end
   end
   
   def finish(resolution, save_work)

data/app/views/backlogs/_tasks.rhtml

@@ -10,11 +10,20 @@
       <% current_period = nil %>
       <% for @task in tasks %>
 	    <% if @task.period != current_period %>
-		  <tr><th colspan="6"><%=link_to (@task.period ? h(@task.period.name) : l(:unplanned_tasks)), :controller => 'periods', :action => :show, :id => @task.period%></th></tr>
-          <%= render :partial => '/tasks/fields_header', :locals => {:backlog => @backlog, :track_times => tasks.find {|t|t.period && t.period.active?}} %>
+          <%= render :partial => '/tasks/period_header', :locals => {:period => @task.period} %>
+          <%= render :partial => '/tasks/fields_header', :locals => {:backlog => @backlog, :active => true, :track_times => tasks.find {|t|t.period && t.period.active?}} %>
           <% current_period = @task.period %>
 		<% end %>
         <%= render :partial => '/tasks/task/', :locals => {:task => @task, :i => i += 1, :active => true, :highlight_task => false, :update => :maincontent} %>
       <% end %>
     </table>
   </div>
+
+  <div class="mainblock">
+    <div class="btitle">
+    	<h4><%=l(:completed_tasks) %></h4>
+  	</div>
+
+	<%=render :partial => '/tasks/completed', :locals => {:i => 1} %>
+  </div>
+

data/app/views/backlogs/edit.rhtml

@@ -10,14 +10,5 @@
 
 <div id="maincontent">
 	<%=render :partial => 'tasks', :locals => {:tasks => @tasks}%>
-
-  <div class="mainblock">
-    <div class="btitle">
-    	<h4><%=l(:completed_tasks) %></h4>
-  	</div>
-
-	<%=render :partial => '/tasks/completed', :locals => {:i => 1} %>
-  </div>
-
 </div>
 

data/app/views/periods/_show_active.rhtml

@@ -38,7 +38,7 @@ function handleEvent(field, event, id) {
   <% if task.backlog != current_backlog %>
     <%= '<tr><td>&nbsp;</td></tr>' if current_backlog %>
     <%=render :partial => '/tasks/backlog_header', :locals => { :backlog => task.backlog, :track_times => @tasks.find {|t|t.period && t.period.active?} } %>
-    <%= render :partial => '/tasks/fields_header', :locals => { :backlog => task.backlog, :track_times => @tasks.find {|t|t.period && t.period.active?} } %>
+    <%= render :partial => '/tasks/fields_header', :locals => { :backlog => task.backlog, :active => true, :track_times => @tasks.find {|t|t.period && t.period.active?} } %>
     <% current_backlog = task.backlog %>
   <% end %>
   <%=render :partial => '/tasks/task', :locals => { :task => task, :i => i, :active => true, :highlight_task => task == @selected_task, :update => :spotlight } %>

data/app/views/tasks/_completed.rhtml

@@ -1,4 +1,5 @@
 <table class="input">
+<!--
   <tr>
     <th/>
     <th align="left"><%=l :resolution %></th>
@@ -6,7 +7,14 @@
     <th></th>
     <th><%=l(:done) if (@period && @period.track_work?) || @completed_tasks.find {|t| t.track_times?} %></th>
   </tr>
+-->
+<% current_period = @completed_tasks.map(&:period).uniq.size == 1 ? @completed_tasks.first.period : nil%>
 <% for task in @completed_tasks %>
+  <% if task.period != current_period %>
+      <%= render :partial => '/tasks/period_header', :locals => {:period => task.period} %>
+      <%= render :partial => '/tasks/fields_header', :locals => {:backlog => @backlog, :active => false, :track_times => false} %>
+      <% current_period = task.period %>
+  <% end %>
   <%=render :partial => '/tasks/task', :locals => { :task => task, :i => i, :active => false, :highlight_task => task == @selected_task } %>
   <% i += 1 %>
 <% end %>

data/app/views/tasks/_fields_header.rhtml

@@ -1,6 +1,6 @@
   <tr>
     <th/>
-    <th align="center">#</th>
+    <th align="center"><%=active ? '#' : l(:resolution)%></th>
     <th><%=l :task %></th>
     <th><%=l :start if backlog.track_times? && track_times %></th>
     <th><%=l :done if backlog.track_done? %></th>

data/app/views/tasks/_form.rhtml

@@ -29,12 +29,12 @@
 <p><label for="task_description"><%=l :task%></label><br/>
 <%= text_field 'task', 'description', :size => 64, :maxlength => 80 %></p>
 
-<% if @task.period.nil? || @task.enable_customer? %>
+<% if @task.enable_customer? %>
 <p><label for="task_customer"><%=l :customer%></label><br/>
 <%= text_field 'task', 'customer', :size => 64  %></p>
 <% end %>
 
-<% if @task.period.nil? || @task.track_todo? %>
+<% if @task.track_todo? %>
 <p><label for="task_initial_estimate"><%=l :estimate%></label><br/>
 <%= text_field 'task', 'initial_estimate', :size => 4  %></p>
 <% end %>

data/app/views/tasks/_period_header.rhtml

@@ -0,0 +1 @@
+		  <tr><th colspan="6"><%=link_to (period ? h(period.name) : l(:unplanned_tasks)), :controller => 'periods', :action => :show, :id => period%></th></tr>

data/app/views/tasks/_task.rhtml

@@ -85,10 +85,10 @@
     </td>
   </tr>
 
-<% if active && (@task.period.nil? || @task.period.active_or_future?) && (not @task.work_started?) %>
-<%= draggable_element "task_#{@task.id}", :revert => true, :constraint => "'vertical'" %>
+<% if active && (@task.period.nil? || @task.period.active_or_future?) %>
+<%= draggable_element "task_#{@task.id}", :revert => true %>
 <%= drop_receiving_element "task_#{@task.id}",
-      :update => update, :url => options_for_detour({:controller => 'tasks', :action => "move_to", :period_id => @task.period_id, :position => @task.position, :layout => false}),
+      :update => update, :url => with_detour({:controller => 'tasks', :action => "move_to", :period_id => @task.period_id, :position => @task.position, :layout => false}),
       :accept => "tasks",
       :loading => "",
       :complete => "",

data/db/migrate/019_remove_unique_index_for_period_position.rb

@@ -0,0 +1,9 @@
+class RemoveUniqueIndexForPeriodPosition < ActiveRecord::Migration
+  def self.up
+    remove_index "periods", ["party_id", "position"]
+  end
+
+  def self.down
+    add_index "periods", ["position", "party_id"], :name => "index_periods_on_party_id_and_position", :unique => true
+  end
+end

data/db/schema.rb

@@ -2,7 +2,7 @@
 # migrations feature of ActiveRecord to incrementally modify your database, and
 # then regenerate this schema definition.
 
-ActiveRecord::Schema.define(:version => 18) do
+ActiveRecord::Schema.define(:version => 19) do
 
   create_table "backlogs", :force => true do |t|
     t.string  "name",            :limit => 64,                    :null => false
@@ -44,8 +44,6 @@ ActiveRecord::Schema.define(:version => 18) do
     t.integer "party_id"
   end
 
-  add_index "periods", ["position", "party_id"], :name => "index_periods_on_party_id_and_position", :unique => true
-
   create_table "tasks", :force => true do |t|
     t.integer  "position"
     t.string   "description",      :limit => 80,                                                :null => false

data/test/fixtures/periods.yml

@@ -1,4 +1,3 @@
-# Read about fixtures at http://ar.rubyonrails.org/classes/Fixtures.html
 past:
   id: 1
   party_id: 1
@@ -11,4 +10,9 @@ active:
   position: 2
   start_on: 2007-06-25
   end_on: 2907-07-08
-  
+future:
+  id: 3
+  party_id: 1
+  position: 3
+  start_on: 2907-07-09
+  end_on: 2907-07-22

data/test/fixtures/tasks.yml

@@ -5,26 +5,32 @@ first:
   backlog_id: 1
   period_id: 1
   description: first task
-  position: 0
+  position: 1
 another:
   id: 2
   created_at: 2007-06-12
   backlog_id: 1
   period_id: 2
   description: second task
-  position: 0
+  position: 1
 started:
   id: 3
   created_at: 2007-08-02 14:15:42
-  backlog_id: 1
+  backlog_id: 2
   period_id: 2
   description: third task
-  position: 1
-last:
+  position: 2
+in_last_period:
   id: 4
   created_at: 2007-08-02 14:15:42
   backlog_id: 1
-  period_id: 2
-  description: last task
-  position: 2
+  period_id: 3
+  description: task in last period
+  position: 1
+not_planned:
+  id: 5
+  created_at: 2007-08-02 14:15:42
+  backlog_id: 1
+  description: unplanned task
+  position: 1
   

data/test/functional/periods_controller_test.rb

@@ -12,6 +12,11 @@ class PeriodsControllerTest < Test::Unit::TestCase
     @request    = ActionController::TestRequest.new
     @response   = ActionController::TestResponse.new
     @request.session[:user_id] = 1000001
+    assert_sequences
+  end
+
+  def teardown
+    assert_sequences
   end
 
   def test_index
@@ -45,7 +50,7 @@ class PeriodsControllerTest < Test::Unit::TestCase
     post :create, :period => {:party_id => 1, :start_on => '2007-06-11', :end_on => '2007-06-24'}
 
     assert_response :redirect
-    assert_redirected_to :action => :show, :id => 2
+    assert_redirected_to :action => :show, :id => 3
 
     assert_equal num_periods + 1, Period.count
   end

data/test/functional/tasks_controller_test.rb

@@ -54,7 +54,8 @@ class TasksControllerTest < Test::Unit::TestCase
   def test_create
     num_tasks = Task.count
 
-    post :create, :task => {:description => 'an important task', :backlog_id => '2', :period_id => '2'}
+    post :create, :task => {:description => 'an important task', :backlog_id => '2',
+                            :period_id => '2', :position => '4'}
 
     task = assigns(:task)
     assert_equal [], task.errors.full_messages
@@ -68,7 +69,8 @@ class TasksControllerTest < Test::Unit::TestCase
   def test_create_without_period
     num_tasks = Task.count
 
-    post :create, :task => {:description => 'an important task', :backlog_id => '2'}
+    post :create, :task => {:description => 'an important task', :backlog_id => '2',
+                            :position => '1'}
 
     task = assigns(:task)
     assert_equal [], task.errors.full_messages
@@ -127,18 +129,38 @@ class TasksControllerTest < Test::Unit::TestCase
   end
   
   def test_move_to_next_period_at_end
-    before = Task.find(2)
-    assert_equal 2, before.period_id
+    before = Task.find(tasks(:in_last_period).id)
+    assert_equal tasks(:in_last_period).period_id, before.period_id
 
-    post :move_to_next_period, :id => 2
+    post :move_to_next_period, :id => tasks(:in_last_period).id
     
     assert_response :redirect
     assert_redirected_to :controller => 'periods',
                           :action => :new, 
                           :period => {:party_id => 1}
 
-    after = Task.find(2)
-    assert_equal 2, before.period_id
+    before.reload
+    assert_equal tasks(:in_last_period).period_id, before.period_id
+
+    after = Task.find(tasks(:in_last_period).id)
+    assert_equal before.period_id, after.period_id
+  end
+  
+  def test_move_to_period
+    highest = Task.find(:first, :order => 'id DESC')
+    post :move_to_period, :id => 'task_2',
+                          :detour => {:action => 'show_no_layout', :id => '2', :controller => 'periods'}, 
+                          :period_id => '3'
+    new_task = Task.find(:first, :order => 'id DESC')
+    assert new_task.id != highest.id
+    assert_equal 3, new_task.period_id
+  end
+  
+  def test_move_to_period_past
+    post :move_to_period, :id => 'task_2',
+                          :detour => {:action => 'show_no_layout', :id => '2', :controller => 'periods'}, 
+                          :period_id => '1'
+    assert_equal 2, tasks(:another).period_id
   end
   
 end

data/test/functional/works_controller_test.rb

@@ -5,113 +5,106 @@ require 'works_controller'
 class WorksController; def rescue_action(e) raise e end; end
 
 class WorksControllerTest < Test::Unit::TestCase
-  fixtures :users, :groups_users, :backlogs, :periods, :tasks, :works, :estimates
-
+  fixtures :parties, :users, :groups, :groups_users, :backlogs, :periods, :tasks, :works, :estimates
+  
   def setup
     @controller = WorksController.new
     @request    = ActionController::TestRequest.new
     @response   = ActionController::TestResponse.new
     @request.session[:user_id] = 1000001
+    assert_sequences
   end
-
+  
+  def teardown
+    assert_sequences
+  end
+  
   def test_index
     get :index
     assert_response :success
     assert_template 'list'
   end
-
+  
   def test_list
     get :list
-
+    
     assert_response :success
     assert_template 'list'
-
+    
     assert_not_nil assigns(:works)
   end
-
+  
   def test_show
     get :show, :id => 1
-
+    
     assert_response :success
     assert_template 'show'
-
+    
     assert_not_nil assigns(:work)
     assert assigns(:work).valid?
   end
-
+  
   def test_new
     get :new, :work => {:task_id => 1}
-
+    
     assert_response :success
     assert_template 'new'
-
+    
     assert_not_nil assigns(:work)
   end
-
+  
   def test_create
     num_works = Work.count
-
+    
     post :create, :work => {:task_id => 1, :started_at => Time.now.iso8601}
-
+    
     assert_response :redirect
     assert_redirected_to :controller => 'periods', :action => 'show', :id => 1, :task_id => 1
-
+    
     assert_equal num_works + 1, Work.count
   end
-
+  
   def test_edit
     get :edit, :id => 1
-
+    
     assert_response :success
     assert_template 'edit'
-
+    
     assert_not_nil assigns(:work)
     assert assigns(:work).valid?
   end
-
+  
   def test_update
     post :update, :id => 1
     assert_response :redirect
     assert_redirected_to :action => 'show', :id => 1
   end
-
+  
   def test_destroy
     assert_not_nil Work.find(1)
-
+    
     post :destroy, :id => 1
     assert_response :redirect
     assert_redirected_to :controller => 'periods', :action => 'list_work'
-
+    
     assert_raise(ActiveRecord::RecordNotFound) {
       Work.find(1)
     }
   end
-
+  
   def test_update_with_finish
     num_tasks = Task.count
     num_open_tasks = Task.find_open.size
     task = tasks(:started)
     work = works(:started)
-
+    
     post :update, "commit"=>"Lagre", "action"=>"update", "id"=>work.id.to_s, "controller"=>"works", "estimate"=>{"todo"=>"0"}, "work"=>{"completed_at"=>"2007-08-02 14:15", "task_id"=> task.id.to_s, "user_id"=>"1000001", "started_at"=>"2007-08-02 14:00"}
     
     assert_response :redirect
     assert_redirected_to :controller => 'periods', :action => :show, :id => task.period, :task => nil
-
+    
     assert_equal num_tasks, Task.count
     assert_equal num_open_tasks - 1, Task.find_open.size
-    assert_sequence
-  end
-
-  
-  private
-  
-  def assert_sequence
-    Period.find(:all).each do |p|
-      p.open_tasks.each_with_index do |t, i|
-        assert_equal i, t.position
-      end
-    end
   end
   
 end

data/test/test_helper.rb

@@ -17,18 +17,38 @@ class Test::Unit::TestCase
   # don't care one way or the other, switching from MyISAM to InnoDB tables
   # is recommended.
   self.use_transactional_fixtures = true
-
+  
   # Instantiated fixtures are slow, but give you @david where otherwise you
   # would need people(:david).  If you don't want to migrate your existing
   # test cases which use the @david style and don't mind the speed hit (each
   # instantiated fixtures translates to a database query per test method),
   # then set this back to true.
   self.use_instantiated_fixtures  = false
-
+  
   # Add more helper methods to be used by all tests here...
-
+  
   def add_stored_detour
     @request.session[:detours] = [{:controller => 'bogus', :action => :location}]
   end
-
+  
+  def assert_sequences
+    Backlog.find(:all).each do |b|
+      b.unplanned_tasks.each_with_index do |t, i|
+        assert_equal i+1, t.position
+      end
+    end
+    
+    Party.find(:all).each do |p|
+      p.periods.each_with_index do |period, i|
+        assert_equal i+1, period.position
+      end
+    end
+    
+    Period.find(:all).each do |p|
+      p.open_tasks.each_with_index do |t, i|
+        assert_equal i+1, t.position
+      end
+    end
+  end
+  
 end

metadata

@@ -3,8 +3,8 @@ rubygems_version: 0.9.4
 specification_version: 1
 name: backlog
 version: !ruby/object:Gem::Version 
-  version: 0.6.6
-date: 2007-08-07 00:00:00 +02:00
+  version: 0.7.0
+date: 2007-08-14 00:00:00 +02:00
 summary: Application to aid collecting, processing, organizing, reviewing and doing tasks.
 require_paths: 
 - lib
@@ -114,6 +114,7 @@ files:
 - db/migrate
 - db/migrate/008_add_backlog_options.rb
 - db/migrate/014_add_customer_option.rb
+- db/migrate/019_remove_unique_index_for_period_position.rb
 - db/migrate/011_login_sugar.rb
 - db/migrate/010_add_work_start.rb
 - db/migrate/017_insert_datek_projects.rb
@@ -226,7 +227,6 @@ files:
 - config/environments/datek_production.rb
 - LICENSE_LOCALIZATION
 - README.txt
-- doc
 - Manifest.txt
 - vendor
 - vendor/plugins
@@ -1730,6 +1730,7 @@ files:
 - app/views/tasks/_backlog_header.rhtml
 - app/views/tasks/list.rhtml
 - app/views/tasks/_form.rhtml
+- app/views/tasks/_period_header.rhtml
 - app/views/tasks/new.rhtml
 - app/views/tasks/_fields_header.rhtml
 - app/views/tasks/_completed.rhtml