MAGETWO-95238: Cannot reset customer password from Admin Panel

Author: irenelagno

app/code/Magento/Customer/Block/Adminhtml/Edit/ResetPasswordButton.php

@@ -25,10 +25,7 @@ public function getButtonData()
             $data = [
                 'label' => __('Reset Password'),
                 'class' => 'reset reset-password',
-                'data_attribute' => [
-                    'url' => $this->getResetPasswordUrl()
-                ],
-                'on_click' => '',
+                'on_click' => sprintf("location.href = '%s';", $this->getResetPasswordUrl()),
                 'sort_order' => 60,
             ];
         }

app/code/Magento/Customer/Controller/Adminhtml/Index/ResetPassword.php

@@ -5,11 +5,11 @@
  */
 namespace Magento\Customer\Controller\Adminhtml\Index;
 
-use Magento\Framework\App\Action\HttpPostActionInterface;
+use Magento\Framework\App\Action\HttpGetActionInterface;
 use Magento\Framework\Exception\NoSuchEntityException;
 use Magento\Framework\Exception\SecurityViolationException;
 
-class ResetPassword extends \Magento\Customer\Controller\Adminhtml\Index implements HttpPostActionInterface
+class ResetPassword extends \Magento\Customer\Controller\Adminhtml\Index implements HttpGetActionInterface
 {
     /**
      * Reset password handler

app/code/Magento/Customer/view/adminhtml/web/edit/post-wrapper.js

@@ -44,12 +44,4 @@ define([
 
         return false;
     });
-
-    $('#resetPassword').click(function () {
-        var url = $('#resetPassword').data('url');
-
-        getForm(url).appendTo('body').submit();
-
-        return false;
-    });
 });

dev/tests/integration/testsuite/Magento/Customer/Controller/Adminhtml/Index/ResetPasswordTest.php

@@ -34,7 +34,7 @@ public function testResetPasswordSuccess()
         $this->passwordResetRequestEventCreate(
             \Magento\Security\Model\PasswordResetRequestEvent::CUSTOMER_PASSWORD_RESET_REQUEST
         );
-        $this->getRequest()->setPostValue(['customer_id' => '1'])->setMethod(HttpRequest::METHOD_POST);
+        $this->getRequest()->setPostValue(['customer_id' => '1'])->setMethod(HttpRequest::METHOD_GET);
         $this->dispatch('backend/customer/index/resetPassword');
         $this->assertSessionMessages(
             $this->equalTo(['The customer will receive an email with a link to reset password.']),
@@ -44,18 +44,18 @@ public function testResetPasswordSuccess()
     }
 
     /**
-     * Checks reset password functionality cannot be performed with GET request
+     * Checks reset password functionality cannot be performed with POST request
      *
      * @magentoConfigFixture current_store customer/password/limit_password_reset_requests_method 0
      * @magentoConfigFixture current_store customer/password/min_time_between_password_reset_requests 0
      * @magentoDataFixture Magento/Customer/_files/customer.php
      */
-    public function testResetPasswordWithGet()
+    public function testResetPasswordWithPost()
     {
         $this->passwordResetRequestEventCreate(
             \Magento\Security\Model\PasswordResetRequestEvent::CUSTOMER_PASSWORD_RESET_REQUEST
         );
-        $this->getRequest()->setPostValue(['customer_id' => '1'])->setMethod(HttpRequest::METHOD_GET);
+        $this->getRequest()->setPostValue(['customer_id' => '1'])->setMethod(HttpRequest::METHOD_POST);
         $this->dispatch('backend/customer/index/resetPassword');
         $this->assertEquals('noroute', $this->getRequest()->getControllerName());
     }
@@ -74,7 +74,7 @@ public function testResetPasswordMinTimeError()
         $this->passwordResetRequestEventCreate(
             \Magento\Security\Model\PasswordResetRequestEvent::CUSTOMER_PASSWORD_RESET_REQUEST
         );
-        $this->getRequest()->setPostValue(['customer_id' => '1'])->setMethod(HttpRequest::METHOD_POST);
+        $this->getRequest()->setPostValue(['customer_id' => '1'])->setMethod(HttpRequest::METHOD_GET);
         $this->dispatch('backend/customer/index/resetPassword');
         $this->assertSessionMessages(
             $this->equalTo(['The customer will receive an email with a link to reset password.']),
@@ -97,7 +97,7 @@ public function testResetPasswordLimitError()
         $this->passwordResetRequestEventCreate(
             \Magento\Security\Model\PasswordResetRequestEvent::CUSTOMER_PASSWORD_RESET_REQUEST
         );
-        $this->getRequest()->setPostValue(['customer_id' => '1'])->setMethod(HttpRequest::METHOD_POST);
+        $this->getRequest()->setPostValue(['customer_id' => '1'])->setMethod(HttpRequest::METHOD_GET);
         $this->dispatch('backend/customer/index/resetPassword');
         $this->assertSessionMessages(
             $this->equalTo(['The customer will receive an email with a link to reset password.']),
@@ -122,7 +122,7 @@ public function testResetPasswordWithSecurityViolationException()
         $this->passwordResetRequestEventCreate(
             \Magento\Security\Model\PasswordResetRequestEvent::ADMIN_PASSWORD_RESET_REQUEST
         );
-        $this->getRequest()->setPostValue(['customer_id' => '1'])->setMethod(HttpRequest::METHOD_POST);
+        $this->getRequest()->setPostValue(['customer_id' => '1'])->setMethod(HttpRequest::METHOD_GET);
         $this->dispatch('backend/customer/index/resetPassword');
         $this->assertSessionMessages(
             $this->equalTo(['The customer will receive an email with a link to reset password.']),