Deprecated method entityManager

CommentFileSizeAuthor
#7 deprecated_method-2865189-7.patch720 bytestassilogroeper
#2 deprecated_method_entitymanager-2865189-2.patch707 bytesalex_optim
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alex_optim created an issue. See original summary.

alex_optim’s picture

alex_optim
Location 🇺🇦 Rivne
CreditAttribution: alex_optim commented
Status: Active » Needs review
FileSize
deprecated_method_entitymanager-2865189-2.patch707 bytes

jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer commented
Status: Needs review » Fixed
yobottehg’s picture

yobottehg CreditAttribution: yobottehg commented
Priority: Normal » Critical
Status: Fixed » Needs work

The method does not exist on entityTypeManager, Head is currently broken.
This should at least be reverted.

jrockowitz’s picture

jrockowitz CreditAttribution: jrockowitz as a volunteer commented

Done! @see http://cgit.drupalcode.org/block_field/commit/?id=bbbad36

tassilogroeper’s picture

tassilogroeper CreditAttribution: tassilogroeper at WONDROUS commented
FileSize
deprecated_method-2865189-7.patch720 bytes

Hi,

I just ran into this again. Sorry to see this bug still in the main Branch :(

so here is the patch. Since the `loadEntityByUuid` has moved to the `EntityRepository` we need to fetch the service just in place there.

https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Entity%21...

tassilogroeper’s picture

tassilogroeper CreditAttribution: tassilogroeper at WONDROUS commented
Status: Needs work » Needs review
bdanin’s picture

bdanin CreditAttribution: bdanin commented
Status: Needs review » Reviewed & tested by the community

confirmed that #7 fixes the PHP fatal error.

berdyshev’s picture

berdyshev CreditAttribution: berdyshev at AMgrade commented

#7 works with latest dev version and fixes deprecation notice

acbramley’s picture

acbramley CreditAttribution: acbramley at PreviousNext for Transport for NSW commented
Status: Reviewed & tested by the community » Fixed

Thanks! Tagging 8.x-1.0-alpha5 for this fix.

Status: Fixed » Closed (fixed)

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