Anomie has uploaded a new change for review.

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

Change subject: API: Remove warning about continuation change
......................................................................

API: Remove warning about continuation change

Change-Id: I2a69e8ae7dc7df915ceaaeed740b74d55f9207ce
---
M includes/api/ApiQuery.php
M tests/phpunit/includes/api/query/ApiQueryContinueTestBase.php
M tests/phpunit/includes/api/query/ApiQueryTestBase.php
3 files changed, 0 insertions(+), 19 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/75/205875/1

diff --git a/includes/api/ApiQuery.php b/includes/api/ApiQuery.php
index 2b18f6c..f4e1acb 100644
--- a/includes/api/ApiQuery.php
+++ b/includes/api/ApiQuery.php
@@ -301,17 +301,6 @@
                } else {
                        $continuationManager->setContinuationIntoResult( 
$this->getResult() );
                }
-
-               /// @todo: Remove this after a suitable period of time. When 
REL1_26 is cut, if not before.
-               if ( $this->mParams['continue'] === null && 
!$this->mParams['rawcontinue'] &&
-                       $this->getResult()->getResultData( 'continue' ) !== null
-               ) {
-                       $this->setWarning(
-                               'Formatting of continuation data has changed. ' 
.
-                               'To receive raw query-continue data, use the 
\'rawcontinue\' parameter. ' .
-                               'To silence this warning, pass an empty string 
for \'continue\' in the initial query.'
-                       );
-               }
        }
 
        /**
diff --git a/tests/phpunit/includes/api/query/ApiQueryContinueTestBase.php 
b/tests/phpunit/includes/api/query/ApiQueryContinueTestBase.php
index db61bc8..72aaa34 100644
--- a/tests/phpunit/includes/api/query/ApiQueryContinueTestBase.php
+++ b/tests/phpunit/includes/api/query/ApiQueryContinueTestBase.php
@@ -57,10 +57,6 @@
                } else {
                        $params['action'] = 'query';
                }
-               // Silence warning
-               if ( !isset( $params['continue'] ) ) {
-                       $params['continue'] = '';
-               }
                $count = 0;
                $result = array();
                $continue = array();
diff --git a/tests/phpunit/includes/api/query/ApiQueryTestBase.php 
b/tests/phpunit/includes/api/query/ApiQueryTestBase.php
index d5fa454..25d1efc 100644
--- a/tests/phpunit/includes/api/query/ApiQueryTestBase.php
+++ b/tests/phpunit/includes/api/query/ApiQueryTestBase.php
@@ -100,10 +100,6 @@
                if ( !array_key_exists( 'action', $req ) ) {
                        $req['action'] = 'query';
                }
-               // Silence warning
-               if ( !isset( $params['continue'] ) ) {
-                       $params['continue'] = '';
-               }
                foreach ( $req as &$val ) {
                        if ( is_array( $val ) ) {
                                $val = implode( '|', array_unique( $val ) );

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2a69e8ae7dc7df915ceaaeed740b74d55f9207ce
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Anomie <[email protected]>

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

Reply via email to