FauxRequest: don’t override getValues()
authorLucas Werkmeister <lucas.werkmeister@wikimedia.de>
Wed, 11 Sep 2019 16:54:19 +0000 (18:54 +0200)
committerLucas Werkmeister <lucas.werkmeister@wikimedia.de>
Wed, 11 Sep 2019 16:56:31 +0000 (18:56 +0200)
As far as I can tell, the override is not necessary (the only other
method that the original implementation uses is getGPCVal(), which
FauxRequest still uses elsewhere), and it breaks tests that rely on the
filtering behavior.

Change-Id: I7dd0a8dd36e31cc6784364d56e460b55bff6ea59

includes/FauxRequest.php

index 78f6ca9..9337270 100644 (file)
@@ -86,14 +86,6 @@ class FauxRequest extends WebRequest {
                return (string)$this->getVal( $name, $default );
        }
 
-       /**
-        * @return array
-        * @suppress PhanParamSignatureMismatch
-        */
-       public function getValues() {
-               return $this->data;
-       }
-
        /**
         * @return array
         */