backlog 0.10.6 → 0.10.7

data/History.txt

@@ -1,3 +1,12 @@
+== 0.10.7 2007-09-26
+
+=== Fixes
+
+* Fixed bug when postponing tasks
+* Changed to propagate messages during redirects to front page.
+* Fixed bugs in listing work for sprint.
+* Made daily work sheet not fail with subtasks.  It still does not work properly.
+
 == 0.10.6 2007-09-25
 
 === Fixes

data/app/controllers/backlogs_controller.rb

@@ -6,6 +6,7 @@ class BacklogsController < ApplicationController
   :redirect_to => { :action => :index }
   
   def index
+    flash.keep
     if Task.find_started.size > 0
       redirect_to :controller => 'tasks', :action => :list_started
       return

data/app/controllers/periods_controller.rb

@@ -10,6 +10,7 @@ class PeriodsController < ApplicationController
     else
       most_urgent_period = active_periods.sort_by {|p| p.required_speed}.last
     end
+    flash.keep
     redirect_to :action => 'show', :id => most_urgent_period
   end
   
@@ -103,9 +104,13 @@ class PeriodsController < ApplicationController
   end
   
   def list_work
-    @period = Period.find(params[:id])
-    @works = @period.tasks.map {|t| t.works}.flatten.sort_by {|w| w.completed_at}
-    render :template => '/works/list'
+    if @period = Period.find_by_id(params[:id])
+      @works = @period.tasks.map {|t| t.works}.flatten.sort_by {|w| [w.completed_at || Time.now.midnight, w.started_at]}
+      render :template => '/works/list'
+    else
+      flash[:notice] = "You need to provide a period ID"
+      redirect_to ''
+    end
   end
   
   def make_link

data/app/models/task.rb

@@ -160,7 +160,6 @@ class Task < ActiveRecord::Base
         new_task.period = new_period
         new_task.description = self.description
         new_task.initial_estimate = self.initial_estimate
-        new_task.save!
         new_task.insert_at 1
         new_task.position = 1
         new_task.save!

data/app/views/works/daily_work_sheet.rhtml

@@ -9,9 +9,12 @@
 <div style="float: right"><%=link_to(image_tag('arrow_right.png'), :id => (@date + 7))%></div>
 <div style="float: right"><%=link_to(image_tag('arrow_right.png'), :id => @date+1)%></div>
 
+<br clear="all" />
+
 <table sstyle="width: 100%" border="0">
   <tr>
     <td/>
+    <th><%=l :backlog %></th>
     <th><%=l :task %></th>
     <% if track_times %>
     <th><%=l :started_at if track_times %></th>
@@ -29,7 +32,7 @@
     </td>
     <td>
       <form>
-      <%=text_field :work, :backlog_name, :id => "work_#{@work.id}_backlog_name", :value => @work.task.backlog.name, :size => 16 %>
+      <%=text_field :work, :backlog_name, :id => "work_#{@work.id}_backlog_name", :value => @work.task.root_task.backlog.name, :size => 16 %>
       </form>
     </td>
     <td>

data/test/fixtures/works.yml

@@ -16,3 +16,8 @@ personal:
   completed_at: 2007-06-12T14:35:00
   user_id: 1000001
   hours: 40.0
+brother_of_started:
+  id: 5
+  task_id: 4
+  started_at: 2007-06-12T13:35:00
+  completed_at: 2007-06-12T14:35:00

data/test/functional/periods_controller_test.rb

@@ -99,4 +99,26 @@ class PeriodsControllerTest < Test::Unit::TestCase
     get :burn_down_chart, :id => 1
   end
   
+  def test_list_work_no_work
+    get :list_work, :id => '1'
+    assert_equal [], assigns(:works)
+  end
+  
+  def test_list_work_some_work
+    get :list_work, :id => '2'
+    assert_equal 2, assigns(:works).size
+  end
+  
+  def test_list_work_with_started_work
+    get :list_work, :id => '3'
+    assert_equal 2, assigns(:works).size
+  end
+  
+  def test_list_work_no_id
+    get :list_work
+    
+    assert_response :redirect
+    assert_redirected_to ''
+  end
+  
 end

metadata

@@ -3,8 +3,8 @@ rubygems_version: 0.9.4
 specification_version: 1
 name: backlog
 version: !ruby/object:Gem::Version 
-  version: 0.10.6
-date: 2007-09-25 00:00:00 +02:00
+  version: 0.10.7
+date: 2007-09-26 00:00:00 +02:00
 summary: Application to aid collecting, processing, organizing, reviewing and doing tasks.
 require_paths: 
 - lib