Fixed GetPrices Api from trying to order a null value

TheKye · Sep 8, 2024 · 1728d47 · 1728d47
1 parent 3c31b58
commit 1728d47
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/Eco.EM.Framework/API/Requests.cs b/Eco.EM.Framework/API/Requests.cs
@@ -42,7 +42,7 @@ public string GetPrices([FromQuery]bool includeOutOfStock = false)
             string noResult = "No Items Found";
             if (BasePlugin.Obj.Config.EnableWebAPI)
             {
-                var result = ShopUtils.GetAllItems(includeOutOfStock).OrderBy(o => o.StoreName);
+                var result = ShopUtils.GetAllItems(includeOutOfStock);
                 if (result is null)
                     return noResult;
                 else

diff --git a/Eco.EM.Framework/Utils/ShopUtils.cs b/Eco.EM.Framework/Utils/ShopUtils.cs
@@ -197,7 +197,10 @@ public static List<OfferedItem> GetAllItems(bool includeOutOfStock = false)
                     }
                 }
                 if (Items.Count > 0)
+                {
+                    Items.OrderBy(o => o.StoreName);
                     return Items;
+                }
                 else
                     return null;
             }