15

我遇到了 Magento 单一优惠券代码的问题，该代码在客户单击“下订单”按钮时被标记为已使用。如果Paypal支付失败或客户在订单完成前离开页面，他将无法使用此设置为仅使用一次且已标记为已使用的优惠券返回重新订购。

我发现一段代码可以减少用户使用优惠券的次数，并允许他重复使用优惠券。不幸的是，他在点击下订单按钮时​​尝试连接 Paypal 页面时出现错误。为了能够再次使用优惠券并访问 Paypal 页面，我必须删除 SQL 数据库中表salesrule_coupon_usage和salesrule_customer中具有此客户 ID 的行。

这是我需要更改以自动删除客户 ID 的优惠券使用信息的代码：

public function cancel($observer)
{
    $order = $observer->getEvent()->getPayment()->getOrder();
    if ($order->canCancel()) {
        if ($code = $order->getCouponCode()) {
            $coupon = Mage::getModel('salesrule/coupon')->load($code, 'code');
            if ($coupon->getTimesUsed() > 0) {
                $coupon->setTimesUsed($coupon->getTimesUsed() - 1);
                $coupon->save();
            }

            $rule = Mage::getModel('salesrule/rule')->load($coupon->getRuleId());
            error_log("\nrule times used=" . $rule->getTimesUsed(), 3, "var/log/debug.log");
            if ($rule->getTimesUsed() > 0) {
                $rule->setTimesUsed($rule->getTimesUsed()-1);
                $rule->save();
            }

            if ($customerId = $order->getCustomerId()) {
                if ($customerCoupon = Mage::getModel('salesrule/rule_customer')->loadByCustomerRule($customerId, $rule->getId())) {
                    $couponUsage = new Varien_Object();
                    Mage::getResourceModel('salesrule/coupon_usage')->loadByCustomerCoupon($couponUsage, $customerId, $coupon->getId());

                    if ($couponUsage->getTimesUsed() > 0) {
                        /* I can't find any #@$!@$ interface to do anything but increment a coupon_usage record */
                        $resource = Mage::getSingleton('core/resource');
                        $writeConnection = $resource->getConnection('core_write');
                        $tableName = $resource->getTableName('salesrule_coupon_usage');

                        $query = "UPDATE {$tableName} SET times_used = times_used-1 "
                            .  "WHERE coupon_id = {$coupon->getId()} AND customer_id = {$customerId} AND times_used > 0";

                        $writeConnection->query($query);
                    }

                    if ($customerCoupon->getTimesUsed() > 0) {
                        $customerCoupon->setTimesUsed($customerCoupon->getTimesUsed()-1);
                        $customerCoupon->save();
                    }
                }
            }
        }
    }
}
4

1 回答 1

1

我相信这是一个从 1.4 到 1.6 左右的老错误。但是，无论您是否有旧版本，如果您了解 Magento 的方法，都可以很容易地解决这个问题。

问题是，当他们单击付款按钮时，您的代码正在更新 salesrule_coupon_usage 表。这根本不是你想要的。您希望将其包装到付款交易中。我不知道这个错误是否发生是因为你有自定义代码或使用的是旧版本的 Magento，但我会告诉你我将如何解决这个问题。然后我会给你一个类似于你建议的修复：

Magento 已经有一个称为“事务”的抽象。事务用于将需要全部成功或全部失败的对象放在一起，没有半途而废。Magento 将通过交易并尝试保存您放置在其中的每个对象。如果其中任何一个失败（例如，付款未通过），则所有已保存的事件都将“回滚”。

幸运的是，Magento 已经创建了一个交易对象来处理需要在成功支付时一起更新的各种事物的支付。您可以利用该交易并使用它来正确更新优惠券使用情况。

这是您需要做的事情的 10,000 英尺视图。

  • 过早地找出更新 salesrule_coupon_usage 表的任何东西并将其杀死。我们将添加我们自己的事务安全版本，因此我们不希望它被保存在其他任何地方。最简单的方法是找出连接到该表的模型并搜索该模型的创建。对于 1.7 和 1.8，这是规则/客户模型。
  • 创建一个观察者来捕捉支付交易的开始。在大多数现代版本的 magento 中，此事件称为 sales_order_payment_place_start 并且可以在 app/code/core/Mage/Sales/Model/Order/Payment.php 中见证
  • 将订单拉出活动，将优惠券代码拉出活动。
  • 拉出您要更新的实际模型。看起来有人在您的代码中找不到它，但应该有一些模型使用 salesrule_coupon_usage 表隐藏在某处。在 .xml 文件中搜索“salesrule_coupon_usage”并查看使用该表的模型。同样，对我来说，在 1.7 上，这就是规则/客户模型。
  • 加载该模型，让客户更改与您的优惠券代码相关的值，以表明客户已使用优惠券，但尚未保存。
  • 从事件中取出交易并使用 addObject 方法注册您更新的优惠券对象。

你完成了。事务将自动尝试保存已添加到其中的所有对象。如果任何部分失败（包括支付失败），它将回滚整个过程，您的优惠券将不会被使用。耶！

现在，在我看来，以上是解决问题的最佳方法，但如果您由于某种原因遇到问题，这里是基于捕获不成功的付款然后使用您的代码的替代方法。

同样，这是 10,000 英尺的视图：

  • 创建一个观察者来捕捉失败的支付事件。在大多数版本中，您想要的事件是 sales_order_payment_cancel。
  • 运行你拥有的代码......它应该这样做。但要澄清其他人：
    • 将订单从活动中拉出，从中拉出优惠券代码和客户 ID。
    • 更新 customer、rule 和 salesrule_coupon_usage 表。（虽然确实应该有一个模型，但我相信你能找到它）

现在，当销售失败时，您可以手动返回并展开所有内容。这不像我的第一个解决方案那样干净，但根据您对 Magento 的熟悉程度，可能对您来说更容易。

我相当确定新的、干净的 Magento 版本没有这个问题，所以让我提供一个相当明显的建议作为第三种解决方案。

  • 更新 Magento
  • 如果 Magento 是最新的，请测试禁用任何自定义模块，因为某些东西破坏了它。我注意到 Amasty 优惠券插件特别有问题。
  • 如果您对 Magento 核心进行了自定义更改...祝您好运。

祝你好运！

于 2014-01-19T07:51:12.350 回答