Remove deprecated usage:

Call to deprecated method applyUpdates() of class Drupal\Core\Entity\EntityDefinitionUpdateManagerInterface

CommentFileSizeAuthor
#8 3072870-remove-cache-and-updates-8.patch84.81 KBpenyaskito
#4 3072870-4.patch66.24 KBpiotr pakulski
#2 3072870-2.patch89.35 KBDeeksha B

Comments

penyaskito created an issue. See original summary.

Deeksha B’s picture

Deeksha B commented
Status: Active » Needs review
StatusFileSize
new 3072870-2.patch89.35 KB

Please review the patch.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Status: Needs review » Needs work
piotr pakulski’s picture

piotr pakulski
Location Poland 🇪🇺
commented
Issue tags: +deprecated, +GCWPoland2020, +GlobalContributionWeekend2020
StatusFileSize
new 3072870-4.patch66.24 KB

Removing deprecated applyUpdates()

c-logemann’s picture

c-logemann
he/him
Primary language German
Location Frankfurt/M, Germany
commented
Issue tags: -GlobalContributionWeekend2020 +ContributionWeekend2020

"ContributionWeekend2020“ is the "official" tag suggested on global event page.

piotr pakulski’s picture

piotr pakulski
Location Poland 🇪🇺
commented
Status: Needs work » Needs review
penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Status: Needs review » Reviewed & tested by the community

This looks great! Thanks for contributing!

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new 3072870-remove-cache-and-updates-8.patch84.81 KB

I'm wondering if this is needed at all after all.

Status: Needs review » Needs work

The last submitted patch, 8: 3072870-remove-cache-and-updates-8.patch, failed testing. View results

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Status: Needs work » Reviewed & tested by the community

Let's commit #4 and take care of #8 in #3116494: Remove unnecessary caches clearing in tests that slow down the test suite

  • penyaskito committed 73cc27e on 8.x-2.x authored by paqu83 
    Issue #3072870 by penyaskito, paqu83, Deeksha B, C_Logemann:  Call to...
penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Status: Reviewed & tested by the community » Fixed

Committed 73cc27e and pushed to 8.x-2.x. Thanks!

Status: Fixed » Closed (fixed)

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