Uncompatibilities with Drupal 9

Fix the incompatibilities

- Fix error -> Call to deprecated function drupal_set_message(). Deprecated in drupal:8.5.0 and is removed from drupal:9.0.0. Use \Drupal\Core\Messenger\MessengerInterface::addMessage() instead.

- Fix warning -> Class Drupal\events_logging\Plugin\EventLog\Storage\DatabaseStorageBackend was not found while trying to analyse it - autoloading is probably not configured properly.

- Fix warning -> Add core_version_requirement: ^8 || ^9 to events_logging.info.yml to designate that the module is compatible with Drupal 9. See https://www.drupal.org/node/3070687.

Original report by @michele.lucchina and @roberto.muzzano

CommentFileSizeAuthor
#4 3108826-4.patch3.5 KBroberto.muzzano

Comments

michele.lucchina created an issue. See original summary.

michele.lucchina’s picture

Issue tags: +ContributionWeekend2020IT
michele.lucchina’s picture

Issue tags: +ContributionWeekend2020Milan
roberto.muzzano’s picture

StatusFileSize
new3.5 KB

Fix the incompatibilities

Fixed error
Call to deprecated function drupal_set_message(). Deprecated in drupal:8.5.0 and is removed from drupal:9.0.0. Use \Drupal\Core\Messenger\MessengerInterface::addMessage() instead.

Fixed warning
Class Drupal\events_logging\Plugin\EventLog\Storage\DatabaseStorageBackend was not found while trying to analyse it - autoloading is probably not configured properly.

Fixed warning
Add core_version_requirement: ^8 || ^9 to events_logging.info.yml to designate that the module is compatible with Drupal 9. See https://www.drupal.org/node/3070687.

michele.lucchina’s picture

Status: Active » Needs review
reinchek’s picture

I've tested the #4 comment's patch. It's worked correctly.

reinchek’s picture

Status: Needs review » Reviewed & tested by the community
avpaderno’s picture

Issue tags: -ContributionWeekend2020IT, -ContributionWeekend2020Milan
kristen pol’s picture

Issue tags: +Drupal 9 compatibility
jonnyeom’s picture

Thoughts on merging this in?

rafuel92’s picture

merged in 8.x-1.5

rafuel92’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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