Follow-up for #2831187: Mark everything inside Drupal\jsonapi\Context, Drupal\jsonapi\Query and Drupal\jsonapi\Routing as @internal.

Just like #2831185: Rename DocumentWrapper to JsonApiDocumentTopLevel and DocumentRootNormalizer to JsonApiDocumentTopLevelNormalizer removed the of an internal class (a JSON API-specific normalizer), we should do the same for all other JSON API-specific normalizers. They'll never have alternative implementations. Hence it's useless to have interfaces for them.

Also mark these classes as @internal.

CommentFileSizeAuthor
#3 jsonapi_normalizer_interfaces-2840948-2.patch12.67 KBwim leers

Comments

Wim Leers created an issue. See original summary.

e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
commented

+1

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Status: Active » Needs review
StatusFileSize
new jsonapi_normalizer_interfaces-2840948-2.patch12.67 KB
wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Priority: Normal » Minor

  • e0ipso committed 52284e3 on 8.x-1.x authored by Wim Leers 
    fix(Maintainability) Remove interfaces of JSON API-specific normalizers...
e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
commented
Status: Needs review » Fixed

Merged. Thanks!

Status: Fixed » Closed (fixed)

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