Undeprecate PARTIAL for array hydration.

[beberlei]

Pair for #11365

Related:

• Deprecate Partial Objects #8471

[greg0ire]

Please update UPGRADE.md as well 🙏

[derrabus]

Shouldn't we target 2.20 for this?

The base branch was changed.

[greg0ire]

Shouldn't we target 2.20 for this?

@derrabus I don't think we should: #11365 will make this the deprecation wrong, which makes this PR a bugfix.

[derrabus]

#11365 will make this the deprecation wrong, which makes this PR a bugfix.

#11365 isn't a bugfix either, which makes this change a forward compat improvement.

[beberlei]

We made forward compatibility improvements in mini versions in DBAL, so I an with @greg0ire here that this can be in 2.19

[derrabus]

But that's not what we agreed on in #11208.

[greg0ire]

@derrabus I think it would qualify as an improvement if there was something that should be deprecated but wasn't. Here, we are wrongly over-deprecating, causing extra work.

Just like static analysis tags can be imprecise and wrong and static analysis changes can go on patch or minor depending on whether they make things more accurate or more precise, this change is about making the deprecation more accurate. When #11365, the deprecation will become wrong, hence this is a bugfix, and complies with #11208 IMO, although I must admit it feels like an edge case/ gray area stuff, because it's correct today, and will suddenly become wrong tomorrow.

This relates to doctrine/doctrine-website#485

[beberlei]

@derrabus @greg0ire if we put this in 2.20 you both ok with it?

[greg0ire]

Yes