Project

General

Profile

Actions

Patch #1202

closed

Feature to view whole the file from /repositories/revision page

Added by Go MAEDA over 16 years ago. Updated over 16 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
SCM
Target version:
Start date:
2008-05-07
Due date:
% Done:

0%

Estimated time:

Description

Here is a patch to add feature to view the whole file of the revision from Redmines /repositories/revision page.

Redmine can view differences against previous revision, but sometimes I want to view the whole file to check the context.


Files

view_link.png (16.2 KB) view_link.png Go MAEDA, 2008-05-07 13:29
view_the_whole_file_patch.txt (927 Bytes) view_the_whole_file_patch.txt Go MAEDA, 2008-05-07 13:34
view_the_whole_file_patch_0508.txt (934 Bytes) view_the_whole_file_patch_0508.txt Go MAEDA, 2008-05-08 02:22
redmine-changeset_r1552-edited.diff (2.12 KB) redmine-changeset_r1552-edited.diff Marc Liyanage, 2008-09-18 14:10
Actions #1

Updated by Go MAEDA over 16 years ago

Sorry, I forgot to attach the patch.

Actions #2

Updated by Go MAEDA over 16 years ago

Updated my patch.
I found that link to /repositories/annotate/ is more usuful for code review.

Actions #3

Updated by Jean-Philippe Lang over 16 years ago

  • Category set to SCM
  • Status changed from New to Closed
  • Target version set to 0.8

As of r1552, each change is a link to the whole file.

Actions #4

Updated by Marc Liyanage about 16 years ago

I applied r1552 to the 0.7.3 sources because we really like this feature.

When I compare that changeset to the patch here I notice two differences:

1.) This patch has a without_leading_slash() call in the link construction logic, but r1552 does not and indeed I had to add it to the patch I apply, r1552 as applied creates broken links for me. Why was that taken out, is it no longer needed against the sources targeting 0.8?

2.) This patch creates "annotate" URLs but r1552 does not, it creates "entry" URLs. I agree with Go MAEDA that annotate is nice and more useful. Is there a reason that was not used as suggested in this patch?

The patch I apply looks like the attached file.

Actions

Also available in: Atom PDF