Definition of IDatabase::buildLike doesn't match Database::buildLike
Closed, DuplicatePublic
Actions

Assigned To

Authored By

	Reedy
	Jun 20 2019, 11:38 PM

Description

Database.php:

	public function buildLike( $param, ...$params ) {

IDatabase.php

	public function buildLike( $param );

Seems to be since https://github.com/wikimedia/mediawiki/commit/725a59f0c70e37de6de8e26f91b0101839d37ff9

This is needed in order for Phan not to consider calls to IDatabase::buildLike as invalid. Interestingly, it does not consider calls to Database::buildLike invalid.

It seems it's not liked now.. From https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Jade/+/511080/ see https://integration.wikimedia.org/ci/job/mwext-php72-phan-docker/353/console

00:31:38 <checkstyle version="6.5">
00:31:38   <file name="maintenance/CleanJudgmentLinks.php">
00:31:38     <error line="171" severity="info" message="Call with 2 arg(s) to \Wikimedia\Rdbms\IDatabase::buildLike() which only takes 1 arg(s) defined at ../../includes/libs/rdbms/database/IDatabase.php:1226" source="PhanParamTooMany"/>
00:31:38   </file>
00:31:38 </checkstyle>

Details

	Subject	Repo	Branch	Lines +/-
	rdbms: Type IDatabase::buildLike param as variadic to Phan	mediawiki/core	master	+2 -0

Customize query in gerrit

Related Objects

Mentioned In: T191668: Define varargs in \IDatabase::buildLike() in a way phan can understand it

Event Timeline

Reedy created this task.Jun 20 2019, 11:38 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJun 20 2019, 11:38 PM

Reedy updated the task description. (Show Details)Jun 20 2019, 11:40 PM

Reedy updated the task description. (Show Details)Jun 20 2019, 11:42 PM

Reedy updated the task description. (Show Details)Jun 21 2019, 12:28 AM

Reedy added a subscriber: Umherirrender.

Let's use @phan-param to let Phan that it really is variadic.

https://github.com/phan/phan/wiki/Annotating-Your-Source-Code#aliases-of-phpdoc-annotations

Change 518320 had a related patch set uploaded (by Krinkle; owner: Krinkle):
[mediawiki/core@master] rdbms: Type IDatabase::buildLike param as variadic to Phan

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

gerritbot added a project: Patch-For-Review.Jun 21 2019, 9:03 PM

Krinkle triaged this task as Low priority.Jun 22 2019, 9:03 PM

Change 518320 merged by jenkins-bot:
[mediawiki/core@master] rdbms: Type IDatabase::buildLike param as variadic to Phan

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

ReleaseTaggerBot added a project: MW-1.34-notes (1.34.0-wmf.11; 2019-06-26).Jun 24 2019, 8:00 PM

Krinkle closed this task as Resolved.Jun 24 2019, 10:59 PM

Krinkle edited projects, added Performance-Team; removed Patch-For-Review.

Krinkle moved this task from Untriaged to Rdbms library on the MediaWiki-libs-Rdbms board.

This is still failing on the same JADE patch...

https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/Jade/+/511080/
https://integration.wikimedia.org/ci/job/mwext-php72-phan-docker/995/console

14:09:11 <?xml version="1.0" encoding="ISO-8859-15"?>
14:09:11 <checkstyle version="6.5">
14:09:11   <file name="maintenance/CleanJudgmentLinks.php">
14:09:11     <error line="171" severity="info" message="Call with 2 arg(s) to \Wikimedia\Rdbms\IDatabase::buildLike() which only takes 1 arg(s) defined at ../../includes/libs/rdbms/database/IDatabase.php:1234" source="PhanParamTooMany"/>
14:09:11   </file>
14:09:11 </checkstyle>

Change 519728 had a related patch set uploaded (by Krinkle; owner: Krinkle):
[mediawiki/core@master] rdbms: Type IDatabase::buildLike param as variadic to Phan (really)

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

gerritbot added a project: Patch-For-Review.Jun 28 2019, 10:50 PM

Krinkle reopened this task as Open.Jun 28 2019, 10:51 PM

Krinkle moved this task from Inbox, needs triage to Blocked (old) on the Performance-Team board.Jul 1 2019, 8:01 PM

Krinkle closed this task as a duplicate of T191668: Define varargs in \IDatabase::buildLike() in a way phan can understand it.Jul 5 2019, 6:41 PM

Krinkle mentioned this in T191668: Define varargs in \IDatabase::buildLike() in a way phan can understand it.

Definition of IDatabase::buildLike doesn't match Database::buildLikeClosed, DuplicatePublicActions

Description

Details

Related Objects

Event Timeline

Definition of IDatabase::buildLike doesn't match Database::buildLike
Closed, DuplicatePublic
Actions