Swap usage of create_function for defined function as a callback
authorSam Reed <reedy@users.mediawiki.org>
Tue, 19 Oct 2010 00:44:28 +0000 (00:44 +0000)
committerSam Reed <reedy@users.mediawiki.org>
Tue, 19 Oct 2010 00:44:28 +0000 (00:44 +0000)
includes/api/ApiBase.php

index 18dfb98..71e9309 100644 (file)
@@ -532,8 +532,8 @@ abstract class ApiBase {
                array_shift( $required );
 
                $intersection = array_intersect( array_keys( array_filter( $params,
-                               create_function( '$x', 'return !is_null($x) && $x !== false;' )
-                       ) ), $required );
+                               "validateParameter" ) ), $required );
+
                if ( count( $intersection ) > 1 ) {
                        $this->dieUsage( 'The parameters ' . implode( ', ', $intersection ) . ' can not be used together', 'invalidparammix' );
                } elseif ( count( $intersection ) == 0 ) {
@@ -541,6 +541,10 @@ abstract class ApiBase {
                }
        }
 
+       private function validateParameter( $x ) {
+               return !is_null( $x ) && $x !== false;
+       }
+
        /**
         * @deprecated use MWNamespace::getValidNamespaces()
         */