Problem/Motivation

ContainerAware is deprecated, so it should be replaced by \Symfony\Component\DependencyInjection\ContainerAwareTrait

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner created an issue. See original summary.

jan.stoeckler’s picture

Assigned: Unassigned » jan.stoeckler
Issue tags: +neworleans2016

I'll try.

jibran’s picture

jan.stoeckler’s picture

Should work.

jan.stoeckler’s picture

Status: Active » Needs review

Sorry, status change.

dawehner’s picture

Status: Needs review » Reviewed & tested by the community

Looks perfect for me

jibran’s picture

Issue tags: +Quickfix

Nice work @jan.stoeckler

tim.plunkett’s picture

Title: Replace ContainerAware by ContainterAwareTrait » Replace ContainerAware with ContainerAwareTrait

Containter, heh.

alexpott’s picture

Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs reroll
jan.stoeckler’s picture

Status: Needs work » Needs review
Issue tags: -Needs reroll
FileSize
5.01 KB

This should apply cleanly (rebased changes on latest 8.2.x).

jibran’s picture

Status: Needs review » Reviewed & tested by the community

Thanks back to RTBC.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 2e5da97 and pushed to 8.2.x. Thanks!

  • alexpott committed 2e5da97 on 8.2.x
    Issue #2720891 by jan.stoeckler: Replace ContainerAware with...

Status: Fixed » Closed (fixed)

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