backlog 0.12.0 → 0.12.1

data/History.txt

@@ -1,3 +1,18 @@
+== 0.12.1 2007-11-07
+
+===Features
+
+* Made "Move to next sprint" and "Abort" functions use ajax to speed up response.
+* Added scaffold for Customers.
+
+=== Fixes
+
+* Fixed some errors in the Show Sprint view.
+
+=== Known bugs
+
+* Finishing or reopening tasks makes remaining tasks show wrong priority number.
+
 == 0.12.0 2007-11-06
 
 ===Features

data/app/controllers/application.rb

@@ -74,6 +74,16 @@ class ApplicationController < ActionController::Base
     redirect_to(options)
   end
   
+  def rjs_detour_to(options)
+    store_detour(params)
+    rjs_redirect_to(options)
+  end
+  
+  def rjs_redirect_to(options)
+    @options = options
+    render :template => 'redirect', :layout => false
+  end
+  
   def store_detour(options)
     if session[:detours] && session[:detours].last == options
       logger.debug "duplicate detour: #{options}"
@@ -177,4 +187,11 @@ class ApplicationController < ActionController::Base
       User.find_by_id(user_id) if user_id
     end
     
+    def setup_remove
+      @last_active = @task.higher_item.nil? && @task.lower_item.nil? 
+      @last_active_in_backlog = @last_active || ((@task.higher_item.nil? || @task.higher_item.backlog != @task.backlog) && (@task.lower_item.nil? || @task.lower_item.backlog != @task.backlog))
+      finished_count = Task.count(:conditions => ['period_id = ? AND finished_at IS NOT NULL', @task.period_id])
+      @first_finished = finished_count == 0
+    end
+        
   end

data/app/controllers/customers_controller.rb

@@ -0,0 +1,51 @@
+class CustomersController < ApplicationController
+  def index
+    list
+    render :action => 'list'
+  end
+
+  # GETs should be safe (see http://www.w3.org/2001/tag/doc/whenToUseGet.html)
+  verify :method => :post, :only => [ :destroy, :create, :update ],
+         :redirect_to => { :action => :list }
+
+  def list
+    @customer_pages, @customers = paginate :customers, :per_page => 10
+  end
+
+  def show
+    @customer = Customer.find(params[:id])
+  end
+
+  def new
+    @customer = Customer.new
+  end
+
+  def create
+    @customer = Customer.new(params[:customer])
+    if @customer.save
+      flash[:notice] = 'Customer was successfully created.'
+      redirect_to :action => 'list'
+    else
+      render :action => 'new'
+    end
+  end
+
+  def edit
+    @customer = Customer.find(params[:id])
+  end
+
+  def update
+    @customer = Customer.find(params[:id])
+    if @customer.update_attributes(params[:customer])
+      flash[:notice] = 'Customer was successfully updated.'
+      redirect_to :action => 'show', :id => @customer
+    else
+      render :action => 'edit'
+    end
+  end
+
+  def destroy
+    Customer.find(params[:id]).destroy
+    redirect_to :action => 'list'
+  end
+end

data/app/controllers/estimates_controller.rb

@@ -1,4 +1,6 @@
 class EstimatesController < ApplicationController
+  helper :tasks
+  
   verify :method => :post, :redirect_to => ''
   
   def create
@@ -16,10 +18,7 @@ class EstimatesController < ApplicationController
   
   def create_ajax
     if @task = Task.find_by_id(params[:id])
-      @last_active = @task.lower_item.nil?
-      @last_active_in_backlog = @last_active || @task.lower_item.backlog != @task.backlog
-      finished_count = Task.count(:conditions => ['period_id = ? AND finished_at IS NOT NULL', @task.period_id])
-      @first_finished = finished_count == 0
+      setup_remove
     end
     @success, @message = do_create
     # Only necessary since we have an unecessary complex update of estimates

data/app/controllers/tasks_controller.rb

@@ -17,6 +17,7 @@ class TasksController < ApplicationController
     @task.backlog_id ||= @task.period && @task.period.most_frequent_backlog
     @backlogs = Backlog.find(:all, :order => 'name')
     @periods = Period.find_active_or_future
