- Issue created by @bluegeek9
- ๐ฎ๐ณIndia Rajas Samse
Rajas Samse โ made their first commit to this issueโs fork.
- Assigned to bluegeek9
- ๐ฎ๐ณIndia Rajas Samse
Apologies for the delay. I've resolved the issue #3420267 but unfortunately, I'm facing an error when attempting to submit the merge request. I've included the file contents in the comments. Will submit ASAP once the GitLab glitch is sorted.
AuthmapAlterSubscriber.php file
<?php
declare(strict_types=1);
namespace Drupal\authorization\EventSubscriber;
use Drupal\externalauth\Event\ExternalAuthEvents;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
/**
* Authmap event subscriber.
*/
class AuthmapAlterSubscriber implements EventSubscriberInterface {
/**
* Action to take on authorization.
*/
// method removed
/**
* {@inheritdoc}
*/
public static function getSubscribedEvents() {
$events[ExternalAuthEvents::AUTHMAP_ALTER][] = ['onAuthmapAlter'];
return $events;
}
}LoginSubscriber.php file
<?php
declare(strict_types=1);
namespace Drupal\authorization\EventSubscriber;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
use Symfony\Component\HttpKernel\KernelEvents;
/**
* Login subscriber.
*/
class LoginSubscriber implements EventSubscriberInterface {
/**
* Action on request.
*/
// removed request method
/**
* Action on login.
*/
// removed onLogin method
/**
* {@inheritdoc}
*/
public static function getSubscribedEvents() {
$events[KernelEvents::REQUEST][] = ['onRequest'];
return $events;
}
}RegisterSubscriber.php file
<?php
declare(strict_types=1);
namespace Drupal\authorization\EventSubscriber;
use Drupal\externalauth\Event\ExternalAuthEvents;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
/**
* Registration subscriber.
*/
class RegisterSubscriber implements EventSubscriberInterface {
/**
* Action on register.
*/
// removed onRegister method
/**
* {@inheritdoc}
*/
public static function getSubscribedEvents() {
$events[ExternalAuthEvents::REGISTER][] = ['onRegister'];
return $events;
}
} - ๐บ๐ธUnited States bluegeek9
@Rajas Samse,
Please don't commit to my branch. If you don't use Drupal you shouldn't be submitting patches.
- ๐ฎ๐ณIndia Rajas Samse
Understood. I appreciate your concern. If there are specific issues with my code, please provide feedback so I can address them. I'm committed to following project guidelines.
-
bluegeek9 โ
committed a330dd3b on 8.x-1.x
Issue #3420267: Remove EventSubscriber
-
bluegeek9 โ
committed a330dd3b on 8.x-1.x
- Issue was unassigned.
- Status changed to Fixed
9 months ago 9:51pm 12 February 2024 Automatically closed - issue fixed for 2 weeks with no activity.