Better handle unspecified amounts

OrchardCMS · Apr 4, 2022 · e9dba99 · e9dba99
1 parent 1e230ae
commit e9dba99
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/MoneyDataType/Amount.cs b/MoneyDataType/Amount.cs
@@ -15,6 +15,11 @@ namespace Money
     [DebuggerDisplay("{DebuggerDisplay,nq}")]
     public struct Amount : IEquatable<Amount>, IComparable<Amount>
     {
+        public Amount() {
+            Value = 0;
+            Currency = Money.Currency.UnspecifiedCurrency;
+        }
+
         public Amount(decimal value, RegionInfo region)
         {
             if (region == null)

diff --git a/OrchardCore.Commerce/Services/SimplePriceStrategy.cs b/OrchardCore.Commerce/Services/SimplePriceStrategy.cs
@@ -18,7 +18,7 @@ public class SimplePriceStrategy : IPriceSelectionStrategy
         public Amount SelectPrice(IEnumerable<PrioritizedPrice> prices)
             => prices is null
                 || !prices.Any()
-                    ? new Amount()
+                    ? new Amount(0, Currency.UnspecifiedCurrency)
                     : prices
                         .Where(pp => pp.Priority == prices.Max(pp => pp.Priority))
                         .Min(pp => pp.Price);