Project

General

Profile

Actions

Patch #20661

closed

Show visible spent time link for users allowed to view time entries.

Added by Martin Dubé over 9 years ago. Updated almost 8 years ago.

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

0%

Estimated time:

Description

With this little change, the spent time is back, but showing only hours visible by the user according to the new "time log visibility" permission.


Files

show.html.erb.patch (631 Bytes) show.html.erb.patch Martin Dubé, 2015-09-02 15:25
spent-time-issues-index.png (10.8 KB) spent-time-issues-index.png Go MAEDA, 2017-01-19 13:30
spent-time-issues-show.png (17.8 KB) spent-time-issues-show.png Go MAEDA, 2017-01-19 13:35
20661-show-visible-spent-time.diff (1.23 KB) 20661-show-visible-spent-time.diff Go MAEDA, 2017-01-19 13:57

Related issues

Related to Redmine - Patch #25526: Revert API change in spent_hours field in issue#showClosedJean-Philippe Lang

Actions
Actions #1

Updated by Toshi MARUYAMA about 9 years ago

  • Status changed from New to Needs feedback

There are many "show.html.erb". Which show.html.erb?

Actions #2

Updated by Martin Dubé about 9 years ago

app/views/issues/show.html.erb

Thanks

Actions #3

Updated by Toshi MARUYAMA about 9 years ago

  • Status changed from Needs feedback to New
Actions #4

Updated by Robert Korulczyk almost 8 years ago

+1

Works great on 3.3.1. Thanks.

Actions #5

Updated by Go MAEDA almost 8 years ago

  • Category changed from UI to Time tracking

By this patch, different spent hours value is displayed for each user.
I am worried that it may cause confusion.

Actions #6

Updated by Robert Korulczyk almost 8 years ago

Go MAEDA wrote:

By this patch, different spent hours value is displayed for each user.
I am worried that it may cause confusion.

It already work in this way in multiple places (issues list for example) - user can see only his own time log.

Personally I think that much more confusing is situation, where issues list contain more details about issue than issue detail view, and user can't easily see how many hours his spend on the issue.

Actions #7

Updated by Go MAEDA almost 8 years ago

Robert Korulczyk wrote:

Go MAEDA wrote:

By this patch, different spent hours value is displayed for each user.
I am worried that it may cause confusion.

It already work in this way in multiple places (issues list for example) - user can see only his own time log.

Thank you for letting me know it. I didn't know. I agree with you now. I think that it is inconsistent.

Actions #8

Updated by Go MAEDA almost 8 years ago

Updated the patch.
Also Issue.load_visible_total_spent_hours should be called to show correct value of total_spent_hours.

Actions #10

Updated by Go MAEDA almost 8 years ago

  • Target version set to Candidate for next minor release
Actions #11

Updated by Go MAEDA almost 8 years ago

  • Target version changed from Candidate for next minor release to 3.3.3

Let's resolve the inconsistency.
Setting target version to 3.3.3.

Actions #12

Updated by Jean-Philippe Lang almost 8 years ago

  • Status changed from New to Resolved
  • Assignee set to Jean-Philippe Lang

Committed, thanks.

Actions #13

Updated by Jean-Philippe Lang almost 8 years ago

  • Status changed from Resolved to Closed
Actions #14

Updated by Jean-Philippe Lang almost 8 years ago

  • Target version changed from 3.3.3 to 3.2.6
Actions #15

Updated by Jean-Philippe Lang over 7 years ago

  • Related to Patch #25526: Revert API change in spent_hours field in issue#show added
Actions

Also available in: Atom PDF