Description: ------------ The rule identifies problems below if no {}, I believe it is because you are inside switch. There's something in this latest version corrected about that, I had noticed that in the earlier version just had not reported. In the function "teste" is identified a problem in the first break, after putting in {} in if "teste2" the error is not identified. Test script: --------------- function teste($a, $b){ switch($a){ case 1: if (empty($b)) return 0; break; default: return 2; } return 1; } function teste2($a, $b){ switch($a){ case 1: if (empty($b)){ return 0; } break; default: return 2; } return 1; }

There is also no problems in else without {} <?php function teste($a){ if (empty($a)) echo '1'; else return 0; echo "oi"; } ?>

Can't people just use braces... :) I'll see if there is anything I can do to detect this properly in the sniff.

Fixed in github repo: https://github.com/squizlabs/PHP_CodeSniffer/commit/01754d9c1a622def2f7d04747a b7ebcce767aef4

The throw also generates the problem function fun($a, $b){ switch ($a){ case 1: if ($b == 0) throw new Exception("zero"); echo 'oi'; break; } } fun(1, 1);

I have fixed this generically so the THROW statement no longer throws non executable code errors either.

The bug continues, sample code: class classFragment { function aplicaValidacao($texto){ if (empty($this->validacao)) return $texto; switch ($this->validacao){ case "STRORMOEDA": if (is_numeric($texto)) return Conversao::converter(DINHEIRO, $texto, TELA); return $texto; case "DATA_EXTENSO": return Conversao::dataExtenso($texto); default: return Conversao::converter($this->validacao, $texto, TELA); } } }

This new issue is unrelated to the one in this report. Turns out it is actually a core tokenizing issue where the RETURN in the IF statement is being assigned as the closer of the first CASE statement. Needs a core fix, not a sniff fix, in this case. The following code can be used to replicate: switch ($foo) { case 'foo': if ($foo) return $foo; return $bar; default: return $bar; }

Problem was caused by fix for bug 19755 and fix committed to github repo here: https://github.com/squizlabs/PHP_CodeSniffer/commit/6f77d5a5bf0669eb302073d86d a91abec7e0ae23