lib/CrEOF/Geo/WKB/Reader.php
The 'getMachineByteOrder()' method which returns a boolean should be named 'is...()' or 'has...()' Open
Open
private function getMachineByteOrder()
{
if (null === self::$machineByteOrder) {
$result = unpack('S', "\x01\x00");
- Read upRead up
- Exclude checks
BooleanGetMethodName
Since: 0.2
Looks for methods named 'getX()' with 'boolean' as the return type. The convention is to name these methods 'isX()' or 'hasX()'.
Example
class Foo {
/**
* @return boolean
*/
public function getFoo() {} // bad
/**
* @return bool
*/
public function isFoo(); // ok
/**
* @return boolean
*/
public function getFoo($bar); // ok, unless checkParameterizedMethods=true
}