cviking26_issue-9489 // added query param to url build - description …

[cviking26]

There is an error while click on "resend confirmation mail". The mail address will transfered without special chars like "+". This is important for the new gmail feature which provides simply adding aliases.

Fixed Issues

• Error while resend account confirmation link with special chars (e.g. "+" in email) #9489

Preconditions

• Magento 2.1.2
• Enable Double-OptIn Mails for Customer Registration

Steps to reproduce

• Register as a new Customer with email like "[email protected]" (customer/account/create/)
• After Redirect to customer/account/login/ click on "Resend Confirmation Link"

Actual Result

• This redirect to customer/account/confirmation/email/[email protected]/ - The Parameter email in url is not urlencoded, so the param is missing in the input field.
  Parameter on this page in input field is now "firstname.lastname [email protected]" without the plus

Expected result

• Click on "Resend Confirmation Link"
• Get URL encoded mail in input field on customer/account/confirmation/email/firstname.lastname%[email protected]/
  so Parameter in input field is with "+" char e.g. "[email protected]"

Solution

Bug can solved simply adding "_query" in htdocs/vendor/magento/module-customer/Model/Url.php:221

return $this->urlBuilder->getUrl('customer/account/confirmation', array('_query' => ['email' => $email]));

[magento-cicd2]

All committers have signed the CLA.

[ishakhsuvarov]

Please use short array syntax ([] instead of array())

[cviking26]

@ishakhsuvarov thank's. Adapted and pushed.

[ishakhsuvarov]

@cviking26 Thank you for the update. We are going to proceed with review and merge