Merge "SpecialLog: Don't put assignment within another expression"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Sat, 9 Apr 2016 17:50:51 +0000 (17:50 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Sat, 9 Apr 2016 17:50:51 +0000 (17:50 +0000)
includes/specials/SpecialLog.php

index 95b7b7e..bbe0e0d 100644 (file)
@@ -147,7 +147,8 @@ class SpecialLog extends SpecialPage {
 
        private function parseParams( FormOptions $opts, $par ) {
                # Get parameters
-               $parms = explode( '/', ( $par = ( $par !== null ) ? $par : '' ) );
+               $par = $par !== null ? $par : '';
+               $parms = explode( '/', $par );
                $symsForAll = [ '*', 'all' ];
                if ( $parms[0] != '' &&
                        ( in_array( $par, $this->getConfig()->get( 'LogTypes' ) ) || in_array( $par, $symsForAll ) )