diff --git a/app/views/timelog/edit.rhtml b/app/views/timelog/edit.rhtml index c403d8f2519dfc11e22610cc2cd1873aba19cff9..085f3d805b98741ee80328f5110c42368f8baffd 100644 --- a/app/views/timelog/edit.rhtml +++ b/app/views/timelog/edit.rhtml @@ -1,6 +1,6 @@ <h2><%= l(:label_spent_time) %></h2> -<% labelled_tabular_form_for :time_entry, @time_entry, :url => {:action => 'edit', :project_id => @time_entry.project} do |f| %> +<% labelled_tabular_form_for :time_entry, @time_entry, :url => {:action => 'edit', :id => @time_entry, :project_id => @time_entry.project} do |f| %> <%= error_messages_for 'time_entry' %> <%= back_url_hidden_field_tag %> diff --git a/test/functional/timelog_controller_test.rb b/test/functional/timelog_controller_test.rb index f2d0f3141447ecba839c22601fffe66a1fb3731f..46c3f3ef80ac8e5ad0cc8b656a24b29836eb29e0 100644 --- a/test/functional/timelog_controller_test.rb +++ b/test/functional/timelog_controller_test.rb @@ -62,6 +62,15 @@ class TimelogControllerTest < Test::Unit::TestCase :content => 'Development' end + def test_get_edit_existing_time + @request.session[:user_id] = 2 + get :edit, :id => 2, :project_id => nil + assert_response :success + assert_template 'edit' + # Default activity selected + assert_tag :tag => 'form', :attributes => { :action => '/projects/ecookbook/timelog/edit/2' } + end + def test_post_edit # TODO: should POST to issues’ time log instead of project. change form # and routing