Deprecate ::supportedInterfaceOrClass property on normalizer/denormalizers

I bet it needs to convert and clean-up all usages in core

here's a different approach - removal of property definition and throwing deprecation message

It could use existing change record https://www.drupal.org/node/3359695 as the method now more useful

But maybe it should use parent method implementation (from Symfony)

fix cs

Updated IS a bit

Fix few tests

fix cs

Fix last tests

Hi applied succesfully in drupal 11.x version
Tests are passing
Deprecations are corrects (deprecation message follows the required patters)
Property has been removed has issue's summary says

Moving to RTBC

Not seeing the push yet?

Also unsure if this needs a change record as per the tag?

I think it needs follow-up to remove other mentions in code-comments

Thanks @andypost. I've opened #3361401: Remove remaining occurences of ::supportedInterfaceOrClass property for that.

I was (pleasantly!) surprised to already see this land due to its BC break implications … but I see it landed only in 11.x! 🤯 Will this not be a problem for 10.2!

- Deprecation warning is on 10.2 in the 11.x branch
- The 10.2.x branch will be branched from 11.x branch at the time of 10.2.0-alpha1, so that will "inherit"the deprecation.

I think we're good.

Here's patch #12 but without deprecation

TestBot is unhappy about the 9.5.x patch :/

The patch is for 10.1 which has SF 6.3 and can benefit from the change, 9.5 I triggered just for fun

Thanks @andypost, I was already wondering about SF 6.3 and 9.5.x.

Also, we had a few escapees (See #3360280: Add missing ::hasCacheableSupportsMethod deprecations), may we should get those in first and then do the backport?
Nvm, outdated issue by now.

Good idea to combine backport with polishing, we can include #3361401: Remove remaining occurences of ::supportedInterfaceOrClass property

So marking as the tests seem green. Change record appears to cover what's happening too.

I'd like to add some information to the change record. I got hit with a deprecation that referenced the change record associated with this issue.

https://git.drupalcode.org/project/rest_views/-/commit/69d089ef7794badda...

The change record did not seem to address this case. I'd propose updating the title to something like:

Normalizers/Denormalizers should implement ::getSupportedTypes() instead of ::hasCacheableSupportsMethod() or using protected $supportedInterfaceOrClass

Then in the change record showing a before and after:

Before

  /**
   * The interface or class that this Normalizer supports.
   *
   * @var array
   */
  protected $supportedInterfaceOrClass = [RenderableData::class];

After

  /**
   * {@inheritdoc}
   */
  public function getSupportedTypes(?string $format): array {
    return [
      RenderableData::class => TRUE,
    ];
  }