As entityManager() going to deprecate in future, it should be replace with entityTypeManager() in code.

CommentFileSizeAuthor
#4 2888901-4-lineendings.patch4.07 KBHongPong
#2 entity-manager-deprecated-method-2888901.patch573 bytesbapi_22
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bapi_22 created an issue. See original summary.

bapi_22’s picture

bapi_22 CreditAttribution: bapi_22 at Cybage Software Pvt Ltd. commented
Status: Active » Needs review
FileSize
entity-manager-deprecated-method-2888901.patch573 bytes

Please check attached patch.

bapi_22’s picture

bapi_22 CreditAttribution: bapi_22 at Cybage Software Pvt Ltd. commented
Assigned: bapi_22 » Unassigned
HongPong’s picture

HongPong CreditAttribution: HongPong as a volunteer and at kor group commented
FileSize
2888901-4-lineendings.patch4.07 KB

It appears that the file has Windows line endings. This patch has unix line endings and applied the patch against head, no other changes.

rlhawk’s picture

rlhawk
Primary language English
Location Seattle, Washington, United States
CreditAttribution: rlhawk commented
Status: Needs review » Reviewed & tested by the community

This looks good, thanks.

  • rlhawk committed e6042ac on 8.x-3.x authored by HongPong 
    Issue #2888901 by bapi_22, HongPong: Removed Deprecated usage of...
rlhawk’s picture

rlhawk
Primary language English
Location Seattle, Washington, United States
CreditAttribution: rlhawk commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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