+    @customers = Customer.find(:all)
   end
   
   def create
@@ -52,6 +53,7 @@ class TasksController < ApplicationController
     @task.period_id = params[:task][:period_id] if params[:task] && params[:task][:period_id]
     @periods = Period.find_active_or_future
     @backlogs = Backlog.find(:all, :order => 'name')
+    @customers = Customer.find(:all)
   end
   
   def update
@@ -185,59 +187,47 @@ class TasksController < ApplicationController
     back_or_redirect_to :controller => 'periods', :action => :show_no_layout, :id => task.period_id, :task => task.id
   end
   
-  def move_to_period(with_layout = false)
+  def move_to_period
     params[:id] = $1 if params[:id] =~ /^task_(\d*)$/
-    task = Task.find(params[:id])
+    @task = Task.find(params[:id])
     if request.post?
       period = params[:period_id] && Period.find(params[:period_id])
       if period && period.active_or_future?
-        next_task = task.higher_item ? task.higher_item : task.lower_item ? task.lower_item : task
-        task.move_to_period period
-        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_no_layout, :id => next_task.period, :task_id => next_task.id
-        end
+        next_task = @task.higher_item ? @task.higher_item : @task.lower_item ? @task.lower_item : @task
+        setup_remove
+        @task.move_to_period period
+        render :action => :move_to_period, :layout => false
       else
-        detour_to :controller => 'periods', :action => :new, :period => {:party_id => task.period.party_id}, :layout => with_layout && determine_layout
+        rjs_detour_to :controller => 'periods', :action => :new, :period => {:party_id => @task.period.party_id}, :layout => false
       end
     else
-      redirect_to :controller => 'periods', :action => :show, :id => task.period, :task_id => task.id, :layout => with_layout && determine_layout
+      redirect_to :controller => 'periods', :action => :show, :id => @task.period, :task_id => @task.id, :layout => false
     end
   end
   
   def move_to_next_period
-    task = Task.find(params[:id])
-    if task.period
-      next_period = task.period.lower_item
+    @task = Task.find(params[:id])
+    if @task.period
+      next_period = @task.period.lower_item
       next_period = next_period.lower_item while next_period && next_period.passed?
       params[:period_id] = next_period && next_period.id
-      move_to_period(true)
+      move_to_period
     else
-      redirect_to :action => :edit, :id => task
+      redirect_to :action => :edit, :id => @task
     end
   end
   
   def finish
     @task = Task.find(params[:id])
-    @task.finish(Task::COMPLETED, true)
-    back_or_redirect_to :controller => 'periods', :action => :show, :id => @task.period, :task_id => @task.id
-  end
-  
-  def finish_ajax
-    @task = Task.find(params[:id])
-    lower_item = @task.lower_item
-    @last_active = @task.lower_item.nil?
-    @last_active_in_backlog = @last_active || @task.lower_item.backlog != @task.backlog
-    finished_count = Task.count(:conditions => ['period_id = ? AND finished_at IS NOT NULL', @task.period_id])
-    @first_finished = finished_count == 0
+    setup_remove
     @task.finish(Task::COMPLETED, true)
   end
   
   def abort
-    task = Task.find params[:id]
-    task.abort
-    back_or_redirect_to(:controller => "periods", :action => "show", :id => task.period)
+    @task = Task.find params[:id]
+    setup_remove
+    @task.abort
+    render :action => :finish
   end
   
   def reopen
@@ -246,13 +236,11 @@ class TasksController < ApplicationController
     @task.reload
     finished_count = Task.count(:conditions => ['period_id = ? AND finished_at IS NOT NULL', @task.period_id])
     @last_finished = finished_count == 0
-#    back_or_redirect_to :controller => 'periods', :action => :show, :id => @task.period, :task => @task.id
   end
   
   def start_work
     @task = Task.find(params[:id])
     @task.start_work
-#    back_or_redirect_to :controller => 'periods', :action => :show, :id => task.period, :task => task.id
   end
   
   def end_work

data/app/helpers/application_helper.rb

