Fix/34127

[alexiglesias31]

Submission Review Guidelines:

• I have followed the WooCommerce Contributing Guidelines and the WordPress Coding Standards.
• I have checked to ensure there aren't other open Pull Requests for the same update/change.
• I have reviewed my code for security best practices.
• Following the above guidelines will result in quick merges and clear and detailed feedback when appropriate.

Changes proposed in this Pull Request:

Prevent to call to login function if the user is already logged in.

Closes woocommerce#34127.

How to test the changes in this Pull Request:

1. New user is created during checkout.
2. User gets a link in email to set their password.
3. User follows link and sets new password.
4. User is taken to the login page with message “Your password has been reset successfully.”
5. User enters email and password, clicks “Login”. The login operation should be successful at the first try.

[github-actions]

Hi @dpanta94,

Apart from reviewing the code changes, please make sure to review the testing instructions as well.

You can follow this guide to find out what good testing instructions should look like:
https://github.com/woocommerce/woocommerce/wiki/Writing-high-quality-testing-instructions

[dpanta94]

I think only missing the changelog and if we could clear the commits from other contributors would be great

[alexiglesias31]

Got it, the other commits are because our repo is outdated, need to re-updated it

[dpanta94]

Just refetched the trunk branch here, a merge could resolve it

[msaggiorato]

We likely need to rebase here

[msaggiorato]

We'll need to exclude these files from being in the commits

[msaggiorato]

We are including more than 1 patch here.

[msaggiorato]

This seems like a whitespace change related to the gitattributes

[dpanta94]

@alexiglesias31 if already accepted in woo lets close this 😄