...

Inspection name

Error message

Example code

Problem description

Solved

Drupal_Sniffs_Formatting_SpaceOperatorSniff

A opeator statement must be followed by a single space

Code Block

language	php

$this->Application =& kApplication::Instance();

Inspections should check, that operands are separated from operators by a single space.

Code Block

language	php
title	Good

$a + $b

Code Block

language	php
title	Bad

$a+$b

However symbols "=" and "&" are operators on it's own ("equals" and "bitwise and") and probably during syntax check inspection isn't aware, that together ("=&") they represent assigning a value by reference.

Status


colour	RedGreen
title	No

YES

Drupal_Sniffs_Array_ArraySniff

A comma should follow the last multiline array item. Found: /*'k2',*/

Code Block

language	php

$array = Array (
	'k1',
	/*'k2',*/
);

Inspection forces usage of trailing comma after last array element. However in this example commented out code is present before array declaration end, which makes inspection go crazy and not notice actual comma after 'k1' code.

Status


colour	Red
title	No

Green
title	YES

PEAR_Sniffs_Functions_FunctionCallSignatureSniff

Multi-line function call not indented correctly; expected 7 spaces but found 4

Code Block

language	php

str_replace(
	'one',
	'two',
	'three'
);

Inspection thinks, that spaces need to be used to indent stuff and not TABs. It's not an error inside an inspection, because it's PEAR standard inspection, where this exactly is needed to be done.

We need to either found similar inspection, that uses TABs or create our own based on this one.

Status


colour	Green
title	YES

Coding standards used by other software

http://forge.typo3.org/projects/team-php_codesniffer/wiki/FLOW3__TYPO3_Phoenix_Sniffs_with_code_examples

Version	Old Version 2	New Version Current
Changes made by	Alex	Alex
Saved on	Mar 10, 2013	Sep 09, 2015

Versions Compared

Key

Coding standards used by other software

Page Comparison

Versions Compared

Key

Coding standards used by other software