This error has just started appearing in Travis testing

1x: \Drupal::entityManager() is deprecated in Drupal 8.0.0 and will be removed before Drupal 9.0.0. Use \Drupal::entityTypeManager() instead in most cases. If the needed method is not on \Drupal\Core\Entity\EntityTypeManagerInterface, see the deprecated \Drupal\Core\Entity\EntityManager to find the correct interface or service. See https://www.drupal.org/node/2549139
1x in SchedulerDefaultTimeTest::testDefaultTime from Drupal\Tests\scheduler\Functional

1x: EntityManagerInterface::getStorage() is deprecated in drupal:8.0.0 and is removed from drupal:9.0.0. Use \Drupal\Core\Entity\EntityTypeManager::getStorage() instead. See https://www.drupal.org/node/2549139
1x in SchedulerDefaultTimeTest::testDefaultTime from Drupal\Tests\scheduler\Functional

We had removed all ocurrences in #2867272: Replace all usages of deprecated EntityManager but because Core did not actually create a deprecation message (until now) this allowed me to inadvertantly add back a new one in SchedulerDefaultTimeTest.php in #2957490: Default time tests fail in run-up to Daylight Saving clock change

CommentFileSizeAuthor
#2 3059741-2.entity_manager.patch1.14 KBjonathan1055

Comments

jonathan1055 created an issue. See original summary.

jonathan1055’s picture

added 'suppress-deprecations: false' into drupalci.yml to show that this patch fixes the warning.

  • jonathan1055 committed 9c6f9b6 on 8.x-1.x
    Issue #3059741 by jonathan1055: Replace deprecated Drupal::entityManager...
jonathan1055’s picture

Title: Replace deprecated Drupal::entityManager() with \Drupal::entityTypeManager() » Replace deprecated entityManager() with entityTypeManager()
Status: Needs review » Fixed

Fixed.
Decided to add 'suppress-deprecations: false' permanently, so that we get alerted right away for new deprecation triggers.

Status: Fixed » Closed (fixed)

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