Vendure电商平台订单合并策略中的产品线删除问题分析

问题背景

在Vendure电商平台中，当用户以访客身份添加商品到购物车后登录账户时，系统会执行订单合并操作。平台提供了多种合并策略，其中UseGuestStrategy策略的设计初衷是保留访客订单中的商品，而忽略已登录用户购物车中的商品。

然而，在实际使用中发现一个严重问题：当同一个商品同时存在于访客订单和用户订单中时，该商品会从最终合并后的订单中完全消失，导致用户购物车被清空。

问题重现与影响

通过以下步骤可以稳定重现该问题：

1. 用户登录后创建订单并添加商品A
2. 用户退出登录
3. 以访客身份再次添加相同的商品A
4. 用户重新登录账户

此时观察到的结果是购物车变为空状态，而不是预期的保留访客订单中的商品A。

这个问题直接影响用户体验，可能导致用户重复添加商品或直接放弃购买，对电商平台的转化率产生负面影响。

技术原因分析

深入分析Vendure核心代码中的订单合并逻辑，发现问题出在order.service.ts文件中的合并执行顺序：

1. 策略本身返回的合并结果(mergedLines)是正确的
2. 但在后续处理中，系统先执行了商品线的插入操作
3. 然后执行了商品线的删除操作
4. 由于相同的商品线被同时标记为插入和删除，最终导致商品被移除

这种执行顺序导致了逻辑冲突，使得应该保留的商品线被意外删除。

解决方案

解决这个问题的关键在于调整订单合并操作的执行顺序：

1. 首先执行所有需要删除的商品线操作
2. 然后执行需要修改的商品线操作
3. 最后执行需要新增的商品线操作

这种顺序确保了：

• 需要保留的商品不会被后续的删除操作影响
• 需要删除的商品被优先处理
• 新增的商品不会被误删

修复验证

为了确保修复的有效性，我们添加了专门的端到端测试用例：

it('UseGuestStrategy with conflicting lines', async () => {
    const result = await testMerge({
        strategy: new UseGuestStrategy(),
        customerEmailAddress: customers[3].emailAddress,
        existingOrderLines: [
            { productVariantId: 'T_5', quantity: 1 },
            { productVariantId: 'T_3', quantity: 1 },
        ],
        guestOrderLines: [{ productVariantId: 'T_5', quantity: 3 }],
    });

    expect(
        result.lines.sort(sortById).map(line => ({
            productVariantId: line.productVariant.id,
            quantity: line.quantity,
        })),
    ).toEqual([{ productVariantId: 'T_5', quantity: 3 }]);
})

这个测试明确验证了当同一商品存在于访客订单和用户订单时，系统能够正确保留访客订单中的商品及其数量。

总结

订单合并是电商平台中一个关键但容易出错的场景，特别是在处理用户身份转换(访客→登录用户)时。Vendure平台通过策略模式提供了灵活的合并方案，但在实现细节上需要注意操作顺序对最终结果的影响。

本次修复不仅解决了具体问题，还通过添加测试用例确保了类似问题不会再次出现，提高了系统的稳定性和可靠性。对于电商开发者来说，这是一个很好的案例，提醒我们在处理数据合并操作时要特别注意操作顺序可能带来的副作用。