Description: ------------ The traits aren't currently analyzed by PHP_CodeSniffer_Standards_AbstractVariableSniff sniff. I think they should be .

Also the T_TRAIT should be added to Squiz check, because it's in Zend check. And id makes sense.

The AbstractVariableSniff now checks traits: https://github.com/squizlabs/PHP_CodeSniffer/commit/07c51b1c5dcf7feb2a734b9dc80 44ca115c7b97e Not sure what you mean by Zend and Squiz doing different things. You didn't specify what sniffs you were talking about. If this change doesn't fix it, please give me a bit more info and I'll take a look.

Ah, I wasn't aware that properties of the traits aren't being scanned at all. Thanks for fixing that. What I mean was the "hasCondition" lines of "Squiz_Sniffs_NamingConventions_ValidVariableNameSniff" sniff: - https://github.com/squizlabs/PHP_CodeSniffer/blob/master/CodeSniffer/Standards/Sq uiz/Sniffs/NamingConventions/ValidVariableNameSniff.php#L119-119 - https://github.com/squizlabs/PHP_CodeSniffer/blob/master/CodeSniffer/Standards/Sq uiz/Sniffs/NamingConventions/ValidVariableNameSniff.php#L228 As you can see when determining if variable is within a class the T_TRAIT token isn't checked. In contrast in the "Zend_Sniffs_NamingConventions_ValidVariableNameSniff" sniff this has been checked (2 places): https://github.com/squizlabs/PHP_CodeSniffer/blob/master/CodeSniffer/Standards/Ze nd/Sniffs/NamingConventions/ValidVariableNameSniff.php#L125-125 Also the "processVariableInString" implementation of both mentioned sniffs the leading "_" stripping code is never invoked because variables in string aren't tokenized so for: $var = "ups {$this->_test}"; code the variable name being analyzed is "this" not the "_test" as with regular tokenized code.

Yep, you are right about those sniffs. I've committed some additional changes here: https://github.com/squizlabs/PHP_CodeSniffer/commit/b6a56f67fa8894251019336636 84bef558566b2d

Maybe regex in processVariableInString can be improved to capture complex variable names as well, e.g. " ... {$this::ConstantName} ..." then we can tokenize each of them and determine what we need?