Page MenuHomePhabricator

Add a special page to simply view a file to CodeReview
Closed, DeclinedPublicFeature

Description

Author: mike.lifeguard+bugs

Description:
Currently, CodeReview handles diffs for every commit. However, you cannot view full files (except in the diff when added/deleted). Please add that capacity. This would remove the need for a separate ViewVC install, and would allow whole-file review in the same environment as per-diff review.


Version: unspecified
Severity: enhancement

Details

Reference
bz22131

Event Timeline

bzimport raised the priority of this task from to Low.Nov 21 2014, 10:46 PM
bzimport set Reference to bz22131.
bzimport added a subscriber: Unknown Object (MLST).

mike.lifeguard+bugs wrote:

(In reply to comment #1)

Isn't that the goal of CodeBrowse?
http://svn.wikimedia.org/viewvc/mediawiki/trunk/extensions/CodeBrowse/

Thanks for informing me about this extension. However, it seems to be broken:

*"Binary files will not be shown." for non-binary files (well, everything, actually)
*Causes errors with CodeReview: Fatal error: Call to undefined method CodeRepoListView::getNavItem() in /var/www/w/extensions/CodeReview/ui/SpecialCode.php on line 101
*Has errors itself: Fatal error: Class 'CodeRepository' not found in /var/www/w/extensions/CodeBrowse/CodeBrowseRepoListView.php on line 10

It seems abandoned. But it's a start point.

Aklapper changed the subtype of this task from "Task" to "Feature Request".Feb 4 2022, 11:01 AM
Aklapper removed a subscriber: wikibugs-l-list.