Deprecate type inference in Query::setParameter

[beberlei]

We should assume every object passed must be an entity or specify a $type as third argument. Only primitive types should work with type inference. A deprecation is in order to allow users. to migrate of this behavior.

Problems with type inference:

• Performance problems due to trying to load mapping files and failing Massive performance issue with setParameter() and DateTime* objects #8113
• Ambiguities when inferring DateTime, to datetime or datetime_utc. Insufficient autodetection of query parameter type #6443

[beberlei]

In addition we should think about differentiating between setParameter for single valued paraemters and setParameterList for an array of values of the specified type. Then this could allow support for arrays of more complex types:

$dates = [new DateTime, new DateTime];
$query->setParameterList(1, $dates, "datetime");

[beberlei]

Needs more research

[sips-richard]

The optional type argument on setParameter() and the baked in behaviour of inferType() for booleans and datetimes is confusing and sets the wrong expectation for consumers. I've just been digging into this to understand why these types were treated differently from other registered types. We will be updating our company coding standard to mandate that the type is always specified for consistency and clarity (obviously Connection::PARAM_STR_ARRAY etc. isn't ideal). I would welcome this change in Doctrine.