Creating new prices with IPriceDetailService removes decimals

Morten Kristensen

Vote:

I feel like i'm missing something obvious here. I'm trying to insert a price of 8.12 DKK, but when saving to the database the value changes to 8.00 DKK.

I then create the price using:

var variant = (from method overload - not null)
var currentMarket = ServiceLocator.Current.GetInstance();

var priceValue = new PriceDetailValue {
					CatalogKey = new CatalogKey(AppContext.Current.ApplicationId, variant.Code),
					CustomerPricing = CustomerPricing.AllCustomers,
					MarketId = currentMarket.GetCurrentMarket().MarketId,
					MinQuantity = 1,
					ValidFrom = DateTime.Now,
					ValidUntil = DateTime.Now.AddDays(10),
					UnitPrice = new Money(new decimal(8.12), "DKK")
				};

ServiceLocator.Current.GetInstance().Save(priceValue);

The market has DKK and EUR as active currencies.

Any help would be greatly appreciated.

Best Regards
Morten Kristensen

#114548

Dec 15, 2014 12:35

Morten Kristensen

Vote:

Oh, and for the record: I'm using EPiServer Commerce 8.5 (build 0)

#114549

Dec 15, 2014 12:37

Quan Mai

Vote:

Hi,

Did you tried with

UnitPrice = new Money(8.12m, "DKK") instead?

#114578

Edited, Dec 16, 2014 1:24

Morten Kristensen

Vote:

No luck - same result with shorthand decimal :o(

The code above is an extract of a Unit Test of the method i use to (dynamically) insert prices. The method containing the code receives a decimal UnitPrice (and Currency string).

#114585

Dec 16, 2014 9:01

Quan Mai

Vote:

Hi,

Can you check if the price stored in database was correct? It should be in PriceDetail table.

The price is stored as Money, which should keep the precision of ten-thousandth. As we have intensive tests in this area then I'm pretty sure that we did it right.

Regards.

#114586

Dec 16, 2014 9:18

Morten Kristensen

Vote:

The value is inserted as 8.00 DKK in the PriceDetail table.

I'm sure you did it right, but in my case it's not behaving as expected (and yes, Money data type allows 4 decimals, the EPiServer UI/API however supports only 2 decimals in practice - but thats beside the point).
I am however having a hard time debugging from the point of the .Save() of IPriceDetailService.

#114587

Dec 16, 2014 9:28

Quan Mai

Vote:

Hi,

What if you use the Catalog UI to edit prices? Are them saved properly?

#114588

Dec 16, 2014 9:40

Morten Kristensen

Vote:

Yes, the UI saves the prices as expected (entering 8,12 - note the comma) saves the price as 8.12 DKK.

Could it be related to the different decimal mark i'm wondering (even though the actual decimal inserted through the API is 8.12)?

#114589

Dec 16, 2014 9:47

Quan Mai

Vote:

Might be - logically it's weird if .NET simply parse 8.12 as 8.0 if the current culture uses , as the decimal parameter - but it won't hurt to test with 8.12, right? ;)

Regards.

#114591

Dec 16, 2014 9:56

Morten Kristensen

Vote:

Well, the EPiServer Catalog UI changes the price to 8.00 if i enter 8.12 and press enter :o)

I have no clue what to try next though :o/

#114592

Dec 16, 2014 10:03

Quan Mai

Vote:

It might sound stupid but make sure you don't have custom implementation of IPriceDetailService. The default implentation use ecf_PriceDetail_Save stored procedure, which is quite straighforward and no convert/truncating should happen.

If this issue is serious enough - I suggest to contact our support service for proper investigation.

Regards.

#114595

Dec 16, 2014 10:25

Morten Kristensen

Vote:

Nope, I'm not using a custom implementation (but considering doing so - would prefer not to though). The ecf_PriceDetail_Save procedure is used in this case also (tried renaming it to make sure it failed).

I will make some further attempts and contact support service if necessary. Thanks for the effort though Quan - greatly appreciated :o)

#114596

Dec 16, 2014 10:51

Morten Kristensen

Vote:

Oh and by the way, i can resolve the issue by (and this is a temporary workaround):

1. Save the price as described above.
2. Get the price i just created using IPriceDetailService.Get()
3. Invoke new PriceDetailValue(priceJustCreated)
4. Set the price again.
5. Save yet again.

Just in case anyone else needs a workaround.

I'm opening an issue in with support service :o)

#114600

Dec 16, 2014 11:02

Try our conversational search powered by Generative AI!

Creating new prices with IPriceDetailService removes decimals