rename closeTabIDs parameter to closedTabIDs

Author: dev-javascript

src/useDynamicTabs/useDynamicTabs.js

@@ -28,9 +28,9 @@ function useDynamicTabs(getDeps, options = {}) {
   }, []);
   useLayoutEffect(() => {
     const oldState = api.getCopyPerviousData(),
-      [openedTabIDs, closeTabIDs] = api.helper.getArraysDiff(state.openTabIDs, oldState.openTabIDs),
+      [openedTabIDs, closedTabIDs] = api.helper.getArraysDiff(state.openTabIDs, oldState.openTabIDs),
       isSwitched = oldState.selectedTabID !== state.selectedTabID;
-    api.onChange({newState: state, oldState, closeTabIDs, openedTabIDs, isSwitched});
+    api.onChange({newState: state, oldState, closedTabIDs, openedTabIDs, isSwitched});
   }, [state]);
   useLayoutEffect(() => {
     api.trigger('_onFlushEffects', api.userProxy, {currentData: api.getCopyData(), instance: api.userProxy});

src/utils/api/api.factory.js

@@ -51,9 +51,9 @@ const _apiProps = {
     return this;
   },
   _subscribeSelectedTabsHistory: function () {
-    this.on('onChange', ({currentData, perviousData, closeTabIDs}) => {
-      for (let i = 0, l = closeTabIDs.length; i < l; i++) {
-        this.activedTabsHistory.remove(closeTabIDs[i]);
+    this.on('onChange', ({currentData, perviousData, closedTabIDs}) => {
+      for (let i = 0, l = closedTabIDs.length; i < l; i++) {
+        this.activedTabsHistory.remove(closedTabIDs[i]);
       }
       const isSwitched = perviousData.selectedTabID !== currentData.selectedTabID;
       if (isSwitched && this.isOpen(perviousData.selectedTabID) && !this.isSelected(perviousData.selectedTabID))
@@ -167,16 +167,16 @@ const _apiProps = {
   },
 };
 Helper.setNoneEnumProps(_apiProps, {
-  onChange: function ({newState, oldState, closeTabIDs, openedTabIDs, isSwitched}) {
-    if (isSwitched || openedTabIDs.length || closeTabIDs.length) {
+  onChange: function ({newState, oldState, closedTabIDs, openedTabIDs, isSwitched}) {
+    if (isSwitched || openedTabIDs.length || closedTabIDs.length) {
       this.trigger('onChange', this.userProxy, {
         currentData: {...newState},
         perviousData: {...oldState},
-        closeTabIDs,
+        closedTabIDs,
         openedTabIDs,
       });
       openedTabIDs.length && this.trigger('onOpen', this.userProxy, openedTabIDs);
-      closeTabIDs.length && this.trigger('onClose', this.userProxy, closeTabIDs);
+      closedTabIDs.length && this.trigger('onClose', this.userProxy, closedTabIDs);
       isSwitched &&
         this.trigger('onSelect', this.userProxy, {
           currentSelectedTabId: newState.selectedTabID,

src/utils/api/api.factory.test.js

@@ -323,7 +323,7 @@ describe('context of callback options should be userProxy object : ', () => {
     };
     const obj = new apiConstructor(getDeps, {options});
     expect(options.onChange.mock.calls.length === 0).toBe(true);
-    obj.trigger('onChange', obj.userProxy, {currentData: {}, perviousData: {}, closeTabIDs: [], openTabIDs: []});
+    obj.trigger('onChange', obj.userProxy, {currentData: {}, perviousData: {}, closedTabIDs: [], openTabIDs: []});
     expect(options.onChange.mock.calls.length === 1).toBe(true);
   });
 });
@@ -475,8 +475,8 @@ describe('Api.prototype._subscribeSelectedTabsHistory : ', () => {
     obj.activedTabsHistory.remove = jest.fn(() => {});
     const currentData = {selectedTabID: '2', openTabIDs: ['2']};
     const perviousData = {selectedTabID: '2', openTabIDs: ['2']};
-    const closeTabIDs = ['1', '3'];
-    obj.trigger('onChange', obj.userProxy, {currentData, perviousData, closeTabIDs, openTabIDs: []});
+    const closedTabIDs = ['1', '3'];
+    obj.trigger('onChange', obj.userProxy, {currentData, perviousData, closedTabIDs, openTabIDs: []});
     expect(obj.activedTabsHistory.remove.mock.calls.length).toBe(2);
     expect(obj.activedTabsHistory.remove.mock.calls[0][0]).toBe('1');
     expect(obj.activedTabsHistory.remove.mock.calls[1][0]).toBe('3');
@@ -494,7 +494,7 @@ describe('Api.prototype._subscribeSelectedTabsHistory : ', () => {
     });
     const currentData = {selectedTabID: '2', openTabIDs: ['1', '2']};
     const perviousData = {selectedTabID: '1', openTabIDs: ['1', '2']};
-    obj.trigger('onChange', obj.userProxy, {currentData, perviousData, closeTabIDs: [], openTabIDs: []});
+    obj.trigger('onChange', obj.userProxy, {currentData, perviousData, closedTabIDs: [], openTabIDs: []});
     expect(obj.activedTabsHistory.add.mock.calls.length).toBe(1);
     expect(obj.activedTabsHistory.add.mock.calls[0][0]).toBe('1');
   });