Squiz/ValidFunctionName: don't hide one message behind another

Author: jrfnl

src/Standards/Squiz/Sniffs/NamingConventions/ValidFunctionNameSniff.php

@@ -39,7 +39,8 @@ protected function processTokenOutsideScope(File $phpcsFile, $stackPtr)
         if (preg_match('|^__[^_]|', $functionName) !== 0) {
             $error = 'Function name "%s" is invalid; only PHP magic methods should be prefixed with a double underscore';
             $phpcsFile->addError($error, $stackPtr, 'DoubleUnderscore', $errorData);
-            return;
+
+            $functionName = ltrim($functionName, '_');
         }
 
         if (Common::isCamelCaps($functionName, false, true, false) === false) {

src/Standards/Squiz/Tests/NamingConventions/ValidFunctionNameUnitTest.php

@@ -35,7 +35,7 @@ public function getErrorList()
             11 => 1,
             12 => 1,
             13 => 1,
-            14 => 1,
+            14 => 2,
         ];
 
     }//end getErrorList()