Problem/Motivation

The documentation for \Drupal\serialization\RegisterEntityResolversCompilerPass says that adds services tagged with normalizer and encoder to the Serializer. In fact, it adds services tagged with entity_resolver.

  foreach ($container->findTaggedServiceIds('entity_resolver') as $id => $attributes) {

Proposed resolution

Make comment more clear.

Remaining tasks

- Reviews needed

CommentFileSizeAuthor
#2 3159102-2.patch659 bytesNiklan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Niklan created an issue. See original summary.

Niklan’s picture

Niklan
Primary language Russian
Location Russia, Perm
CreditAttribution: Niklan as a volunteer commented
Status: Active » Needs review
FileSize
3159102-2.patch659 bytes
andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented
Issue tags: +Quick fix
andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented
Status: Needs review » Reviewed & tested by the community

Yes, it consumes only entity_resolver

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott as a volunteer and at Acro Commerce, Thunder commented
Version: 9.0.x-dev » 8.9.x-dev
Status: Reviewed & tested by the community » Fixed

Committed and pushed 3dbcad2dd1 to 9.1.x and ce6bafa10a to 9.0.x and 426b129607 to 8.9.x. Thanks!

As a docs fix backported to 8.9.x

  • alexpott committed 3dbcad2 on 9.1.x 
    Issue #3159102 by Niklan: Documentation for \Drupal\serialization\...

  • alexpott committed ce6bafa on 9.0.x 
    Issue #3159102 by Niklan: Documentation for \Drupal\serialization\...

  • alexpott committed 426b129 on 8.9.x 
    Issue #3159102 by Niklan: Documentation for \Drupal\serialization\...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.