Transformers: Allow setting properties to false.

[obenland]

Fixes #1565.

Proposed changes:

• Adds an explicit check for false values when evaluating getter results.

Other information:

• Have you written new tests for your changes, if applicable?

Testing instructions:

• Go to '..'

Changelog entry

• Automatically create a changelog entry from the details below.

Changelog Entry Details

Significance

• Patch
• Minor
• Major

Type

• Added - for new features
• Changed - for changes in existing functionality
• Deprecated - for soon-to-be removed features
• Removed - for now removed features
• Fixed - for any bug fixes
• Security - in case of vulnerabilities

Message

Transformers allow settings values to false again, a regression from 5.5.0.

@Menrath

[pfefferle]

I think we should unify all of the transformation methods!

See #1568

[pfefferle]

I think it is fine to use isset( $value ) for everything. We should skip null, but I think false, '' and array() are fine.

[pfefferle]

and we should also check the rest of the code. there is at least the from_array, that needs to be also fixed.

[Menrath]

I think it is fine to use isset( $value ) for everything. We should skip null, but I think false, '' and array() are fine.

Note that we in the events world have a case where a decidated null might have information, but as long as one can still can the setter manually this is totally fine.