Bartosz Dziewoński has uploaded a new change for review.

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

Change subject: SpecialLog: Don't put assignment within another expression
......................................................................

SpecialLog: Don't put assignment within another expression

Change-Id: I0a0cd9ee0647e8778d2e835cc02c571a83b3a8c2
---
M includes/specials/SpecialLog.php
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/35/281735/1

diff --git a/includes/specials/SpecialLog.php b/includes/specials/SpecialLog.php
index 7132207..85728ef 100644
--- a/includes/specials/SpecialLog.php
+++ b/includes/specials/SpecialLog.php
@@ -147,7 +147,8 @@
 
        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 ) )

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0a0cd9ee0647e8778d2e835cc02c571a83b3a8c2
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Bartosz Dziewoński <matma....@gmail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to