[#4181] Redirect DeliverablesController list and show to the Contract page.

app/controllers/deliverables_controller.rb

@@ -6,12 +6,20 @@ class DeliverablesController < InheritedResources::Base
   before_filter :find_contract
   before_filter :authorize
 
+  def index
+    redirect_to contract_url(@project, @contract)
+  end
+
   def create
     @deliverable = begin_of_association_chain.deliverables.build(params[:deliverable])
     @deliverable.type = 'FixedDeliverable'
     create! { contract_url(@project, @contract) }
   end
 
+  def show
+    redirect_to contract_url(@project, @contract)
+  end
+
   def destroy
     destroy! { contract_url(@project, @contract) }
   end

init.rb

@@ -16,7 +16,7 @@ Redmine::Plugin.register :redmine_contracts do
   project_module :contracts do
     permission(:manage_budget, {
                  :contracts => [:index, :new, :create, :show, :edit, :update, :destroy],
-                 :deliverables => [:new, :create, :destroy]
+                 :deliverables => [:index, :new, :create, :show, :destroy]
                }, :public => true)
   end
 

test/integration/deliverables_list_test.rb

@@ -0,0 +1,19 @@
+require 'test_helper'
+
+class DeliverablesListTest < ActionController::IntegrationTest
+  include Redmine::I18n
+
+  def setup
+    @project = Project.generate!(:identifier => 'main')
+    @contract = Contract.generate!(:project => @project)
+    @manager = User.generate!
+    @deliverable = FixedDeliverable.generate!(:contract => @contract, :manager => @manager)
+  end
+
+  should "redirect to the contract page" do
+    visit "/projects/#{@project.identifier}/contracts/#{@contract.id}/deliverables/"
+    assert_response :success
+    assert_template 'contracts/show'
+    
+  end
+end

test/integration/deliverables_show_test.rb

@@ -0,0 +1,19 @@
+require 'test_helper'
+
+class DeliverablesShowTest < ActionController::IntegrationTest
+  include Redmine::I18n
+  
+  def setup
+    @project = Project.generate!(:identifier => 'main')
+    @contract = Contract.generate!(:project => @project)
+    @manager = User.generate!
+    @deliverable = FixedDeliverable.generate!(:contract => @contract, :manager => @manager)
+  end
+
+  should "redirect to the contract page" do
+    visit "/projects/#{@project.identifier}/contracts/#{@contract.id}/deliverables/#{@deliverable.id}"
+    assert_response :success
+    assert_template 'contracts/show'
+    
+  end
+end