Project

General

Profile

Actions

Defect #35134

closed

Change total spent time link to global time entries when issue has subtasks that can be on non descendent projects

Added by Dmitry Makurin over 3 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Normal
Category:
Issues
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

Parent issue displays total spent hours. It includes self time entries and all of subtasks and also gives user a link to time_entries controller. Link leads to time_entries inside a parent issue project so if any of subtask will be in different project than parent you won't see it.

Issue.total_spent_hours does not filter time entries by a project.

I guess it's more natural to give link to time_entries instead projects/:id/time_entries.

Index: app/helpers/issues_helper.rb
<+>UTF-8
===================================================================
diff --git a/app/helpers/issues_helper.rb b/app/helpers/issues_helper.rb
--- a/app/helpers/issues_helper.rb    (revision ad087516f4f11bb530ea96777e3edc32940e323b)
+++ b/app/helpers/issues_helper.rb    (revision 3c8f6a9152e28c5057710f68b13559db56990dbf)
@@ -259,7 +259,7 @@

   def issue_spent_hours_details(issue)
     if issue.total_spent_hours > 0
-      path = project_time_entries_path(issue.project, :issue_id => "~#{issue.id}")
+      path = time_entries_path(:issue_id => "~#{issue.id}")

       if issue.total_spent_hours == issue.spent_hours
         link_to(l_hours_short(issue.spent_hours), path)


Files

Actions #1

Updated by Marius BĂLTEANU over 3 years ago

  • Assignee set to Marius BĂLTEANU
  • Target version set to 4.1.4
Actions #2

Updated by Marius BĂLTEANU over 3 years ago

  • Status changed from New to Needs feedback
  • Assignee deleted (Marius BĂLTEANU)

Indeed, giving link to time_entries instead projects/:id/time_entries fixes this, but with a downside, the user context is changed from project level to global level.

What about setting the link to time_entries only when the "Allow cross-projects subtasks" is set to "With all projects"? If I'm not wrong, this is the only case when this issue reproduces.

Actions #3

Updated by Marius BĂLTEANU over 3 years ago

  • Assignee set to Marius BĂLTEANU
Actions #4

Updated by Dmitry Makurin over 3 years ago

Marius BALTEANU wrote:

What about setting the link to time_entries only when the "Allow cross-projects subtasks" is set to "With all projects"? If I'm not wrong, this is the only case when this issue reproduces.

Yes, it is. Agreed that is a better approach, I'll try to redo the patch.

Actions #6

Updated by Marius BĂLTEANU over 3 years ago

The implementation links to global path instead of project path only when:
  • the total spent time is greater than spent time (means that the issue has subtasks with time entries logged) and
  • setting "Allow cross-project subtasks" is "With all projects", "With project tree" or "With project hierarchy".
Actions #7

Updated by Marius BĂLTEANU about 3 years ago

  • Status changed from Needs feedback to Resolved

Patch committed.

Actions #8

Updated by Marius BĂLTEANU about 3 years ago

  • Resolution set to Fixed
Actions #9

Updated by Marius BĂLTEANU about 3 years ago

  • Status changed from Resolved to Closed
Actions #10

Updated by Go MAEDA about 3 years ago

  • Status changed from Closed to Reopened

IssuesHelperTest in 4.2-stable and 4.1-stable branches fails with Ruby 2.4.

laphroaig:4.2-stable maeda$ ruby test/helpers/issues_helper_test.rb
Run options: --seed 4533

# Running:

....F

Failure:
IssuesHelperTest#test_issue_spent_hours_details_should_link_to_project_time_entries_depending_on_cross_project_setting [test/helpers/issues_helper_test.rb:385]:
Expected /href="\/projects\/ecookbook\/time_entries\?issue_id=~1"/ to match "154.25 h (Total: <a href=\"/projects/ecookbook/time_entries?issue_id=%7E1\">161.25 h</a>)".

bin/rails test test/helpers/issues_helper_test.rb:379

...........................F

Failure:
IssuesHelperTest#test_issue_spent_hours_details_should_link_to_global_time_entries_depending_on_cross_project_setting [test/helpers/issues_helper_test.rb:396]:
Expected /href="\/time_entries\?issue_id=~1"/ to match "154.25 h (Total: <a href=\"/time_entries?issue_id=%7E1\">161.25 h</a>)".

bin/rails test test/helpers/issues_helper_test.rb:390
Actions #11

Updated by Marius BĂLTEANU about 3 years ago

Thanks for reporting the failures, I've committed the fixes.

Considering that the issue reproduces only on Ruby 2.4 which is no longer supported on trunk, is it ok to fix the tests only on 4.2-stable and 4.1-stable branches?

Actions #12

Updated by Marius BĂLTEANU about 3 years ago

  • Status changed from Reopened to Closed
Actions #13

Updated by Marius BĂLTEANU about 3 years ago

  • Subject changed from Change total spent time link to global time entries to Change total spent time link to global time entries when issue has subtasks that can be on non descendent projects
Actions #14

Updated by Marius BĂLTEANU about 3 years ago

  • Category set to Issues
Actions

Also available in: Atom PDF