Problem/Motivation

NormalizerBase currently extends SerializerAwareNormalizer, which is deprecated since Symfony 3.1 and will be removed in version 4.0. It is suggested that SerializerAwareTrait be used instead.

Proposed resolution

Alter NormalizerBase so it does not extend SerializerAwareNormalizer, but implements SerializerAwareInterface directly and uses SerializerAwareTrait;

Remaining tasks

None.

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#2 2958292-2.patch962 bytesrlhawk
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rlhawk created an issue. See original summary.

rlhawk’s picture

Status: Active » Needs review
FileSize
962 bytes

Here's a patch that implements this change.

Wim Leers’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Symfony2

Perfect! Thank you for helping us stay on top of things! I hope to see you around more, it's extremely rare that one can RTBC the first iteration of a patch! :D (Be proud!)

Wim Leers’s picture

Priority: Normal » Minor
alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 33519ea and pushed to 8.6.x. Thanks!

  • alexpott committed 33519ea on 8.6.x
    Issue #2958292 by rlhawk: Use SerializerAwareTrait instead of extending...

Status: Fixed » Closed (fixed)

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