Description: ------------ There seems to be an error in the PEAR standard, in PEAR.Functions.FunctionCallSignature.Indent. This incorrectly errors with "Multi-line function call not indented correctly; expected (x) spaces but found (x+4)" when presented with a multi-line function call that contains nested object notation. The test should detect that there is no comma after the first nested line, and that the second starts with an object operator (->), and so not put out an error in this case. Test script: --------------- public function doSomething() { return $this->getFoo() ->doBar( $this->getX() // no comma here ->doY() // this is still the first method argument ->doZ() // this is still the first method argument ); } // test using phpcs --standard=PEAR Expected result: ---------------- Expected that this should pass the indent test Actual result: -------------- -------------------------------------------------------------------------------- FOUND 2 ERROR(S) AFFECTING 2 LINE(S) -------------------------------------------------------------------------------- 79 | ERROR | Multi-line function call not indented correctly; expected 16 | | spaces but found 20 80 | ERROR | Multi-line function call not indented correctly; expected 16 | | spaces but found 20 --------------------------------------------------------------------------------

Added patch object-operator-indent-detection.

Added a proposed patch. Please tell me if it is not in the correct format.

Patch worked great, thanks. Fix committed to github repo: https://github.com/squizlabs/PHP_CodeSniffer/commit/d269206f5fca287b277c832fd0 2bcfcf55378b4a