Squiz.Scope.MethodScope.Missing error message now mentions 'visibility' instead of 'scope modifier'

Author: gsherwood

CodeSniffer/Standards/Squiz/Sniffs/Scope/MethodScopeSniff.php

@@ -64,7 +64,7 @@ protected function processTokenWithinScope(PHP_CodeSniffer_File $phpcsFile, $sta
 
         $modifier = $phpcsFile->findPrevious(PHP_CodeSniffer_Tokens::$scopeModifiers, $stackPtr);
         if (($modifier === false) || ($tokens[$modifier]['line'] !== $tokens[$stackPtr]['line'])) {
-            $error = 'No scope modifier specified for function "%s"';
+            $error = 'Visibility must be declared on method "%s"';
             $data  = array($methodName);
             $phpcsFile->addError($error, $stackPtr, 'Missing', $data);
         }

package.xml

@@ -55,6 +55,8 @@ http://pear.php.net/dtd/package-2.0.xsd">
   - Squiz ClassFileNameSniff no longer throws errors when checking STDIN
   - Squiz CSS sniffs no longer generate errors for IE filters
   - Squiz LogicalOperatorSpacingSniff now ignores whitespace at the end of a line
+  - Squiz.Scope.MethodScope.Missing error message now mentions 'visibility' instead of 'scope modifier'
+    -- Thanks to Renat Akhmedyanov for the patch
   - The PSR2 standard no longer throws errors for additional spacing after a type hint
   - Fixed bug #20026 : Check for multi-line arrays that should be single-line is slightly wrong
     -- Adds new error message for single-line arrays that end with a comma