@darkpos/pricing 1.0.71 → 1.0.72

package/__TEST__/modifier/hasModifier.test.js

@@ -1651,7 +1651,13 @@ describe('hasModifier Function', () => {
       },
     };
 
-    const item = { _id: 1, itemId: 'ab', modifiers: [groupMod] };
+    const item = {
+      _id: 1,
+      itemId: 'ab',
+      modifiers: [groupMod],
+      price: 500,
+      total: 10000,
+    };
 
     const order = {
       items: [item],
@@ -1683,6 +1689,25 @@ describe('hasModifier Function', () => {
 
     expect(updatedOrder.items.length).toBe(2);
 
+    expect(updatedOrder.items[1]).toMatchObject({
+      _id: 1,
+      itemId: 'ab',
+      modifiers: [
+        {
+          _id: 'groupMod10',
+          attributes: ['group'],
+          properties: {
+            group: {
+              items: [{ _id: 'abc', name: 'related item abc', itemId: 'abc' }],
+              modifiers: [{ _id: '123', name: 'related mod' }],
+            },
+          },
+        },
+      ],
+      price: 500,
+      total: 10000,
+    });
+
     const result = pricingService.item.hasModifier({
       item: updatedOrder.items[1],
       modifier: groupMod,
@@ -1693,5 +1718,47 @@ describe('hasModifier Function', () => {
     });
 
     expect(result).toBe(true);
+
+    const { updatedOrder: updatedOrder2 } = pricingService.order.addItem({
+      order,
+      item: {
+        ...relatedModItem,
+        properties: {
+          relatedItem: true,
+          parentItemId: item.itemId,
+          parentId: item._id,
+          includeParent: false,
+          relatedItemId: relatedModItem._id,
+        },
+      },
+    });
+
+    expect(updatedOrder2.items.length).toBe(2);
+
+    expect(updatedOrder2.items[1]).toMatchObject({
+      _id: 1,
+      itemId: 'ab',
+      modifiers: [],
+      price: 0,
+      total: 0,
+      subTotals: {
+        _included: 0,
+        _xincluded: 0,
+        _direct: 0,
+        _xdirect: 0,
+        _actual: 0,
+      },
+    });
+
+    const result2 = pricingService.item.hasModifier({
+      item: updatedOrder2.items[1],
+      modifier: groupMod,
+      relatedItems: pricingService.order.getRelatedItems({
+        order: updatedOrder2,
+        item: updatedOrder2.items[1],
+      }),
+    });
+
+    expect(result2).toBe(true);
   });
 });

package/lib/modifier/isGroupOfModifiers.js

@@ -6,7 +6,8 @@ module.exports = ({ actions, constants }) => {
       actions.hasAttribute(modifier, Modifier.Attributes.GROUP) &&
       properties &&
       properties.group &&
-      (properties.group.modifiers || []).length
+      ((properties.group.modifiers || []).length ||
+        !!properties.group.manualModifiers)
     );
   };
 };

package/lib/order/addItem.js

@@ -168,8 +168,8 @@ module.exports = ({ actions, itemActions, modifierActions, settings, _ }) => {
         if (!_.isEmpty(inheritedModifiers))
           modifiersToAdd.push(...inheritedModifiers);
         // Remove parent if it is only a repair Item
-        if (!itemActions.isParentIncluded(orderItem)) {
-          order = actions.removeItem({ order, item: parent });
+        if (!itemActions.isParentIncluded(orderItem) && !!parent) {
+          order = actions.resetItem({ order, item: parent });
         }
       }
     }
@@ -249,7 +249,11 @@ module.exports = ({ actions, itemActions, modifierActions, settings, _ }) => {
       nextOrder.items.splice(idxToRemove, 1);
     }
 
-    if (itemActions.isRelatedItem(item) && itemActions.hasAddModifiers(item)) {
+    if (
+      itemActions.isRelatedItem(item) &&
+      itemActions.hasAddModifiers(item) &&
+      itemActions.isParentIncluded(item)
+    ) {
       const parentItem = itemActions.getParentItem(nextOrder.items, item);
       const parentIndex = actions.getItemIndex({
         order: nextOrder,

package/lib/order/index.js

@@ -88,6 +88,7 @@ const applyPayment = require('./applyPayment');
 const getBalance = require('./getBalance');
 const getLastLocation = require('./getLastLocation');
 const getModifierRelations = require('./getModifierRelations');
+const resetItem = require('./resetItem');
 
 const orderActions = (deps = {}) => {
   const actions = {};
@@ -187,6 +188,7 @@ const orderActions = (deps = {}) => {
     getOrdersBalance: getOrdersBalance(innerDeps),
     getLastLocation: getLastLocation(innerDeps),
     getModifierRelations: getModifierRelations(innerDeps),
+    resetItem: resetItem(innerDeps),
   });
 
   Object.keys(freezedActions).forEach(actionName => {

package/lib/order/resetItem.js

@@ -0,0 +1,25 @@
+module.exports = ({ itemActions }) =>
+  function resetItem({ item, order }) {
+    if (!item) return undefined;
+
+    const itemReset = itemActions.calculate(
+      {
+        ...item,
+        price: 0,
+        modifiers: [],
+      },
+      {}
+    );
+
+    const nextItems = order.items;
+    const parentIndex = order.items.findIndex(
+      eachItem => eachItem._id === itemReset._id
+    );
+
+    nextItems.splice(parentIndex, 1, itemReset);
+
+    return {
+      ...order,
+      items: nextItems,
+    };
+  };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@darkpos/pricing",
-  "version": "1.0.71",
+  "version": "1.0.72",
   "description": "Pricing calculator",
   "author": "Dark POS",
   "license": "ISC",
@@ -49,5 +49,5 @@
     "supertest": "^6.2.3",
     "supervisor": "^0.12.0"
   },
-  "gitHead": "95b6471cd80c91a665a81a66fe95e12cdf45ca27"
+  "gitHead": "2dc88094c2dbcebc04e0fbd817b45809a44dc403"
 }