Drupal 10 compatibility

I created a patch using drupal-rector and grabbing the files it modified, after flushing cache and using Upgrade Status → , it recognizes the module as a compatible D10 one.

While testing the project where I'm using this module I found an error:

Drupal\Core\Entity\Query\QueryException: Entity queries must explicitly set whether the query should be access checked or not. See Drupal\Core\Entity\Query\QueryInterface::accessCheck(). in Drupal\Core\Entity\Query\Sql\Query->prepare() (line 141 of /app/public_html/core/lib/Drupal/Core/Entity/Query/Sql/Query.php).

I noticed in this module, some entity queries are run and needed that line, so I create a new patch for it.

I've merged the fixes in #6 with MR and removed Drupal 8 compatibility as Drupal 8 is long unsupported.

Merged into the main development branch. I will close and make a release after this has been tested on a project for some time to make sure we don't have any regressions.

Notes:
- I changed accessCheck() to accessCheck(TRUE); I prefer to be explicit with access control, it makes it easier to read if you don't remember what is the default.
- Removed a duplicate call to access check, kept the one before the subscriber so that subscribers can change it if they have a legitimate reason.
- I reverted the change in the declaration of public static function getSubscribedEvents() to match that of the interface - see Symfony\Component\EventDispatcher\EventSubscriberInterface.
- Good with removing Drupal 8 support, event dispatching is not compatible anymore.

Thanks!

@krystalcode return typehints can be more explicit than the interface. Reverting that just means you'll have to do it for Drupal 11 compatibility.

@alexpott why would that be the case? Drupal 11 relies on Symfony 7 (https://git.drupalcode.org/project/drupal/-/blob/11.x/core/composer.json...) which also has the same method declaration (https://github.com/symfony/event-dispatcher/blob/7.1/EventSubscriberInte...).

Updated credits.