cakephp/cakephp — pull request #5084 by dereuromark (2014-11-05 12:06:49+00:00)
Title: Correct doc block return types.

To stay BC I did not modify any code and rather fixed the doc block - even though in some places one should. It would be cleaner to always return string (empty then instead of null).

E.g.
```php
/**
* Returns the schema name. Override this in subclasses.
*
* @return string|null schema name
*/
public function getSchemaName() {
return null;
}
```
should be
/**
* Returns the schema name. Override this in subclasses.
*
* @return string Schema name
*/
public function getSchemaName() {
return '';
}
```

And stuff like
```php
/**
* Get the last error as a string.
*
* @return string|null Last error
*/
public function lastError() {
if (!empty($this->lastError)) {
return $this->lastError['num'] . ': ' . $this->lastError['str'];
}
return null;
}
```
should be
```php
/**
* Get the last error as a string.
*
* @return string Last error
*/
public function lastError() {
if (!empty($this->lastError)) {
return $this->lastError['num'] . ': ' . $this->lastError['str'];
}
return '';
}
```

Maybe for 2.6?