SpecialCode.php on line 134
Closed, ResolvedPublicPRODUCTION ERROR
Actions

Description

Error

MediaWiki version: 1.35.0-wmf.16

message

Argument 3 passed to CodeRevisionView::__construct() must be an instance of User, string given, called in /srv/mediawiki/php-1.35.0-wmf.16/extensions/CodeReview/includes/ui/SpecialCode.php on line 134

Impact

Fatal exception on GET for URL.

Notes

6 of these since deploy of 1.35.0-wmf.16 to group0.

Details

Request ID: XideeQpAIDEAACSkotYAAACS
Request URL: https://www.mediawiki.org/wiki/Special:Code/pywikipedia/11203

Stack Trace

exception.trace

#0 /srv/mediawiki/php-1.35.0-wmf.16/extensions/CodeReview/includes/ui/SpecialCode.php(134): CodeRevisionView->__construct(CodeRepository, User, string)
#1 /srv/mediawiki/php-1.35.0-wmf.16/extensions/CodeReview/includes/ui/SpecialCode.php(49): SpecialCode->getViewFrom(string)
#2 /srv/mediawiki/php-1.35.0-wmf.16/includes/specialpage/SpecialPage.php(575): SpecialCode->execute(string)
#3 /srv/mediawiki/php-1.35.0-wmf.16/includes/specialpage/SpecialPageFactory.php(611): SpecialPage->run(string)
#4 /srv/mediawiki/php-1.35.0-wmf.16/includes/MediaWiki.php(298): MediaWiki\Special\SpecialPageFactory->executePath(Title, RequestContext)
#5 /srv/mediawiki/php-1.35.0-wmf.16/includes/MediaWiki.php(967): MediaWiki->performRequest()
#6 /srv/mediawiki/php-1.35.0-wmf.16/includes/MediaWiki.php(530): MediaWiki->main()
#7 /srv/mediawiki/php-1.35.0-wmf.16/index.php(46): MediaWiki->run()
#8 /srv/mediawiki/w/index.php(3): require(string)
#9 {main}

	Project	Branch	Lines +/-	Subject
	mediawiki/extensions/CodeReview	wmf/1.35.0-wmf.16	+3 -3	Follow-Up 81f5fb: Remove use of global $wgUser (part 10)
	mediawiki/extensions/CodeReview	master	+3 -3	Follow-Up 81f5fb: Remove use of global $wgUser (part 10)

Customize query in gerrit

Related Objects

Mentioned Here: T242677: CodeReview needs uses of global $wgUser removed
rECREebbc00a88269: Remove use of global $wgUser (part 8)
T159299: Deprecate and remove $wgUser

Event Timeline

brennen created this task.Jan 21 2020, 8:47 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJan 21 2020, 8:47 PM

By way of changelog for wmf.16, maybe this is related to @DannyS712's work on T159299?

brennen added a project: User-brennen.Jan 21 2020, 8:55 PM

brennen moved this task from Backlog to Logs/Train on the User-brennen board.

In T243337#5820990, @brennen wrote:

By way of changelog for wmf.16, maybe this is related to @DannyS712's work on T159299?

Indeed. This is T242677: CodeReview needs uses of global $wgUser removed / rECREebbc00a882698d42dcd7e2608115a00068f53190

Llooking at the current version[1] of Special:Code, line 134 is

$view = new CodeRevisionView( $repo, $user, $params[1] );

with $user coming from

$user = $this->getUser();

[1] https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/extensions/CodeReview/+/ebbc00a882698d42dcd7e2608115a00068f53190/includes/ui/SpecialCode.php

Change 566360 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/CodeReview@master] Follow-Up 81f5fb: Remove use of global $wgUser (part 10)

https://gerrit.wikimedia.org/r/566360

gerritbot added a project: Patch-For-Review.Jan 21 2020, 9:06 PM

DannyS712 claimed this task.Jan 21 2020, 9:09 PM

DannyS712 moved this task from Unsorted to Awaiting review and deployment on the User-DannyS712 board.

brennen added a parent task: T233864: 1.35.0-wmf.16 deployment blockers.Jan 21 2020, 9:44 PM

I guess possibly the deployment blocker relationship is overkill here - it doesn't seem high impact enough to warrant rolling group0 back, and I'm not sure it has much impact beyond group0?

brennen removed a parent task: T233864: 1.35.0-wmf.16 deployment blockers.Jan 21 2020, 10:20 PM

Change 566360 merged by jenkins-bot:
[mediawiki/extensions/CodeReview@master] Follow-Up 81f5fb: Remove use of global $wgUser (part 10)

https://gerrit.wikimedia.org/r/566360

Change 566385 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/CodeReview@wmf/1.35.0-wmf.16] Follow-Up 81f5fb: Remove use of global $wgUser (part 10)

https://gerrit.wikimedia.org/r/566385

ReleaseTaggerBot added a project: MW-1.35-notes (1.35.0-wmf.18; 2020-02-04).Jan 22 2020, 12:00 AM

Agusbou2015 added a subscriber: Agusbou2015.Jan 22 2020, 12:59 AM

Change 566385 merged by jenkins-bot:
[mediawiki/extensions/CodeReview@wmf/1.35.0-wmf.16] Follow-Up 81f5fb: Remove use of global $wgUser (part 10)

https://gerrit.wikimedia.org/r/566385

ReleaseTaggerBot edited projects, added MW-1.35-notes (1.35.0-wmf.16; 2020-01-21); removed MW-1.35-notes (1.35.0-wmf.18; 2020-02-04).Jan 22 2020, 10:00 PM

Maintenance_bot removed a project: Patch-For-Review.Jan 22 2020, 10:10 PM

Mentioned in SAL (#wikimedia-operations) [2020-01-22T22:19:55Z] <catrope@deploy1001> Synchronized php-1.35.0-wmf.16/extensions/CodeReview/: T243337 (duration: 01m 06s)

Tested at: https://www.mediawiki.org/wiki/Special:Code/pywikipedia/11203

Looking good.

DannyS712 moved this task from Awaiting review and deployment to Merged on the User-DannyS712 board.Feb 18 2020, 10:26 PM

Argument 3 passed to CodeRevisionView::__construct() must be an instance of User, string given, called in /srv/mediawiki/php-1.35.0-wmf.16/extensions/CodeReview/includes/ui/SpecialCode.php on line 134Closed, ResolvedPublicPRODUCTION ERRORActions