Skip to content
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

2.3.4 Invoice always sent (if exist) when order placed. #27656

Closed
ilnytskyi opened this issue Apr 8, 2020 · 6 comments · Fixed by #32580
Closed

2.3.4 Invoice always sent (if exist) when order placed. #27656

ilnytskyi opened this issue Apr 8, 2020 · 6 comments · Fixed by #32580
Assignees
Labels
Component: Quote Fixed in 2.4.x The issue has been fixed in 2.4-develop branch Issue: Confirmed Gate 3 Passed. Manual verification of the issue completed. Issue is confirmed Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development Priority: P1 Once P0 defects have been fixed, a defect having this priority is the next candidate for fixing. Progress: done Reported on 2.3.4 Indicates original Magento version for the Issue report. Reproduced on 2.4.x The issue has been reproduced on latest 2.4-develop branch Severity: S1 Affects critical data or functionality and forces users to employ a workaround. Triage: Done Has been reviewed and prioritized during Triage with Product Managers

Comments

@ilnytskyi
Copy link
Contributor

ilnytskyi commented Apr 8, 2020

Preconditions (*)

related to attempt to fix
#13466

commit
5ccd367

  1. h3. Steps to reproduce (*)

  2. Place order with a payment method that instantly generates an invoice
  3. The module/code that generated an invoice now is responsible for checking it's state and send to customer
  4. but here 896ae3b#diff-5e96777dde905631ad1729751f5cc83fR72 magento checks if invoice exist and send it to customer
  5. If so, pleas give the possibility to disable this in sales*model_service_quote_submit*success without code modifying or changing \Magento\Quote\Observer\SubmitObserver::execute or its dependencies

Expected result (*)

  1. Invoice Sending is managed by modules that created this invoice and should be configurable
  2. Invoice is not sent the same time the order placed
  3. or this feature is disabled and can be configured

Actual result (*)

  1. Invoice is always sent if exists 896ae3b#diff-5e96777dde905631ad1729751f5cc83fR7
  2. The original intention to fix Invoice Sales Email not send #13466 if so the PayPal module is responsible for sending its invoice NOT MAGENTO QUOTE MODULE
@m2-assistant
Copy link

m2-assistant bot commented Apr 8, 2020

Hi @ilnytskyi. Thank you for your report.
To help us process this issue please make sure that you provided the following information:

  • Summary of the issue
  • Information on your environment
  • Steps to reproduce
  • Expected and actual results

Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:

@magento give me 2.4-develop instance - upcoming 2.4.x release

For more details, please, review the Magento Contributor Assistant documentation.

@ilnytskyi do you confirm that you were able to reproduce the issue on vanilla Magento instance following steps to reproduce?

  • yes
  • no

@ghost ghost added this to Ready for QA in Community Backlog Apr 8, 2020
@magento-engcom-team magento-engcom-team added the Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed label Apr 8, 2020
@ilnytskyi ilnytskyi changed the title 2.3.4 Invoice always sent if exist when order placed. 2.3.4 Invoice always sent (if exist) when order placed. Apr 8, 2020
@ilnytskyi
Copy link
Contributor Author

ilnytskyi added a commit to ilnytskyi/magento2 that referenced this issue Apr 11, 2020
@ghost ghost assigned ilnytskyi Apr 11, 2020
@ghost ghost moved this from Ready for QA to PR In Progress in Community Backlog Apr 11, 2020
@sdzhepa sdzhepa added Triage: Ready for Triage Issue is ready to me triaged with Product Manager Priority: P1 Once P0 defects have been fixed, a defect having this priority is the next candidate for fixing. Severity: S1 Affects critical data or functionality and forces users to employ a workaround. labels Apr 30, 2020
@sdzhepa sdzhepa linked a pull request Apr 30, 2020 that will close this issue
4 tasks
@sdzhepa sdzhepa added Triage: Done Has been reviewed and prioritized during Triage with Product Managers Component: Quote Reproduced on 2.4.x The issue has been reproduced on latest 2.4-develop branch Issue: Confirmed Gate 3 Passed. Manual verification of the issue completed. Issue is confirmed and removed Triage: Ready for Triage Issue is ready to me triaged with Product Manager labels Apr 30, 2020
@magento-engcom-team
Copy link
Contributor

✅ Confirmed by @sdzhepa
Thank you for verifying the issue. Based on the provided information internal tickets MC-33893 were created

Issue Available: @sdzhepa, You will be automatically unassigned. Contributors/Maintainers can claim this issue to continue. To reclaim and continue work, reassign the ticket to yourself.

@magento-engcom-team magento-engcom-team added the Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development label Apr 30, 2020
@sidolov sidolov added this to Ready for Development in Low Priority Backlog Sep 24, 2020
@m2-community-project m2-community-project bot moved this from Ready for Development to Pull Request In Progress in Low Priority Backlog Sep 24, 2020
@m2-community-project m2-community-project bot added Progress: PR Created Indicates that Pull Request has been created to fix issue and removed Progress: PR in progress labels Sep 24, 2020
@sidolov sidolov added this to Pull Request In Progress in High Priority Backlog Oct 20, 2020
@ghost ghost removed this from PR In Progress in Community Backlog Oct 20, 2020
@ghost ghost removed this from Pull Request In Progress in Low Priority Backlog Oct 20, 2020
@ghost ghost added Progress: PR in progress and removed Progress: PR Created Indicates that Pull Request has been created to fix issue labels Oct 20, 2020
@magento-engcom-team magento-engcom-team added the Reported on 2.3.4 Indicates original Magento version for the Issue report. label Nov 13, 2020
@m2-community-project m2-community-project bot moved this from Pull Request In Progress to Dev In Progress in High Priority Backlog Nov 13, 2020
@m2-community-project m2-community-project bot moved this from Dev In Progress to Pull Request In Progress in High Priority Backlog Mar 23, 2021
@m2-community-project m2-community-project bot moved this from Dev In Progress to Pull Request In Progress in High Priority Backlog Mar 23, 2021
@zakdma
Copy link
Contributor

zakdma commented Mar 23, 2021

@magento i am working on it

@zakdma
Copy link
Contributor

zakdma commented Mar 23, 2021

We discussed this task and made decision we'll create separate observer for invoice emails so isolate invoice email sending from order email sending. Developer can disable the observer following the documentation instructions https://devdocs.magento.com/guides/v2.4/extension-dev-guide/events-and-observers.html
Also invoice email sending will depend on sales_email/invoice/enabled configuration (Yes by default)
So functionality will work as before but there is possibility to change it behaviour as developer need.

@magento-engcom-team
Copy link
Contributor

Hi @ilnytskyi, @ilnytskyi, @zakdma.

Thank you for your report and collaboration!

The related internal Jira ticket MC-33893 was closed as Fixed.

The fix will be available with the upcoming 2.4.3 release.

@m2-community-project m2-community-project bot moved this from Pull Request In Progress to Done in High Priority Backlog Mar 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Quote Fixed in 2.4.x The issue has been fixed in 2.4-develop branch Issue: Confirmed Gate 3 Passed. Manual verification of the issue completed. Issue is confirmed Issue: Format is valid Gate 1 Passed. Automatic verification of issue format passed Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development Priority: P1 Once P0 defects have been fixed, a defect having this priority is the next candidate for fixing. Progress: done Reported on 2.3.4 Indicates original Magento version for the Issue report. Reproduced on 2.4.x The issue has been reproduced on latest 2.4-develop branch Severity: S1 Affects critical data or functionality and forces users to employ a workaround. Triage: Done Has been reviewed and prioritized during Triage with Product Managers
Projects
5 participants