@darkpos/pricing 1.0.24 → 1.0.26

package/__TEST__/order/order.test.js

@@ -10,6 +10,7 @@ const orderModifiersPartiallyPaid = require('../mocks/partially-paid/order-modif
 const inputItemsPartiallyPaidOrder = require('../mocks/partially-paid/input-items.json');
 
 const orderCredit = require('../mocks/order-credit.json');
+const { Types } = require('../../lib/constants/Modifier/Compute');
 
 const session = {
   store: mockStores[0],
@@ -124,7 +125,7 @@ describe('Order actions', () => {
     expect(newOrder).toHaveProperty('subTotals', {
       discount: -2,
     });
-    expect(newOrder.items[0]).toHaveProperty('total', 58.29);
+    expect(newOrder.items[0]).toHaveProperty('total', 58.285714285714285);
     expect(newOrder.items[0]).toHaveProperty('subTotals', {
       discount: -1.7142857142857142,
       _included: 0,
@@ -134,7 +135,7 @@ describe('Order actions', () => {
       _simple: 60,
       _actual: 60,
     });
-    expect(newOrder.items[1]).toHaveProperty('total', 9.71);
+    expect(newOrder.items[1]).toHaveProperty('total', 9.714285714285714);
     expect(newOrder.items[1]).toHaveProperty('subTotals', {
       discount: -0.2857142857142857,
       _included: 0,
@@ -173,7 +174,7 @@ describe('Order actions', () => {
     expect(newOrder).toHaveProperty('subTotals', {
       discount: -2.06,
     });
-    expect(newOrder.items[0]).toHaveProperty('total', 14.39);
+    expect(newOrder.items[0]).toHaveProperty('total', 14.391);
     expect(newOrder.items[0]).toHaveProperty('subTotals', {
       discount: -1.599,
       _included: 0,
@@ -183,7 +184,7 @@ describe('Order actions', () => {
       _simple: 15.99,
       _actual: 15.99,
     });
-    expect(newOrder.items[1]).toHaveProperty('total', 4.19);
+    expect(newOrder.items[1]).toHaveProperty('total', 4.185);
     expect(newOrder.items[1]).toHaveProperty('subTotals', {
       discount: -0.465,
       _included: 0,
@@ -682,9 +683,9 @@ describe('Order actions', () => {
       discount: -16.41,
       tax: 7.21,
     });
-    expect(newOrder.items[0]).toHaveProperty('total', 80.6);
-    expect(newOrder.items[1]).toHaveProperty('total', 13.69);
-    expect(newOrder.items[2]).toHaveProperty('total', 5.91);
+    expect(newOrder.items[0]).toHaveProperty('total', 80.597);
+    expect(newOrder.items[1]).toHaveProperty('total', 13.69233125);
+    expect(newOrder.items[2]).toHaveProperty('total', 5.90739375);
     expect(newOrder.items[0].modifiers[0].compute).toHaveProperty(
       'amount',
       13.2
@@ -700,7 +701,7 @@ describe('Order actions', () => {
     );
     expect(newOrder.items[1].modifiers[1].compute).toHaveProperty(
       'amount',
-      0.985025
+      0.98483125
     );
 
     expect(newOrder.items[2].modifiers[0].compute).toHaveProperty(
@@ -709,7 +710,7 @@ describe('Order actions', () => {
     );
     expect(newOrder.items[2].modifiers[1].compute).toHaveProperty(
       'amount',
-      0.4247
+      0.42489375
     );
   });
 
@@ -774,9 +775,9 @@ describe('Order actions', () => {
       discount: -16.41,
       tax: 7.21,
     });
-    expect(newOrder.items[0]).toHaveProperty('total', 80.6);
-    expect(newOrder.items[1]).toHaveProperty('total', 13.69);
-    expect(newOrder.items[2]).toHaveProperty('total', 5.91);
+    expect(newOrder.items[0]).toHaveProperty('total', 80.597);
+    expect(newOrder.items[1]).toHaveProperty('total', 13.69233125);
+    expect(newOrder.items[2]).toHaveProperty('total', 5.90739375);
     expect(newOrder.items[0].modifiers[0].compute).toHaveProperty(
       'amount',
       13.2
@@ -866,9 +867,9 @@ describe('Order actions', () => {
       discount: -16.41,
       tax: 7.21,
     });
-    expect(newOrder.items[0]).toHaveProperty('total', 80.6);
-    expect(newOrder.items[1]).toHaveProperty('total', 13.69);
-    expect(newOrder.items[2]).toHaveProperty('total', 5.91);
+    expect(newOrder.items[0]).toHaveProperty('total', 80.597);
+    expect(newOrder.items[1]).toHaveProperty('total', 13.69233125);
+    expect(newOrder.items[2]).toHaveProperty('total', 5.90739375);
     expect(newOrder.items[0].modifiers[0]._computed).toHaveProperty(
       'amount',
       -13.2
@@ -958,9 +959,9 @@ describe('Order actions', () => {
       discount: -16.41,
       tax: 7.21,
     });
-    expect(newOrder.items[0]).toHaveProperty('total', 80.6);
-    expect(newOrder.items[1]).toHaveProperty('total', 13.69);
-    expect(newOrder.items[2]).toHaveProperty('total', 5.91);
+    expect(newOrder.items[0]).toHaveProperty('total', 80.597);
+    expect(newOrder.items[1]).toHaveProperty('total', 13.69233125);
+    expect(newOrder.items[2]).toHaveProperty('total', 5.90739375);
     expect(newOrder.items[0].modifiers[0]._computed).toHaveProperty(
       'amount',
       -13.2
@@ -976,7 +977,7 @@ describe('Order actions', () => {
     );
     expect(newOrder.items[1].modifiers[1]._computed).toHaveProperty(
       'amount',
-      0.985025
+      0.98483125
     );
 
     expect(newOrder.items[2].modifiers[0]._computed).toHaveProperty(
@@ -985,7 +986,7 @@ describe('Order actions', () => {
     );
     expect(newOrder.items[2].modifiers[1]._computed).toHaveProperty(
       'amount',
-      0.4247
+      0.42489375
     );
   });
 
@@ -1139,16 +1140,16 @@ describe('Order actions', () => {
       fixed: -1,
       tax: 0.16,
     });
-    expect(newOrder.items[0]).toHaveProperty('total', 0.72);
-    expect(newOrder.items[1]).toHaveProperty('total', 0.72);
-    expect(newOrder.items[2]).toHaveProperty('total', 0.72);
+    expect(newOrder.items[0]).toHaveProperty('total', 0.7183333333333333);
+    expect(newOrder.items[1]).toHaveProperty('total', 0.7183333333333334);
+    expect(newOrder.items[2]).toHaveProperty('total', 0.7183333333333334);
     expect(newOrder.items[0].modifiers[0]._computed).toHaveProperty(
       'amount',
       -0.3333333333333334
     );
     expect(newOrder.items[0].modifiers[1]._computed).toHaveProperty(
       'amount',
-      0.051925
+      0.05166666666666666
     );
 
     expect(newOrder.items[1].modifiers[0]._computed).toHaveProperty(
@@ -1157,7 +1158,7 @@ describe('Order actions', () => {
     );
     expect(newOrder.items[1].modifiers[1]._computed).toHaveProperty(
       'amount',
-      0.051925
+      0.05166666666666667
     );
 
     expect(newOrder.items[2].modifiers[0]._computed).toHaveProperty(
@@ -1166,7 +1167,7 @@ describe('Order actions', () => {
     );
     expect(newOrder.items[2].modifiers[1]._computed).toHaveProperty(
       'amount',
-      0.051925
+      0.05166666666666667
     );
   });
 
@@ -1261,19 +1262,19 @@ describe('Order actions', () => {
 
     const [item1, item2, item3] = newOrder.items;
     expect(newOrder).toHaveProperty('total', 12.5);
-    expect(item1).toHaveProperty('total', 3.65);
+    expect(item1).toHaveProperty('total', 3.653846153846154);
     expect(item1.modifiers[1].compute).toHaveProperty(
       'amount',
       5.846153846153846
     );
 
-    expect(item2).toHaveProperty('total', 5.38);
+    expect(item2).toHaveProperty('total', 5.3846153846153854615384615385);
     expect(item2.modifiers[1].compute).toHaveProperty(
       'amount',
       8.615384615384615
     );
 
-    expect(item3).toHaveProperty('total', 3.46);
+    expect(item3).toHaveProperty('total', 3.461538461538462);
     expect(item3.modifiers[1].compute).toHaveProperty(
       'amount',
       5.538461538461538
@@ -1611,10 +1612,10 @@ describe('Order actions', () => {
       discount: -10,
       fee: 2,
     });
-    expect(newOrder.items[0]).toHaveProperty('total', 8.07);
-    expect(newOrder.items[1]).toHaveProperty('total', 7.71);
-    expect(newOrder.items[2]).toHaveProperty('total', 11.36);
-    expect(newOrder.items[3]).toHaveProperty('total', 7.3);
+    expect(newOrder.items[0]).toHaveProperty('total', 8.074852767962309);
+    expect(newOrder.items[1]).toHaveProperty('total', 7.709658421672556);
+    expect(newOrder.items[2]).toHaveProperty('total', 11.361601884570083);
+    expect(newOrder.items[3]).toHaveProperty('total', 7.303886925795053);
   });
   test('Get calculated PARTIALLY PAID Order, multiple items and paymentModifiers', () => {
     const paymentModifiers = orderModifiersPartiallyPaid.map(orderMod =>
@@ -1639,10 +1640,10 @@ describe('Order actions', () => {
       discount: -10,
       fee: 4,
     });
-    expect(newOrder.items[0]).toHaveProperty('total', 8.08);
-    expect(newOrder.items[1]).toHaveProperty('total', 7.7);
-    expect(newOrder.items[2]).toHaveProperty('total', 11.36);
-    expect(newOrder.items[3]).toHaveProperty('total', 9.3);
+    expect(newOrder.items[0]).toHaveProperty('total', 8.083543307086615);
+    expect(newOrder.items[1]).toHaveProperty('total', 7.703779527559055);
+    expect(newOrder.items[2]).toHaveProperty('total', 11.36191226096738);
+    expect(newOrder.items[3]).toHaveProperty('total', 9.300764904386952);
   });
 
   test('CU-8678p4ky3. Get Balance to Pay from OrderItems', () => {
@@ -1917,144 +1918,326 @@ describe('Order actions', () => {
 
     expect(balanceToPay).toBe(102.79);
   });
-});
 
-test('CU-8678p4ky3. Get Calculated order. Multiple Items. 2 Modifiers. 1 Item tax. 1 Orde discount', () => {
-  const taxMod = {
-    _id: 2,
-    name: `7.75% tax`,
-    type: 'tax',
-    compute: {
-      amount: 7.75,
-      type: 'percentage',
-      action: 'add',
-    },
-    properties: {
-      sort: 2,
-    },
-  };
-
-  const discountMod = {
-    _id: 1,
-    compute: {
-      amount: 15,
+  test('CU-8678p4ky3. Get Calculated order. Multiple Items. 2 Modifiers. 1 Item tax. 1 Orde discount', () => {
+    const taxMod = {
+      _id: 2,
+      name: `7.75% tax`,
+      type: 'tax',
+      compute: {
+        amount: 7.75,
+        type: 'percentage',
+        action: 'add',
+      },
+      properties: {
+        sort: 2,
+      },
+    };
+
+    const discountMod = {
+      _id: 1,
+      compute: {
+        amount: 15,
+        type: 'percentage',
+        action: 'subtract',
+      },
+      name: `15%`,
       type: 'percentage',
-      action: 'subtract',
-    },
-    name: `15%`,
-    type: 'percentage',
-    properties: {
-      sort: 1,
-    },
-  };
-
-  const item1 = {
-    _id: 1,
-    price: 35.95,
-    quantity: 2,
-    modifiers: [taxMod],
-  };
-  const item2 = {
-    _id: 2,
-    price: 28.95,
-    quantity: 2,
-    modifiers: [taxMod],
-  };
-  const item3 = {
-    _id: 3,
-    price: 17.95,
-    quantity: 4,
-    modifiers: [taxMod],
-  };
-  const item4 = {
-    _id: 4,
-    price: 158,
-    quantity: 1,
-    modifiers: [taxMod],
-  };
-  const item5 = {
-    _id: 5,
-    price: 148,
-    quantity: 2,
-    modifiers: [taxMod],
-  };
-  const item6 = {
-    _id: 6,
-    price: 120,
-    quantity: 1,
-    modifiers: [taxMod],
-  };
-
-  const order = {
-    items: [item1, item2, item3, item4, item5, item6],
-    modifiers: [discountMod],
-  };
-
-  const newOrder = pricingService.order.calculate(order);
-
-  expect(newOrder).toHaveProperty('total', 710.35);
-  expect(newOrder).toHaveProperty('subTotal', 775.6);
-  expect(newOrder).toHaveProperty('subTotals', {
-    percentage: -116.34,
-    tax: 51.09,
+      properties: {
+        sort: 1,
+      },
+    };
+
+    const item1 = {
+      _id: 1,
+      price: 35.95,
+      quantity: 2,
+      modifiers: [taxMod],
+    };
+    const item2 = {
+      _id: 2,
+      price: 28.95,
+      quantity: 2,
+      modifiers: [taxMod],
+    };
+    const item3 = {
+      _id: 3,
+      price: 17.95,
+      quantity: 4,
+      modifiers: [taxMod],
+    };
+    const item4 = {
+      _id: 4,
+      price: 158,
+      quantity: 1,
+      modifiers: [taxMod],
+    };
+    const item5 = {
+      _id: 5,
+      price: 148,
+      quantity: 2,
+      modifiers: [taxMod],
+    };
+    const item6 = {
+      _id: 6,
+      price: 120,
+      quantity: 1,
+      modifiers: [taxMod],
+    };
+
+    const order = {
+      items: [item1, item2, item3, item4, item5, item6],
+      modifiers: [discountMod],
+    };
+
+    const newOrder = pricingService.order.calculate(order);
+
+    expect(newOrder).toHaveProperty('total', 710.35);
+    expect(newOrder).toHaveProperty('subTotal', 775.6);
+    expect(newOrder).toHaveProperty('subTotals', {
+      percentage: -116.34,
+      tax: 51.09,
+    });
+    expect(newOrder.items[0]).toHaveProperty('total', 65.8514125);
+    expect(newOrder.items[1]).toHaveProperty('total', 53.0291625);
+    expect(newOrder.items[2]).toHaveProperty('total', 65.759825);
+    expect(newOrder.items[3]).toHaveProperty('total', 144.70825);
+    expect(newOrder.items[4]).toHaveProperty('total', 271.099);
+    expect(newOrder.items[5]).toHaveProperty('total', 109.905);
+    expect(newOrder.items[0].modifiers[0]._computed).toHaveProperty(
+      'amount',
+      -10.785
+    );
+    expect(newOrder.items[0].modifiers[1]._computed).toHaveProperty(
+      'amount',
+      4.7364125
+    );
+
+    expect(newOrder.items[1].modifiers[0]._computed).toHaveProperty(
+      'amount',
+      -8.685
+    );
+    expect(newOrder.items[1].modifiers[1]._computed).toHaveProperty(
+      'amount',
+      3.8141625
+    );
+
+    expect(newOrder.items[2].modifiers[0]._computed).toHaveProperty(
+      'amount',
+      -10.77
+    );
+    expect(newOrder.items[2].modifiers[1]._computed).toHaveProperty(
+      'amount',
+      4.729825
+    );
+
+    expect(newOrder.items[3].modifiers[0]._computed).toHaveProperty(
+      'amount',
+      -23.7
+    );
+    expect(newOrder.items[3].modifiers[1]._computed).toHaveProperty(
+      'amount',
+      10.40825
+    );
+
+    expect(newOrder.items[4].modifiers[0]._computed).toHaveProperty(
+      'amount',
+      -44.4
+    );
+    expect(newOrder.items[4].modifiers[1]._computed).toHaveProperty(
+      'amount',
+      19.499
+    );
+
+    expect(newOrder.items[5].modifiers[0]._computed).toHaveProperty(
+      'amount',
+      -18
+    );
+    expect(newOrder.items[5].modifiers[1]._computed).toHaveProperty(
+      'amount',
+      7.905
+    );
+  });
+
+  test('CU-8678tdrvq. Get Calculated order. Multiple Items. 1 Order tax', () => {
+    const taxMod = {
+      _id: 2,
+      name: `7.75% tax`,
+      type: 'tax',
+      compute: {
+        amount: 7.75,
+        type: 'percentage',
+        action: 'add',
+      },
+      properties: {
+        sort: 2,
+      },
+    };
+
+    const item1 = {
+      _id: 1,
+      price: 38,
+      quantity: 1,
+      modifiers: [],
+    };
+    const item2 = {
+      _id: 2,
+      price: 42,
+      quantity: 1,
+      modifiers: [],
+    };
+    const item3 = {
+      _id: 3,
+      price: 48,
+      quantity: 1,
+      modifiers: [],
+    };
+
+    const order = {
+      items: [item1, item2, item3],
+      modifiers: [taxMod],
+    };
+
+    const newOrder = pricingService.order.calculate(order);
+
+    expect(newOrder).toHaveProperty('total', 137.92);
+    expect(newOrder).toHaveProperty('subTotal', 128);
+    expect(newOrder).toHaveProperty('subTotals', {
+      tax: 9.92,
+    });
+    expect(newOrder.items[0]).toHaveProperty('total', 40.945);
+    expect(newOrder.items[1]).toHaveProperty('total', 45.255);
+    expect(newOrder.items[2]).toHaveProperty('total', 51.72);
+    expect(newOrder.items[0].modifiers[0]._computed).toHaveProperty(
+      'amount',
+      2.945
+    );
+
+    expect(newOrder.items[1].modifiers[0]._computed).toHaveProperty(
+      'amount',
+      3.255
+    );
+    expect(newOrder.items[2].modifiers[0]._computed).toHaveProperty(
+      'amount',
+      3.72
+    );
+  });
+
+  test('CU-86dtxh2y5: #1- Get calculated Order, One item and fixed Fee Modifier', () => {
+    const newOrderFixedModifier = pricingService.order.calculate({
+      items: [
+        {
+          _id: 1,
+          price: 5,
+          quantity: 2,
+        },
+      ],
+      modifiers: [
+        {
+          compute: {
+            amount: 10,
+            type: 'fixed',
+            action: 'add',
+          },
+          name: 'fee 10 USD',
+          type: 'fee',
+        },
+      ],
+    });
+    expect(newOrderFixedModifier).toHaveProperty('total', 20);
+  });
+
+  test('CU-86dtxh2y5: #2.A- Get calculated Order, One item and fixed Fee Modifier (ignoreQuantity: false)', () => {
+    const newOrderFixedModifier = pricingService.order.calculate({
+      items: [
+        {
+          _id: 1,
+          price: 5,
+          quantity: 2,
+          modifiers: [
+            {
+              compute: {
+                amount: 20,
+                type: 'fixed',
+                action: 'add',
+              },
+              name: 'fee 20 USD',
+              type: 'fee',
+            },
+          ],
+        },
+      ],
+    });
+    expect(newOrderFixedModifier).toHaveProperty('total', 50);
+  });
+
+  test('CU-86dtxh2y5: #2.B- Get calculated Order, One item and fixed Fee Modifier (ignoreQuantity: true)', () => {
+    const newOrderFixedModifier = pricingService.order.calculate({
+      items: [
+        {
+          _id: 1,
+          price: 5,
+          quantity: 2,
+          modifiers: [
+            {
+              compute: {
+                amount: 20,
+                type: 'fixed',
+                action: 'add',
+              },
+              name: 'fee 20 USD',
+              type: 'fee',
+              properties: {
+                ignoreQuantity: true,
+              },
+            },
+          ],
+        },
+      ],
+    });
+    expect(newOrderFixedModifier).toHaveProperty('total', 30);
+  });
+
+  test('CU-86dtxh2y5: #3- Get calculated Order, One item and fixed Fee Modifier, using createFeeModifier', () => {
+    const newOrderFixedModifier = pricingService.order.calculate({
+      items: [
+        {
+          _id: 1,
+          price: 10,
+          quantity: 2,
+        },
+      ],
+      modifiers: [
+        pricingService.modifier.createFeeModifier({
+          amount: 100,
+          type: Types.FIXED,
+        }),
+      ],
+    });
+
+    expect(newOrderFixedModifier).toHaveProperty('total', 120);
+  });
+
+  test('CU-86dtxh2y5: #4- Get calculated Order, One item and fixed Fee Modifier, using createFeeModifier (ignoreQuantity: true)', () => {
+    const newOrderFixedModifier = pricingService.order.calculate({
+      items: [
+        {
+          _id: 1,
+          price: 10,
+          quantity: 2,
+        },
+      ],
+      modifiers: [
+        pricingService.modifier.createFeeModifier({
+          amount: 100,
+          type: Types.FIXED,
+          properties: {
+            ignoreQuantity: true,
+          },
+        }),
+      ],
+    });
+
+    expect(newOrderFixedModifier).toHaveProperty('total', 120);
   });
-  expect(newOrder.items[0]).toHaveProperty('total', 65.85);
-  expect(newOrder.items[1]).toHaveProperty('total', 53.03);
-  expect(newOrder.items[2]).toHaveProperty('total', 65.76);
-  expect(newOrder.items[3]).toHaveProperty('total', 144.71);
-  expect(newOrder.items[4]).toHaveProperty('total', 271.1);
-  expect(newOrder.items[5]).toHaveProperty('total', 109.91);
-  expect(newOrder.items[0].modifiers[0]._computed).toHaveProperty(
-    'amount',
-    -10.785
-  );
-  expect(newOrder.items[0].modifiers[1]._computed).toHaveProperty(
-    'amount',
-    4.7364125
-  );
-
-  expect(newOrder.items[1].modifiers[0]._computed).toHaveProperty(
-    'amount',
-    -8.685
-  );
-  expect(newOrder.items[1].modifiers[1]._computed).toHaveProperty(
-    'amount',
-    3.8141625
-  );
-
-  expect(newOrder.items[2].modifiers[0]._computed).toHaveProperty(
-    'amount',
-    -10.77
-  );
-  expect(newOrder.items[2].modifiers[1]._computed).toHaveProperty(
-    'amount',
-    4.729825
-  );
-
-  expect(newOrder.items[3].modifiers[0]._computed).toHaveProperty(
-    'amount',
-    -23.7
-  );
-  expect(newOrder.items[3].modifiers[1]._computed).toHaveProperty(
-    'amount',
-    10.40825
-  );
-
-  expect(newOrder.items[4].modifiers[0]._computed).toHaveProperty(
-    'amount',
-    -44.4
-  );
-  expect(newOrder.items[4].modifiers[1]._computed).toHaveProperty(
-    'amount',
-    19.499
-  );
-
-  expect(newOrder.items[5].modifiers[0]._computed).toHaveProperty(
-    'amount',
-    -18
-  );
-  expect(newOrder.items[5].modifiers[1]._computed).toHaveProperty(
-    'amount',
-    7.905
-  );
 });

package/lib/constants/Modifier/Types.js

@@ -2,6 +2,7 @@
 module.exports = {
   DISCOUNT: 'discount',
   TAX: 'tax',
+  FEE: 'fee',
   GRATUITY: 'gratuity',
   CREDIT: 'credit',
 };

package/lib/item/calculate.js

@@ -73,7 +73,8 @@ module.exports = ({ _, utils, actions, modifierActions }) => {
         let computedAmount =
           _modifier.compute.type === 'percentage' ||
           modifierActions.isIgnoreQuantity(_modifier) ||
-          math.gt(math.abs(computedAmountCalc), computedPriceCalc)
+          (!modifierActions.isFixedAdd(_modifier) &&
+            math.gt(math.abs(computedAmountCalc), computedPriceCalc))
             ? _computed.amount
             : computedAmountCalc;
 
@@ -98,7 +99,7 @@ module.exports = ({ _, utils, actions, modifierActions }) => {
 
     subTotals._actual = math.add(subTotals._simple, subTotals._included);
 
-    const total = math.toDecimalPlaces(actions.getTotal({ subTotals }));
+    const total = actions.getTotal({ subTotals });
 
     return {
       ...item,

package/lib/modifier/calculate.js

@@ -19,7 +19,9 @@ module.exports = ({ _, constants, utils, localization, actions }) => {
     const maxAmountProp = actions.getProperty(modifier, 'maxAmount');
 
     if (!options.skip) {
-      const maxAmountCalc = math.mul(options.price, options.quantity);
+      const maxAmountCalc = actions.isFixedAdd(modifier)
+        ? amount
+        : math.mul(options.price, options.quantity);
 
       const maxAmount = maxAmountProp
         ? math.min(maxAmountProp, maxAmountCalc)

package/lib/modifier/createDiscountModifier.js

@@ -2,7 +2,7 @@ module.exports = ({ constants, actions, localization }) => {
   const { Modifier } = constants;
 
   return function createDiscountModifier(params) {
-    const { amount, type } = params;
+    const { amount, type, properties } = params;
     const { formatAmount } = localization;
 
     const name =
@@ -18,6 +18,9 @@ module.exports = ({ constants, actions, localization }) => {
       },
       type: Modifier.Types.DISCOUNT,
       name: `${name} ${Modifier.Types.DISCOUNT}`,
+      properties: {
+        ...properties,
+      },
     });
   };
 };

package/lib/modifier/createFeeModifier.js

@@ -0,0 +1,27 @@
+module.exports = ({ constants, actions, localization }) => {
+  const { Modifier } = constants;
+
+  return function createFeeModifier(params) {
+    const { amount, type, properties } = params;
+    const { formatAmount } = localization;
+
+    const name =
+      type === Modifier.Compute.Types.PERCENTAGE
+        ? `${amount}%`
+        : `${formatAmount(amount)}`;
+
+    return actions.create({
+      compute: {
+        action: Modifier.Compute.Actions.ADD,
+        amount,
+        type,
+      },
+      type: Modifier.Types.FEE,
+      name: `${name} ${Modifier.Types.FEE}`,
+      properties: {
+        ignoreQuantity: true,
+        ...properties,
+      },
+    });
+  };
+};

package/lib/modifier/createIndirectModifier.js

@@ -35,10 +35,15 @@ module.exports = ({ _, utils, constants, actions }) => {
       !math.isZero(options.orderTotal) &&
       (type !== Modifier.Compute.Types.PERCENTAGE || modifier.type === 'credit')
     ) {
+      const modififierQuantity =
+        modifier.properties && modifier.properties.ignoreQuantity
+          ? 1
+          : options.itemQuantity;
+
       amountToApply = math.div(
         math.mul(modifierAmount, options.itemTotal),
         options.orderTotal,
-        options.itemQuantity
+        modififierQuantity
       );
     }
 

package/lib/modifier/index.js

@@ -8,6 +8,7 @@ const create = require('./create');
 const createAmountOverrideModifier = require('./createAmountOverrideModifier');
 const createCreditModifier = require('./createCreditModifier');
 const createDiscountModifier = require('./createDiscountModifier');
+const createFeeModifier = require('./createFeeModifier');
 const createGroupValuesModifier = require('./createGroupValuesModifier');
 const createIndirectModifier = require('./createIndirectModifier');
 const createSubscriptionModifier = require('./createSubscriptionModifier');
@@ -120,6 +121,10 @@ const isPaymentModifier = require('./isPaymentModifier');
 const createPaymentModifier = require('./createPaymentModifier');
 const isSubtract = require('./isSubtract');
 const isFixed = require('./isFixed');
+const isFixedAdd = require('./isFixedAdd');
+const isFee = require('./isFee');
+const isAdd = require('./isAdd');
+
 const isFixedDiscount = require('./isFixedDiscount');
 const removeLocked = require('./removeLocked');
 const hasItems = require('./hasItems');
@@ -143,6 +148,7 @@ const modifierActions = (deps = {}) => {
     createAmountOverrideModifier: createAmountOverrideModifier(innerDeps),
     createCreditModifier: createCreditModifier(innerDeps),
     createDiscountModifier: createDiscountModifier(innerDeps),
+    createFeeModifier: createFeeModifier(innerDeps),
     createGroupValuesModifier: createGroupValuesModifier(innerDeps),
     createIndirectModifier: createIndirectModifier(innerDeps),
     createSubscriptionModifier: createSubscriptionModifier(innerDeps),
@@ -260,6 +266,9 @@ const modifierActions = (deps = {}) => {
     removeLocked: removeLocked(innerDeps),
     hasItems: hasItems(innerDeps),
     getLockedModifiers: getLockedModifiers(innerDeps),
+    isFixedAdd: isFixedAdd(innerDeps),
+    isFee: isFee(innerDeps),
+    isAdd: isAdd(innerDeps),
   });
 
   Object.keys(freezedActions).forEach(actionName => {

package/lib/modifier/isAdd.js

@@ -0,0 +1,10 @@
+module.exports = ({ constants }) => {
+  const { Modifier } = constants;
+  return function isAdd(modifier) {
+    return !!(
+      modifier &&
+      modifier.compute &&
+      modifier.compute.action === Modifier.Compute.Actions.ADD
+    );
+  };
+};

package/lib/modifier/isFee.js

@@ -0,0 +1,6 @@
+module.exports = ({ constants }) => {
+  const { Modifier } = constants;
+  return function isFee(modifier) {
+    return !!(modifier && modifier.type === Modifier.Types.FEE);
+  };
+};

package/lib/modifier/isFixedAdd.js

@@ -0,0 +1,4 @@
+module.exports = ({ actions }) =>
+  function isFixedAdd(modifier) {
+    return actions.isAdd(modifier) && actions.isFixed(modifier);
+  };

package/lib/modifier/isFixedFee.js

@@ -0,0 +1,4 @@
+module.exports = ({ actions }) =>
+  function isFixedFee(modifier) {
+    return actions.isFee(modifier) && actions.isFixed(modifier);
+  };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@darkpos/pricing",
-  "version": "1.0.24",
+  "version": "1.0.26",
   "description": "Pricing calculator",
   "author": "Dark POS",
   "license": "ISC",
@@ -36,5 +36,5 @@
     "supertest": "^6.2.3",
     "supervisor": "^0.12.0"
   },
-  "gitHead": "a5c31d32fc9c21bc3d643d02e1171438b38f6826"
+  "gitHead": "bc31fb0ba85c5eba20c2d0df28ac5f858566372c"
 }