actually remove links, not only hiding them

git-svn-id: https://dev.finn.de/svn/cockpit/branches/reporting_merge@1631 7926756e-e54e-46e6-9721-ed318f58905e

app/views/hooks/_view_projects_show_sidebar_bottom_hook.rhtml

@@ -3,8 +3,8 @@
 remove_old_sidebar = function() {
   if ($($("sidebar").down()).innerHTML === "<%= l(:label_spent_time) %>") {
     var reporting_link, old_html;
-    // The sidebar is showing spent time, hide the links, which are in the second para
-    $("sidebar").down().siblings()[1].hide();
+    // The sidebar is showing spent time, remove the links, which are in the second para
+    $("sidebar").down().siblings()[1].remove();
     // Make the hours a link, which are in the first para
     reporting_link = "<%= link_to('PLACEHOLDER', {
       :controller => 'cost_reports', :project_id => project }).gsub('"', "'") %>";

features/links.feature

@@ -101,7 +101,7 @@ Feature: Cost Reporting Linkage
     And I should not see "Details" within "#sidebar"
     And I should not see "Report" within "#sidebar"
     When I follow "1.00 hour"
-    Then I should be on "/projects/Standard%20Project/cost_reports"
+    Then I should be on the Cost Reports page for the project called "Standard Project"
 
   @javascript
   Scenario: Jump to project from the cost report jumps to the cost report of the selected project