Hi Matt

Yep had a look through the Docs and came to the same conclusion. Seems simple enough, I want to check the document type of the product and then i can do the calculation dependant on the width and height.

However running into a problem trying to access IPublishedContent to check the DocumentType of the Product, Im passing in the orderline.ProductIdentifier but my code crashes.

Here's what I have so far

[SuppressDependency("TeaCommerce.Api.PriceCalculators.IOrderLineCalculator", "TeaCommerce.Api")]
            public class CustomOrderLineCalculator : OrderLineCalculator
            {

                public CustomOrderLineCalculator(IVatGroupService vatGroupService, IProductInformationExtractor productInformationExtractor) : base(vatGroupService, productInformationExtractor)
                {
                }

                public override Price CalculatePrice(OrderLine orderLine, Currency currency, Order order)
                {
                    if (!string.IsNullOrEmpty(orderLine.ProductIdentifier))
                    {

                        var frameType = new Umbraco.Web.UmbracoHelper();
                        var frame = frameType.TypedContent(orderLine.ProductIdentifier.ToString());


                    }

                    return new Price(100M, orderLine.VatRate, currency);
                }

            }

When you say your code crashes, what exactly do you mean? Do you get an error? If so, what's the stack trace?

Also, you shouldn't neet to go via the Umbraco helper to get the frame node, you could go via the UmbracoContext content cache instead

UmbracoContext.Current.ContentCache.GetById(orderLine.ProductIdentifier.ToString());

Sorry, think it's my setup, keep loosing the Autofac.dll from the build for some reason.

Then when adding the product to the basket I get the blank screen with the formPost.aspx. I've usually had that when there have been more than 20 items in the orders, clearing it out fixes it but it showed up again and it was down to trying to access IPublishedContent

Managed to sort it now, and yep will use the UmbracoContext to get the node details.

Thanks for your help Matt, much appreciated!

Aaaah, awesome.

Glad you got it all working 👍

Hi

I've got a little further today on this, just wondering from a point of returning a new price I need to do anything else?

I'm hitting an error - System.ArgumentException: The product doesn't have a store id associated with it - remember to add the Tea Commerce store picker to your Umbraco content tree

My code so far is: (needs refactoring yet but you get the idea)

        public override Price CalculatePrice(OrderLine orderLine, Currency currency, Order order)
        {
            decimal finalPrice = 0;
            if (!string.IsNullOrEmpty(orderLine.ProductIdentifier))
            {
                var frame = UmbracoContext.Current.ContentCache.GetById(Convert.ToInt32(orderLine.ProductIdentifier));

                if (frame != null && frame.DocumentTypeAlias == "attributeFrame")
                {
                    var frameWidth = Convert.ToInt32(orderLine.Properties["productWidth"]);
                    var frameHeight = Convert.ToInt32(orderLine.Properties["productHeight"]);
                    var framePrice = Convert.ToDecimal(frame.GetPropertyValue("priceGBP"));

                    var frameWidthDouble = frameWidth * 2;
                    var frameHeightDouble = frameHeight * 2;
                    var heightPlusWidth = frameWidthDouble + frameHeightDouble;

                    finalPrice = Convert.ToDecimal(heightPlusWidth * framePrice);
                }
                else
                {
                    //normal product
                    var collectionPrice = orderLine.OriginalUnitPrices.FirstOrDefault();
                    finalPrice = collectionPrice.Value;
                }
            }


            return new Price(finalPrice, orderLine.VatRate, currency);
        }

I'm just getting the blank page again and the URL is - /base/TC/FormPost.aspx

Got this working well now, thank's for your help on it Matt. I can now calc the price dynamically

That's awesome. Glad you got it working 👍