Changeset 80

Timestamp:

02/07/08 14:34:03 (10 months ago)

Author:

jweiss

Message:

Better handling of repeat task by also repeating the description

Location:

trunk

Files:

• CHANGELOG.txt (modified) (1 diff)
• app/controllers/deployments_controller.rb (modified) (1 diff)
• app/models/deployment.rb (modified) (1 diff)
• app/views/deployments/_status.html.erb (modified) (1 diff)
• app/views/projects/dashboard.html.erb (modified) (1 diff)
• app/views/stages/_deployments.html.erb (modified) (1 diff)
• app/views/users/_deployments.html.erb (modified) (1 diff)
• test/unit/deployment_test.rb (modified) (1 diff)

trunk/CHANGELOG.txt

@@ -1 +1 @@
 
 * SVN *
+
+* Better handling of `repeat` task by also repeating the description
 
 * Moved ActionController session key to webistrano_config.rb

trunk/app/controllers/deployments_controller.rb

@@ -31 +31 @@
     @deployment = @stage.deployments.new
     @deployment.task = params[:task]
+    
+    if params[:repeat]
+      @original = @stage.deployments.find(params[:repeat])
+      @deployment = @original.repeat
+    end
   end
 

trunk/app/models/deployment.rb

@@ -102 +102 @@
     end
   end
+  
+  # returns an unsaved, new deployment with the same task/stage/description
+  def repeat
+    returning Deployment.new do |d|
+      d.stage = self.stage
+      d.task = self.task
+      d.description = "Repetition of deployment #{self.id}:\n\n" 
+      d.description += self.description
+    end
+  end
 end

trunk/app/views/deployments/_status.html.erb

@@ -35 +35 @@
         <% if @deployment.completed? %>
         <p>
-          <%= link_to "Repeat", new_project_stage_deployment_path(current_project, current_stage) + '?task=' + h(@deployment.task), :class => 'arrow_link'  %>
+          <%= link_to "Repeat", new_project_stage_deployment_path(current_project, current_stage) + '?repeat=' + h(@deployment.id), :class => 'arrow_link'  %>
         </p>
         <% end %>

trunk/app/views/projects/dashboard.html.erb

@@ -30 +30 @@
         <td>
           <% if deployment.completed? %>
-            <%= link_to "Repeat", new_project_stage_deployment_path(deployment.stage.project, deployment.stage) + '?task=' + h(deployment.task)  %>
+            <%= link_to "Repeat", new_project_stage_deployment_path(deployment.stage.project, deployment.stage) + '?repeat=' + h(deployment.id)  %>
           <% end %>
         </td>

trunk/app/views/stages/_deployments.html.erb

@@ -21 +21 @@
       <td>
         <% if deployment.completed? %>
-          <%= link_to "Repeat", new_project_stage_deployment_path(current_project, @stage) + '?task=' + h(deployment.task)  %>
+          <%= link_to "Repeat", new_project_stage_deployment_path(current_project, @stage) + '?repeat=' + h(deployment.id)  %>
         <% end %>
       </td>

trunk/app/views/users/_deployments.html.erb

@@ -24 +24 @@
       <td>
         <% if deployment.completed? %>
-          <%= link_to "Repeat", new_project_stage_deployment_path(deployment.stage.project, deployment.stage) + '?task=' + h(deployment.task)  %>
+          <%= link_to "Repeat", new_project_stage_deployment_path(deployment.stage.project, deployment.stage) + '?repeat=' + h(deployment.id)  %>
         <% end %>
       </td>

trunk/test/unit/deployment_test.rb

@@ -218 +218 @@
   end
   
+  def test_repeat
+    original = create_new_deployment(:stage => @stage, :description => 'this is foo', :task => 'foo:bar')
+    
+    repeater = original.repeat
+    
+    assert_equal original.task, repeater.task
+    assert_equal "Repetition of deployment #{original.id}:\n\n#{original.description}", repeater.description
+  end
+  
 end