Skip to content

Disable user verification process when running an import #8690

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
mihaijoldis opened this issue Jun 9, 2021 · 2 comments · May be fixed by #8692
Open

Disable user verification process when running an import #8690

mihaijoldis opened this issue Jun 9, 2021 · 2 comments · May be fixed by #8692

Comments

@mihaijoldis
Copy link
Contributor

Enhancement Request

Explain your enhancement (please be detailed)

Logging this after a complaint on the forums here https://wordpress.org/support/topic/email-sent-upon-import/

Justification or use case

Following the scenario of that user to import payments after a server reinstall or any other reasons we could disable the process from running all together.

@robincornett
Copy link
Contributor

It looks like if we add:

remove_action( 'edd_customer_post_attach_payment', 'edd_connect_guest_customer_to_existing_user', 10, 4 );

to the batch payments import init function, it will prevent the emails from being sent.

Noting also that I'm getting quite a few PHP errors when some payments are imported, starting with:

PHP Notice:  Undefined variable: customer in \plugins\easy-digital-downloads\includes\admin\import\class-batch-import-payments.php on line 455

@robincornett
Copy link
Contributor

Note: we need to also make sure that the users are not marked as pending (needing verification) during the import process. I think they won't be, as that happens within the edd_connect_guest_customer_to_existing_user function, but noting to make sure we confirm.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants