Developer forum

Sounds awful complicated. Why not add the product to the users assortment in code...? Maybe with a price of 0 or an other indication that the product cannot be purchased if that is a need.

An alternative could be adding a property on an orderline to not remove it (Stock, price, assortment, other rules). That would be more generic for future scenarios.

BR Nicolai

Dear Nicolai,

Yes, it will be great to have a flag on a product that says, that it should not be removed when performing the check in RemoveNoneActiveProducts. But this should not be cached as it will vary from user to user.

Can that we added easily?

Best regards, Anders

Sorry for pushing here, but we need to send an estimate to a client for handling this scenario, and the solution is going to the test phase in the beginning of November

Hi Nicolai

I have tried to use this new feature to avoid products being removed from quotes when a product is not in the customer's assortment. But have not been able to make it work.

I listen to the event Ecommerce.Order.State.Changed and test whether the order has been changed to a quote and updates all order lines to DisableRemoveFromCartRules = true. But when I log out from the sales person and log in to the customer to approve the quote, products that are not in the customer's assortment is removed.

That property is stateless - so when you logoff and in as an other user, the order is reloaded and that information is gone.

Hey Nicolai

Thanks for the quick response, I will find another solution then. :)

Dear Nicolai,

Will it be possible to have a notificationsubscriber before "Notifications.Ecommerce.Cart.InitiallyLoaded" as we need to set the right values on DisableRemoveFromCartRules, as the products are removed before we can change these values?

See this section from the source code:

        Friend Function LoadCart(user As User, orderContext As OrderContext) As Order
                ...

                        'Check if all the products are still available
                        Services.Orders.RemoveNoneActiveProducts(cart, user, orderContext, Common.Context.StockLocation)

                        NotificationManager.Notify(Notifications.Ecommerce.Cart.InitiallyLoaded, New Notifications.Ecommerce.Cart.InitiallyLoadedArgs(cart))

Best regards,

Anders

Dear Nicolai,

Any update here as our client is waiting for a solution?

Dear Nicolai,

Sorry for spamming this thread, but we really need to find a solution here and I know that you are busy with the 9.10 release

Best regards, Anders

We have to take another approach.

I suggest what you see in the screenshot. An implementation is in the attached pirate 9.9 ecommerce.dll

And the config key for cut and paste: /Globalsettings/Ecom/Cart/DoNotDeleteProductsNotInAssortment

Works?

Dear Nicolai,

No, it is still not working as the products seems to be removed in the frontend then displaying the quote to the user. Please see the call stack here: https://www.screencast.com/t/NEBgEigjI8wy.

Best regards, Anders

Well - that is because the product service of course takes assortments into consideration.

I am affraid it is not easy to solve. Assortments are all over the place and the entire idea is that if you do not have that product in assortment, forget about buying it.

We probably have to go down many paths to override this and that would be risky business - so you have to find another way to do this...

Sorry, Nicolai

Dear Nicolai,

I have done some more debugging and what is missing, is to add false to the line here: Services.Products.GetProductById(ProductId, ProductVariantId, Order.LanguageId, False), so it does not use assortments when loading the products in OrderLine.vb. And I cannot see that is should be necessary to load products with assortments as invalid orderlines/products are removed other places in the code.

So can "False" be added here:

        Public Property Product As Product
            Get
                If Not _productsLoaded Then
                    Order.OrderLines.LoadProducts()
                    _productsLoaded = True
                End If

                Dim productFromDatabase As Product = Services.Products.GetProductById(ProductId, ProductVariantId, Order.LanguageId, False)
                If productFromDatabase Is Nothing Then
                    Return New Product()
                End If
                Return productFromDatabase
            End Get
            Set
                If Value Is Nothing Then
                    ProductId = Nothing
                    ProductVariantId = Nothing
                    _isDirty = True
                End If

                If Value IsNot Nothing AndAlso Not String.Equals(ProductId, Value.Id, StringComparison.Ordinal) AndAlso Not String.Equals(ProductVariantId, Value.VariantId, StringComparison.Ordinal) Then
                    ProductId = Value.Id
                    ProductVariantId = Value.VariantId
                    _isDirty = True
                End If
                CachedPrice = Nothing
            End Set
        End Property

Best regards, Anders

Dear Nicolai,

Will my suggestion be possible for you to change?

Best regards, Anders

Dear Nicolai,

Any news here as our client is awaiting a solution?

Best regards, Anders

Hi Anders,

Just giving you an update: When I try to do what you suggest, some of our other checks break down. Specifically, if you have a product in a restored cart, it wont respect the assortments of the user on restore after your suggested change. This is too breaking a behavior for us to move forward with it.

I will take a look at the places in our software where we use orderline.Product and review what changes need to be implemented to  get you your fix. It's 154 places, some of which are a bit tricky, though, so please be patient while I dig to see IF it's possible (I assume it will be possible, and you'll get a PBI# when I've confirmed that it can be fixed).

BR

Martin

Sounds great! Thank you, Martin,

I appreciate that you look into the issue!

Hi Anders,

I've created the feature Feature 771: Avoid cart logic regarding removal of products

It should be possible to merge this to the DW9 branch, so I expect it to come there as well. Testing this might take a bit of time, though, as it affects a lot of different things.

You'll get an update here once there is something to download.

BR

Martin

Excellent! Thanks! :-)

Dear Martin,

Can you give me an update here as our client is very much awaiting this feature :-)

Best regards, Anders

Hi Anders,

Sorry about that, it's actually been out since ecommerce 1.10.14. Our new setup still needs a few tweaks - fx. how we get back to a post on forum. :/

Let me know if it helps.

BR

Martin