Last modified: 2012-12-25 21:41:26 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T32720, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 30720 - Add revision links to [[mw:Special:Code]] from various places in ViewVC
Add revision links to [[mw:Special:Code]] from various places in ViewVC
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
Subversion (Other open bugs)
unspecified
All All
: Lowest enhancement (vote)
: ---
Assigned To: Ryan Lane
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-09-03 06:23 UTC by T. Gries
Modified: 2012-12-25 21:41 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description T. Gries 2011-09-03 06:23:32 UTC
I suggest to modify the header template for the ViewVC interface to the SVN, so that each header comes with a single-click solution (precomputed) link to our Code Review.

Example:

When you view

http://svn.wikimedia.org/viewvc/mediawiki/trunk/phase3/resources/jquery/jquery.highlightText.js?view=markup&pathrev=94702

I wish to have a direct link to the corresponding code review 

http://www.mediawiki.org/wiki/Special:Code/MediaWiki/94702
Comment 1 Krinkle 2011-09-13 23:44:54 UTC
We actually had links back to mediawiki.org code review at some point. Presumably they got lost when the svn server was re-installed or when ViewVC was upgraded.

Tickets like bug 23177 confirm this.
Comment 2 T. Gries 2011-09-18 12:54:42 UTC
Presumably they got lost when the svn server was re-installed or when ViewVC
was upgraded...

--> if this was present in former ViewVC and MediaWiki SVN revision, than it is not "enhancement" bug but "normal". (setting to "normal")
Comment 3 Andre Klapper 2012-10-25 17:19:34 UTC
Probable WONTFIX with Git and Gerrit in place now.
Comment 4 Chad H. 2012-11-08 23:01:13 UTC
Done in Gerrit change #32465. Just needs merging.
Comment 5 Alex Monk 2012-12-25 21:41:26 UTC
(In reply to comment #4)
> Done in Gerrit change #32465. Just needs merging.

Merged on the 27th of November by Pyoungmeister.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links