Fix usage of current request in the PasswordPolicyEventSubscriber

I am working on it.

Issue fixed.

Needs work to fix missing space on $token =$event->

1. +++ b/src/EventSubscriber/PasswordPolicyEventSubscriber.php
   @@ -106,7 +95,7 @@ class PasswordPolicyEventSubscriber implements EventSubscriberInterface {
   -      $is_ajax = $this->request->headers->get('X_REQUESTED_WITH') === 'XMLHttpRequest';
   +      $is_ajax = $event->getRequest()->headers->get('X_REQUESTED_WITH') === 'XMLHttpRequest';
   

   The request method provides isXmlHttpRequest which could replace this check. That change may be considered out of scope, though.

2. +++ b/src/EventSubscriber/PasswordPolicyEventSubscriber.php
   @@ -126,8 +115,8 @@ class PasswordPolicyEventSubscriber implements EventSubscriberInterface {
   +      $token =$event->getRequest()->query->get('pass-reset-token');
   

   nit: missing space after =

HI @mglaman,

Changes you suggest has been done, Please review.

Thanks
Samit K.

Thanks! Looks great.

Assigning to myself as I'm reviewing/merging ready RTBC fixes/updates over the next few days.

Thanks for the work on this. I reviewed the code and tested various things with 10.1.8, and it worked as expected.

I wouldn't call this a bug though, so moving to task.

The fix has been merged and will be part of the next release.

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

This is part of the new 4.0.1 release → .