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

FIX. Enable re-sending again - http://magento.stackexchange.com/a/376… #64

Merged
merged 2 commits into from
Oct 25, 2016

Conversation

seansan
Copy link
Contributor

@seansan seansan commented Feb 6, 2016

…90/78

FIX. Enable re-sending again - http://magento.stackexchange.com/a/37690/78

Irritating "new" feature so order emails cannot be resent ... fixed

…90/78

FIX. Enable re-sending again - http://magento.stackexchange.com/a/37690/78

Irritating "new" feature so order emails cannot be resent ... fixed
@LeeSaferite
Copy link
Contributor

I think the better solution is to switch to queueNewOrderEmail(true)

@seansan
Copy link
Contributor Author

seansan commented Mar 23, 2016

As long as the problem is fixed ;P Can you create a pull request and add it?

@shirtsofholl
Copy link
Contributor

Can this be accepted?

@sylvainraye
Copy link

+1 for the @shirtsofholl solution

@@ -136,6 +136,8 @@ public function emailAction()
{
if ($order = $this->_initOrder()) {
try {
// FIX. Enable re-sending again - http://magento.stackexchange.com/a/37690/78
$order->setEmailSent(false);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tabs vs spaces... We gotta have some standards.. :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agree .... +1 .. just get this one in ;)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Preferred way to fix is while on your PR branch just replace the tabs with spaces, use git commit -a --amend to fix the existing commit and then git push -f to force-push over the old commit.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think fixed now no?

@colinmollenhour colinmollenhour merged commit 8da96ce into OpenMage:1.9.2.3 Oct 25, 2016
@colinmollenhour
Copy link
Member

Merged 1.9.2.3 into 1.9.3.0

edannenberg pushed a commit to edannenberg/magento-lts that referenced this pull request Dec 6, 2017
@sreichel sreichel added bug review needed Problem should be verified labels Jan 11, 2018
@sreichel sreichel self-assigned this Jan 11, 2018
edannenberg pushed a commit to edannenberg/magento-lts that referenced this pull request Feb 28, 2018
edannenberg pushed a commit to edannenberg/magento-lts that referenced this pull request Jul 17, 2018
edannenberg pushed a commit to edannenberg/magento-lts that referenced this pull request Sep 19, 2018
edannenberg pushed a commit to edannenberg/magento-lts that referenced this pull request Feb 14, 2019
edannenberg pushed a commit to edannenberg/magento-lts that referenced this pull request Apr 1, 2019
edannenberg pushed a commit to edannenberg/magento-lts that referenced this pull request Aug 22, 2019
edannenberg pushed a commit to edannenberg/magento-lts that referenced this pull request Oct 25, 2019
edannenberg pushed a commit to edannenberg/magento-lts that referenced this pull request Aug 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug review needed Problem should be verified
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants