Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 UTC on 18 March 2024, to get $100 off your ticket.
Comment | File | Size | Author |
---|---|---|---|
#4 | replace_entity_load_unchanged-2471593-4.patch | 1.45 KB | Devaraj johnson |
replace_entity_load_unchanged.patch | 1.31 KB | JeroenT | |
Comments
Comment #1
JeroenTComment #2
JeroenTComment #3
Mile23Patch still applies. :-)
EntityManager is deprecated: https://www.drupal.org/node/2549139
Also, we want to use
$this->container
in simpletest, instead of\Drupal
for service location.So something like:
Comment #4
Devaraj johnson CreditAttribution: Devaraj johnson as a volunteer and at Ameex-Drupal Geeks commentedI have updated the patch have a look
Comment #5
Devaraj johnson CreditAttribution: Devaraj johnson as a volunteer and at Ameex-Drupal Geeks commentedComment #6
Mile23Looks good, thanks.
I checked and the patch applies against 8.0.x and 8.1.x, so if this is the wrong branch for this issue then it can just swap out.
Comment #7
catchSince this is test only I think it's fine for both branches. Committed/pushed to 8.1.x and cherry-picked to 8.0.x. Thanks!