Last modified: 2013-08-31 22:33:30 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 T30489, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 28489 - Reflect current revision status in links within BugZilla
Reflect current revision status in links within BugZilla
Status: RESOLVED WORKSFORME
Product: Wikimedia
Classification: Unclassified
Bugzilla (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-04-11 13:41 UTC by Krinkle
Modified: 2013-08-31 22:33 UTC (History)
1 user (show)

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


Attachments

Description Krinkle 2011-04-11 13:41:03 UTC
After brion's bug 27147 comment 3; Either server side or client side.

JS:
The links would be given a class around: http://svn.wikimedia.org/viewvc/mediawiki/trunk/tools/bugzilla/bugzilla-4.0/extensions/Wikimedia/Extension.pm?revision=84488&view=markup#l66 and a call to the mw.org api would get the status.

Serverside:
A way to query the database or api from there serverwise and put it directly in the output.


See also
* (bug 27147) Reflect current revision status in links within CR work flow
Comment 1 Brion Vibber 2011-04-11 17:14:33 UTC
Revision status can be fetched from CodeReview via API:

http://www.mediawiki.org/w/api.php?action=query&list=coderevisions&crrepo=MediaWiki&crrevs=12345&crprop=revid|author|status|timestamp&format=json

{
	"query": {
		"coderevisions": [
			{
				"revid": 12345,
				"status": "old",
				"author": "nikerabbit",
				"timestamp": "2006-01-02T01:00:38Z"
			}
		]
	}
}

Note this is also something that could be done from client-side JS so the lookups don't delay page generation. Emit an appropriate class for the rev links, and JS can go look them up in a batch.
Comment 2 Priyanka Dhanda 2011-06-22 21:26:06 UTC
Bugmeister is the new Bugzilla maintainer and default assignee.
Comment 3 Thehelpfulone 2012-06-22 19:19:01 UTC
Resetting to default per bug 37789
Comment 4 Andre Klapper 2013-08-31 22:33:30 UTC
In times of Gerrit (we abandoned CR which this request is about) we have the Gerrit Notification Bot in Bugzilla which announces
  1) related patches made available for review, and 
  2) related patches having been merged
by adding a comment to the related Bugzilla ticket.

This sounds sufficient to close this as RESOLVED WORKSFORME, but please reopen if you disagree.

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


Navigation
Links