Fix serveral tests pass regardless of whether data is valid or not
Fix transactions do not get rollbacked in saveAssociated/saveMany

Refs #2849
Add some more transaction tests

And remove 2 else clauses.
CS fixes.
Merge pull request #4161 from dereuromark/master-cs-equals

CS fixes.
Added missing return statement.

PhpStorm is flagging this function as missing a return statement. Using a return statement (instead of the default) makes the intent of the source code clearer.
Merge pull request #4169 from thinkingmedia/patch-1

Added missing return statement.
Merge pull request #4140 from chinpei215/master-issue2849-fix

Fix transactions do not get rollbacked in saveAssociated/saveMany.

Fixes #2849