Sort rule by name

saleor · Dec 21, 2023 · 61fc1cf · 61fc1cf
1 parent 1152040
commit 61fc1cf
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 3 deletions.
diff --git a/src/discounts/components/DiscountCreateForm/hooks/useRulesHandlers.ts b/src/discounts/components/DiscountCreateForm/hooks/useRulesHandlers.ts
@@ -1,6 +1,8 @@
 import { Rule } from "@dashboard/discounts/models";
 import { useState } from "react";
 
+import { ruleAlphabetically } from "../../DiscountDetailsForm/utils";
+
 export const useRulesHandlers = () => {
   const [rules, setRules] = useState<Rule[]>([]);
 
@@ -17,7 +19,7 @@ export const useRulesHandlers = () => {
         return rules;
       });
     } else {
-      setRules([...rules, ruleObj]);
+      setRules([...rules, ruleObj].sort(ruleAlphabetically));
     }
   };
 

diff --git a/src/discounts/components/DiscountDetailsForm/hooks/useRulesHandlers.ts b/src/discounts/components/DiscountDetailsForm/hooks/useRulesHandlers.ts
@@ -7,7 +7,7 @@ import {
 import { CommonError } from "@dashboard/utils/errors/common";
 import { useEffect, useState } from "react";
 
-import { getCurrentConditionsValuesLabels } from "../utils";
+import { getCurrentConditionsValuesLabels, ruleAlphabetically } from "../utils";
 
 interface UseRulesHandlersProps {
   data: PromotionDetailsFragment | undefined | null;
@@ -65,7 +65,7 @@ export const useRulesHandlers = ({
 
   const addNewRuleToArray = (rule: Rule) => {
     updateLabels(rule);
-    setRules(prevRules => [...prevRules, rule]);
+    setRules(prevRules => [...prevRules, rule].sort(ruleAlphabetically));
   };
 
   const removeRuleFromArray = (index: number) => {

diff --git a/src/discounts/components/DiscountDetailsForm/utils.ts b/src/discounts/components/DiscountDetailsForm/utils.ts
@@ -12,3 +12,13 @@ export const getCurrentConditionsValuesLabels = (rules: Rule[]) => {
       return acc;
     }, {} as Record<string, string>);
 };
+
+export const ruleAlphabetically = (a: Rule, b: Rule) => {
+  if (a.name < b.name) {
+    return -1;
+  }
+  if (a.name > b.name) {
+    return 1;
+  }
+  return 0;
+};