Description: ------------ According to the ZEND Coding Style, if a function has a long declaration, it can be broken over several lines. In that case, the closing parenthesis goes on a newline, followed by a space, followed by the opening bracket. See http://framework.zend.com/manual/en/coding-standard.coding-style.html section "Function and Method Declaration", second example. PHP_CodeSniffer uses Generic_Sniffs_Functions_OpeningFunctionBraceBsdAllmanSniff for the ZEND standard. This incorrectly triggers an error on such long declarations: "Opening brace should be on a new line". This is incorrect. It should not generate an error. Test script: --------------- Example from the Zend Framework Coding Style linked above: /** * Documentation Block Here */ class Foo { /** * Documentation Block Here */ public function bar($arg1, $arg2, $arg3, $arg4, $arg5, $arg6 ) { // all contents of function // must be indented four spaces } }

The Zend standard is NOT the Zend Framework coding standard. It is simply a collection of sniffs contributed by people working at/with Zend over the last few years. I've been asking if someone wants to create a ZF standard for quite a while, but nobody has contributed anything. Please do not use PHP_CodeSniffer to check ZF code unless you've managed to get a ZF standard from someone.

Perhaps the standard should be renamed in PHP_CodeSniffer then?

I actually want to merge the standard into the Generic standard, but it's a big BC break and needs to be handled carefully with other merges. So I'm not planning a name change, although the standard has a ruleset.xml file in SVN already that says that it is out of date, so it is at least documented in 1.3.0.

Okay, thanks. In the mean time I have scratched my own itch and wrote a sniff that correctly handles the test script I posted above. Perhaps this is something you'd want to have too? If so, where shall I post it? Here or in a new bug?

Sorry it took me so long to reply. I got side-tracked with other work. If you'd like to contribute a sniff, please attach it to this report and I'll test and commit it as soon as possible.

Added patch zend-function-opening-brace-sniff.

Patch added. I hope it's okay like this. I've never made a PEAR patch before.

Is this actually the same standard as PEAR? http://pear.php.net/manual/en/standards.funcdef.php if so, you can just include the PEAR/Functions/FunctionDeclarationSniff into a ZF standard instead of writing new code. Coincidently, found out about this last night: https://github.com/dploeger/CodeSniffer_Zend_New Maybe worth talking to this developer and helping out. He is using the PEAR standard for most sniffs.

You're right. It's the same as PEAR. Perhaps the ZEND standard should use the PEAR function declaration sniff then instead of the Generic BsdAllman sniff? And thanks for the github link. That's going to be very useful!