@@ -38,13 +38,13 @@ module ApplicationHelper
     image_options.update :alt => title, :title => title
     link_to image_tag(image_source, image_options), url_options, post ? {:method => :post} : nil
   end
-
+  
   def image_link_to_remote(image_source, title, url_options, image_options = nil, post = false)
     image_options ||= {:class => 'image-submit'}
     image_options.update :alt => title, :title => title
     link_to_remote image_tag(image_source, image_options), {:url => url_options}, post ? {:method => :post} : nil
   end
-
+  
   def detour?
     not session[:detours].nil?
   end
@@ -85,4 +85,13 @@ module ApplicationHelper
     image_tag image_file, :title => l(@task.resolution.downcase)
   end
   
+  def display_notice(page)
+    if @message
+      page.replace_html :notice, @message
+      page.visual_effect(:appear, :notice)
+    else
+      page.visual_effect(:fade, :notice)
+    end
+  end
+  
 end

data/app/helpers/customers_helper.rb

@@ -0,0 +1,2 @@
+module CustomersHelper
+end

data/app/helpers/tasks_helper.rb

@@ -1,4 +1,73 @@
 module TasksHelper
+  def remove_active_task(page)
+    page.visual_effect :blind_up, "task_#{@task.id}"
+    if @last_active_in_backlog
+      page["task_#{@task.id}"].ancestors.first.ancestors.first.remove
+    else
+      page.remove "task_#{@task.id}"
+    end
+    
+    if @last_active
+      page.visual_effect :blind_up, "active_tasks"
+      page.visual_effect :appear, "no_tasks_message"
+    end
+  end
+  
+  def remove_finished_task(page)
+    page.visual_effect :fade, "task_#{@task.id}"
+    page.remove "task_#{@task.id}"
+    
+    if @last_finished
+      page.select('#completed_tasks tr').first.remove
+    end
+  end
+  
+  def add_active_task(page)
+    if @task.lower_item.nil? || @task.lower_item.backlog != @task.backlog
+      page.insert_html :top, :active_tasks, '<table class="input" style="width: 100%;"/>'
+    elsif @task.lower_item
+      page.select('#active_tasks table tr').first.remove
+      page.select('#active_tasks table tr').first.remove
+    end
+    
+    insert(page, render(:partial => "/tasks/task", :locals => { :task => @task, :i => 1, :active => true, :highlight_task => false, :update => :spotlight, :hidden => true }), '#active_tasks table')
+    
+    fields_header = render(:partial => '/tasks/fields_header', :locals => { :backlog => @task.backlog, :active => true } )
+    insert(page, fields_header, '#active_tasks table')
+    insert(page, render(:partial => "/tasks/backlog_header", :locals => {:backlog => @task.backlog}), '#active_tasks table')
+    
+    page[:no_tasks_message].hide
+    
+    if @task.lower_item.nil?
+      page.visual_effect :appear, "active_tasks"
+    end
+    page.visual_effect :appear, "task_#{@task.id}"
+  end
+  
+  def add_finished_task(page)
+    unless @first_finished
+      page.select('#completed_tasks tr').first.remove
+    end
+    
+    page.insert_html :top, :completed_tasks, :partial => '/tasks/task', :locals => {:active => false, :hidden => true, :highlight_task => false}
+    page.visual_effect :appear, "task_#{@task.id}"
+    
+    page.insert_html :top, :completed_tasks, :partial => '/tasks/fields_header', :locals => {:backlog => @task.backlog, :active => false}
+  end
+  
+  def update_burn_down_chart(page)
+    page['burn_down_chart'].src = url_for(:controller => 'periods', :action => :burn_down_chart_thumbnail, :id => @task.period_id, :rnd => rand)
+  end
+  
+  def record(page, script)
+    page.call("#{script};//")
+  end
+  
+  def insert(page, content, selector, position = :top)
+    escaped_content = content.gsub("\n", '').gsub("'", "\\\\'")
+    record(page, "new Insertion.#{position.to_s.capitalize}($$('#{selector}').first(), '#{escaped_content}')")
+  end
+  
 end
 
 class Time

data/app/views/customers/_form.rhtml

