Add missing return type
Merge pull request #2282 from bcrowe/hotfix-returnType

Add missing return type
Merge branch 'master' into 2.5
Remove CRLF from To headers in MailTransport.

Like other header fields To should not contain CRLF.

Refs #2209
Fix formatting of decimal values when there is no fractionSymbol.

When a currency format does not include a fractionSymbol it should be
able to format fractional values.

Fixes #2253
Merge branch 'master' into 2.5
Split existing tests up, and add tests for Router based image URL's

Refs #2233
Fix missing HTML encoding on URL's generated by Html->meta().

The input URL was not correctly handled as an asset URL and thus was
not correctly HTML or URL escaped. This created invalid HTML when
favicon URLs included query string arguments.

Refs #2233
Step one for moving implementedMethods logic

A behavior should be responsible for saying what methods it implements
allow configuring implmenetedMethods/Finders
allow behavior method aliasing

and change the way finders are stored to remove the find prefix
add test expectations for implementedMethods/Finders
reflection method cache should be static

I.e. apply to all instances of the same behavior class
Minor corrections/simplifications
explicitly define callable methods

It's an easy-win performance gain (by avoiding the use of reflections)
don't force strtolower in implemented* behavior methods

Since it's a public function don't leave open the possibility that
though configuration it doesn't "work" due to case changes. In the
behavior class, return without case manipulation - take care of that in
the behavior registry class.

Also add documentation for implemented* methods
use array_key_change_case and restructure cache

by consistently changing key case, when asking a behavior for which
methods it implements, a few strtolower calls can be avoided.

Using static::$_cacheProperty gets confused as it is shared by all
subclasses... so index the cache by class name.
separate call into call and callFinder

Having explicit methods should aide debugging - and make the intention
clear. also the finder index is indexed by type, not find method (e.g.
"list", not "findlist").
add tests for aliased behavior methods
add documentation and fix typos