Drupal 10 compatibility for entity_reference_integrity_enforce

Proposed wrong resolution. Changed the text.

The patch should take of the deprecation. However, on my local dev machine, the tests are skipped.

It's still missing the core_version_requirement for the entity_reference_integrity_enforce module to be compatible.

This needs to be added along with the core_version_requirement for the entity_reference_integrity_enforce module.

This module is flagged compatible with d10 but isn't until this issues if fixed.

It's still missing the core_version_requirement for the entity_reference_integrity_enforce module to be compatible.

I committed this directly to the 8.x-1.x branch: https://git.drupalcode.org/project/entity_reference_integrity/-/commit/7...

In reviewing this issue, it looks like we can simply delete the entire testHookWeightLegacy() method, since the testHook() tests the same thing, using a method that exists.

I plan to start a new 2.x branch that drops support for Drupal 9, so we can remove testHookWeightLegacy() in that branch and simplify all of this.

Renaming this issue to specifically focus on removing the deprecated method.

Here is the relevant change record which describes the recommended approach: https://www.drupal.org/node/3000490 →

// Getting a list of hook implementors is as simple as making use of the $module variable.
$implementors = [];
$this->moduleHandler->invokeAllWith($hook, function (callable $hook, string $module) use (&$implementors) {
  // There is minimal overhead since the hook is not invoked.
  $implementors[] = $module;
});

The purpose of this test is to ensure that entity_reference_integrity_enforce's implementation of hook_entity_predelete() comes before that of the comment module (to confirm that the code in hook_module_implements_alter() is doing its job).

I will refactor the test to make this more clear.

Source: #2831346: hook_entity_predelete is too late to throw an exception →

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