Bundles – Handle Location for Multi-Item Orders

[pavelbrm]

Summary

This PR updates order creation in the new endpoint (/v1/orders-new) with allowing NULL location on multi-item orders. It resolves #1968.

Main Changes

The main change is that the new endpoint for creating orders (which is not yet in use in Production) supports orders with multiple items that have different locations.

Secondary Changes

To enable that, I've done some refactoring:

• The Create method on the Order repository requires an order prototype object instead of all the details in the open.
• Updated the CreateOrder method on the legacy datastore:
  • It now accepts an object, a prototype of the order, instead of messing around with all the fields.
  • Transaction management must be handled by the caller.
• Updated the legacy CreateOrderFromRequest method on the service to support the changes above.
• Decoupled Stripe checkout session creation from Order. It must not be a method on the Order type.

Type of Change

• Product feature
• Bug fix
• Performance improvement
• Refactor
• Other

Tested Environments

• Development
• Staging
• Production

Before Requesting Review

• Does your code build cleanly without any errors or warnings?
• Have you used auto closing keywords?
• Have you added tests for new functionality?
• Have validated query efficiency for new database queries?
• Have documented new functionality in README or in comments?
• Have you squashed all intermediate commits?
• Is there a clear title that explains what the PR does?
• Have you used intuitive function, variable and other naming?
• Have you requested security/privacy review if needed
• Have you performed a self review of this PR?

---