We have to use entityTypeManager instead of entityManager in the code base

CommentFileSizeAuthor
#5 2874729-5.patch1005 bytespiggito
#2 2874729-2.patch1.61 KBPavan B S
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Pavan B S created an issue. See original summary.

Pavan B S’s picture

Pavan B S CreditAttribution: Pavan B S at Valuebound commented
Assigned: Pavan B S » Unassigned
Status: Active » Needs review
FileSize
2874729-2.patch1.61 KB

Applying the patch, please review.

Status: Needs review » Needs work

The last submitted patch, 2: 2874729-2.patch, failed testing.

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
CreditAttribution: colan at Colan Schwartz Consulting commented

Patch no longer applies. Needs reroll.

piggito’s picture

piggito CreditAttribution: piggito as a volunteer and at Skilld commented
Status: Needs work » Needs review
FileSize
2874729-5.patch1005 bytes

New version of patch for current 8.x-1.x branch

nkoporec’s picture

nkoporec
Primary language English
Location Slovenia
CreditAttribution: nkoporec as a volunteer and at Agiledrop - Your Trusted Drupal Teammates commented
Status: Needs review » Reviewed & tested by the community

Tested your patch @piggito and it's working.Moving to RTBC.

  • colan committed 4843233 on 8.x-1.x authored by Pavan B S 
    Issue #2874729 by Pavan B S, piggito: Replace all usages of deprecated...
colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
CreditAttribution: colan at Colan Schwartz Consulting commented
Status: Reviewed & tested by the community » Fixed

Thanks!

Status: Fixed » Closed (fixed)

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