jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/363385 )

Change subject: Fix "Undefined index: scope_opener" in 
SpaceBeforeClassBraceSniff
......................................................................


Fix "Undefined index: scope_opener" in SpaceBeforeClassBraceSniff

Bug: T169783
Change-Id: I5d4dc2d90b82b5d877f08f5f3c168f50487ed9bd
---
M MediaWiki/Sniffs/WhiteSpace/SpaceBeforeClassBraceSniff.php
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Legoktm: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/MediaWiki/Sniffs/WhiteSpace/SpaceBeforeClassBraceSniff.php 
b/MediaWiki/Sniffs/WhiteSpace/SpaceBeforeClassBraceSniff.php
index 67df109..5eca1d8 100644
--- a/MediaWiki/Sniffs/WhiteSpace/SpaceBeforeClassBraceSniff.php
+++ b/MediaWiki/Sniffs/WhiteSpace/SpaceBeforeClassBraceSniff.php
@@ -31,7 +31,7 @@
         */
        public function process( File $phpcsFile, $stackPtr ) {
                $tokens = $phpcsFile->getTokens();
-               if ( $tokens[$stackPtr]['scope_opener'] === false ) {
+               if ( !isset( $tokens[$stackPtr]['scope_opener'] ) ) {
                        return;
                }
                $openBrace = $tokens[$stackPtr]['scope_opener'];

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5d4dc2d90b82b5d877f08f5f3c168f50487ed9bd
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/tools/codesniffer
Gerrit-Branch: master
Gerrit-Owner: Reedy <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: Reedy <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to