Hi Kenni,

The code on it's own seems correct. Where is this code running though? in a controller? or event handler?

Generally if something doesn't persist my first port of call would be to check all unit's of work have uow.Complete() called on them as if you have any event handlers that don't do this, it can prevent a whole pipeline from persisting.

Hi Matt,

It's running in a SurfaceController which I'm using to create return orders.

I'm have a few unit's of work running for: (Had to split them else they would not save either)

• Assigning order to customer
• Add products
• Set shipping method
• Set payment method and authorize it
• Update order properties
• Set status

They all have uow.Complete(); called on them the order is created correctly except for the status still being new.

They are all within the same try catch, but are running in sequence

            using (var uow = _uniteOfWorkProvider.Create())
            {
                var order = _sessionManager.GetOrCreateCurrentOrder(store.Id).AsWritable(uow);

                var paymentMethods = VendrApi.Instance.GetPaymentMethodsAllowedIn(order.PaymentInfo.CountryId.Value);
                var zeroValuePaymentMethod = paymentMethods.FirstOrDefault(x => x.Alias == VendrCheckoutConstants.PaymentMethods.Aliases.ZeroValue);
                order.SetPaymentMethod(zeroValuePaymentMethod);

                _orderService.SaveOrder(order);

                uow.Complete();
            }

            using (var uow = _uniteOfWorkProvider.Create())
            {
                var order = _sessionManager.GetOrCreateCurrentOrder(store.Id).AsWritable(uow);

                var orderNumberGenerator = new ReturnOrderNumberGenerator(_storeService);
                order.InitializeTransaction(orderNumberGenerator);
                order.Finalize(order.TotalPrice, Guid.NewGuid().ToString("N"), PaymentStatus.Authorized);
                _orderService.SaveOrder(order);

                uow.Complete();
            }

            using (var uow = _uniteOfWorkProvider.Create())
            {
                var originOrder = _orderService.GetOrder(postModel.Id).AsWritable(uow);
                var productDetailsJson = UpdateProductDetails(originOrder.Properties["productDetailsList"], postModel.OrderLines);
                originOrder.SetProperty("productDetailsList", productDetailsJson);

                _orderService.SaveOrder(originOrder);
                uow.Complete();
            }

            using (var uow = _uniteOfWorkProvider.Create())
            {
                var order = _sessionManager.GetOrCreateCurrentOrder(store.Id).AsWritable(uow);
                var orderStatus = _orderStatusService.GetOrderStatus(store.Id, "newReturnOrder");
                order.SetOrderStatus(orderStatus);

                _orderService.SaveOrder(order);
                uow.Complete();
            }

All other snippets are saving the data correct, only the status that is being ignored.

Thanks Matt, using GetCurrentFinalizedOrder instead did the trick. Got tunnelvision and forgot about that method...

I had all of the code in the same uow in the beginning, but then most of it did not get saved, so split them out and it worked. Will test some more on using one uow.

is working on a reply...