[5.7] Cast sqlite foreign key constraints configuration

[roberto-aguilar]

Instead of delegating the value casting to every caller of this method (which is going to be used for boolean operations), it now handles it itself.

This is a safe refactor because missing configurations are going to return null and therefore be casted as false.

[sisve]

This is a safe refactor because missing configurations are going to return null and therefore be casted as false.

Could you expand on this? I imagine that if I have written code that $obj->getForeignKeyConstraintsConfigurationValue() === null, then that would never match anymore.

The return type is part of the method signature, and changing method signatures are often breaking changes.

[roberto-aguilar]

You're right, @sisve

The PR that introduced this method (#26298) was just merged today, and even though this has not been tagged yet, i agree with you that is a breaking change at this point.

Thanks for the feedback!