Allow other modules to prevent redirection (e.g. Commerce)

Hi Everyone,

We will include all the changes in new release with Drupal 10 compatibility.

pvbergen → changed the visibility of the branch 3201520-integration-with-commerce to hidden.

The changes haven't made it into 2.x or 3.x yet. Updated the issue and MR for 3.0.x.

Instead of adding more routes to the list, it would be a better idea to dispatch an event that can prevent the redirection. That would fix the problem for all possible use cases.

I added a new MR that dispatches an event allowing any module to prevent redirection, or alter the redirection URL.

To prevent redirection on the Commerce checkout form, a subscriber can be implemented that checks the route:

<?php

declare(strict_types=1);

namespace Drupal\ekw_test\EventSubscriber;

use Drupal\Core\Routing\RouteMatchInterface;
use Drupal\redirect_after_login\Event\RedirectAfterLoginEvent;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;

/**
 * Subscribes to the RedirectAfterLoginEvent.
 */
class MySubscriber implements EventSubscriberInterface {

  /**
   * Constructs a MySubscriber object.
   *
   * @param \Drupal\Core\Routing\RouteMatchInterface $routeMatch
   *   The current route match.
   */
  public function __construct(
    protected RouteMatchInterface $routeMatch,
  ) {}

  /**
   * Prevent redirect after login in the Commerce checkout.
   */
  public function onRedirectAfterLogin(RedirectAfterLoginEvent $event) {
    $current_route = $this->routeMatch->getRouteName();
    if ($current_route === 'commerce_checkout.form') {
      $event->setRedirectAllowed(FALSE);
    }
  }

  /**
   * {@inheritdoc}
   */
  public static function getSubscribedEvents() {
    return [RedirectAfterLoginEvent::class => 'onRedirectAfterLogin'];
  }

}

Works perfectly @pfrenssen thanks!

I had to use this on the "auto_login_url.login" route otherwise it was breaking the auto login url module.