@@ -0,0 +1,7 @@
+<%= error_messages_for 'customer' %>
+
+<!--[form:customer]-->
+<p><label for="customer_name">Name</label><br/>
+<%= text_field 'customer', 'name'  %></p>
+<!--[eoform:customer]-->
+

data/app/views/customers/edit.rhtml

@@ -0,0 +1,9 @@
+<h1>Editing customer</h1>
+
+<% form_tag :action => 'update', :id => @customer do %>
+  <%= render :partial => 'form' %>
+  <%= submit_tag 'Edit' %>
+<% end %>
+
+<%= link_to 'Show', :action => 'show', :id => @customer %> |
+<%= link_to 'Back', :action => 'list' %>

data/app/views/customers/list.rhtml

@@ -0,0 +1,27 @@
+<h1>Listing customers</h1>
+
+<table>
+  <tr>
+  <% for column in Customer.content_columns %>
+    <th><%= column.human_name %></th>
+  <% end %>
+  </tr>
+  
+<% for customer in @customers %>
+  <tr>
+  <% for column in Customer.content_columns %>
+    <td><%=h customer.send(column.name) %></td>
+  <% end %>
+    <td><%= link_to 'Show', :action => 'show', :id => customer %></td>
+    <td><%= link_to 'Edit', :action => 'edit', :id => customer %></td>
+    <td><%= link_to 'Destroy', { :action => 'destroy', :id => customer }, :confirm => 'Are you sure?', :method => :post %></td>
+  </tr>
+<% end %>
+</table>
+
+<%= link_to 'Previous page', { :page => @customer_pages.current.previous } if @customer_pages.current.previous %>
+<%= link_to 'Next page', { :page => @customer_pages.current.next } if @customer_pages.current.next %> 
+
+<br />
+
+<%= link_to 'New customer', :action => 'new' %>

data/app/views/customers/new.rhtml

@@ -0,0 +1,8 @@
+<h1>New customer</h1>
+
+<% form_tag :action => 'create' do %>
+  <%= render :partial => 'form' %>
+  <%= submit_tag "Create" %>
+<% end %>
+
+<%= link_to 'Back', :action => 'list' %>

data/app/views/customers/show.rhtml

@@ -0,0 +1,8 @@
+<% for column in Customer.content_columns %>
+<p>
+  <b><%= column.human_name %>:</b> <%=h @customer.send(column.name) %>
+</p>
+<% end %>
+
+<%= link_to 'Edit', :action => 'edit', :id => @customer %> |
+<%= link_to 'Back', :action => 'list' %>

data/app/views/periods/_show_active.rhtml

@@ -32,19 +32,22 @@ function handleEvent(field, event, id) {
 </script>
 
 
-<table id="active_tasks" class="input"<%=' style="display: none;"' unless @tasks and not @tasks.empty?%>>
-<% current_backlog = nil %>
-<% for task in @tasks %>
-  <% 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, :active => true, :track_times => @tasks.find {|t|t.period && t.period.active?} } %>
+<div id="active_tasks"<%=' style="display: none;"' unless @tasks and not @tasks.empty?%>>
+<% current_backlog = nil -%>
+<% for task in @tasks -%>
+  <% if task.backlog != current_backlog -%>
+    <%='</table>' if current_backlog%>
     <% current_backlog = task.backlog %>
-  <% end %>
+    <table class="input" style="width: 100%;">
+      <%= '<tr><td>&nbsp;</td></tr>' if false && 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, :active => true, :track_times => @tasks.find {|t|t.period && t.period.active?} } %>
+  <% end -%>
   <%=render :partial => '/tasks/task', :locals => { :task => task, :i => i, :active => true, :highlight_task => task == @selected_task, :update => :spotlight, :hidden => false } %>
   <% i += 1 %>
-<% end %>
-</table>
+<% end -%>
+    <%='</table>' if current_backlog%>
+</div>
 
 <p id="no_tasks_message"<%=' style="display: none;"' if @tasks and not @tasks.empty?%>><%=l :no_pending_tasks%></p>
 

data/app/views/redirect.rjs

@@ -0,0 +1 @@
+page.redirect_to @options