jenkins-bot has submitted this change and it was merged.

Change subject: Remove unused parameters
......................................................................


Remove unused parameters

Change-Id: I7b91c02f18e70d6e47a8f5cbb550279269fff7e0
---
M repo/tests/phpunit/includes/api/CreateRedirectTest.php
M repo/tests/phpunit/includes/api/MergeItemsTest.php
2 files changed, 2 insertions(+), 2 deletions(-)

Approvals:
  JanZerebecki: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/repo/tests/phpunit/includes/api/CreateRedirectTest.php 
b/repo/tests/phpunit/includes/api/CreateRedirectTest.php
index 83a6d90..11d0c0d 100644
--- a/repo/tests/phpunit/includes/api/CreateRedirectTest.php
+++ b/repo/tests/phpunit/includes/api/CreateRedirectTest.php
@@ -74,7 +74,7 @@
 
                $permissionChecker->expects( $this->any() )
                        ->method( 'getPermissionStatusForEntityId' )
-                       ->will( $this->returnCallback( function( User $user, 
$permission, EntityId $id ) {
+                       ->will( $this->returnCallback( function( User $user, 
$permission ) {
                                if ( $user->getName() === 
'UserWithoutPermission' && $permission === 'edit' ) {
                                        return Status::newFatal( 
'permissiondenied' );
                                } else {
diff --git a/repo/tests/phpunit/includes/api/MergeItemsTest.php 
b/repo/tests/phpunit/includes/api/MergeItemsTest.php
index ff71ea3..c091f08 100644
--- a/repo/tests/phpunit/includes/api/MergeItemsTest.php
+++ b/repo/tests/phpunit/includes/api/MergeItemsTest.php
@@ -70,7 +70,7 @@
 
                $permissionChecker->expects( $this->any() )
                        ->method( 'getPermissionStatusForEntityId' )
-                       ->will( $this->returnCallback( function( User $user, 
$permission, EntityId $id ) {
+                       ->will( $this->returnCallback( function( User $user, 
$permission ) {
                                if ( $user->getName() === 
'UserWithoutPermission' && $permission === 'edit' ) {
                                        return Status::newFatal( 
'permissiondenied' );
                                } else {

-- 
To view, visit https://gerrit.wikimedia.org/r/208133
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I7b91c02f18e70d6e47a8f5cbb550279269fff7e0
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Jeroen De Dauw <[email protected]>
Gerrit-Reviewer: Addshore <[email protected]>
Gerrit-Reviewer: JanZerebecki <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to