create helper.assingAll() and

helper.setNoneEnumProps()

Author: biglari-test

src/com/react-dynamic-tabs/utils/api/api.factory.js

@@ -3,16 +3,14 @@ const { throwMissingParam: missingParamEr, throwInvalidParam: invalidParamEr } =
 export const apiConstructor = function (getDeps, param = { options: {} }) {
     param.options = param.options || {};
     const { optionManagerIns, helper, activedTabsHistoryIns } = getDeps.call(this, param.options);
-    this.optionManager = optionManagerIns;
-    this.helper = helper;
-    this.activedTabsHistory = activedTabsHistoryIns;
+    helper.setNoneEnumProps(this, { helper, optionManager: optionManagerIns, activedTabsHistory: activedTabsHistoryIns });
     this._setUserProxy()._alterOnChangeCallback()._subscribeCallbacksOptions()._subscribeSelectedTabsHistory();
 };
-export const apiMethods = {
+const _apiProps = {
     _setUserProxy: function () {
         const userProxy = {};
         for (var prop in this)
-            if (prop[0] !== '_') {
+            if (prop[0] !== '_' && prop !== 'constructor') {
                 const propValue = this[prop];
                 if (typeof propValue === 'function') {
                     userProxy[prop] = propValue.bind(this);
@@ -69,14 +67,6 @@ export const apiMethods = {
     },
     isSelected: function (id = missingParamEr('isSelected')) { return this._state.selectedTabID == id; },
     isOpen: function (id = missingParamEr('isOpen')) { return this._state.openTabIDs.indexOf(id) >= 0; },
-    eventHandlerFactory: function ({ e, id }) {
-        const { beforeClose, beforeSelect } = this.getOptions(), el = e.target, parentEl = el.parentElement
-            , { close, tab } = this.getSetting().cssClasses;
-        if (el.className.includes(close) && parentEl && parentEl.lastChild && (parentEl.lastChild == el) && parentEl.className.includes(tab))
-            beforeClose.call(this.userProxy, e, id) && this.close(id);
-        else
-            beforeSelect.call(this.userProxy, e, id) && this.select(id);
-    },
     _getOnChangePromise: function () {
         return new (Promise)(resolve => { this.one('onChange', () => { resolve.call(this.userProxy); }); });
     },
@@ -138,3 +128,14 @@ export const apiMethods = {
         return result;
     }
 };
+Helper.setNoneEnumProps(_apiProps, {
+    eventHandlerFactory: function ({ e, id }) {
+        const { beforeClose, beforeSelect } = this.getOptions(), el = e.target, parentEl = el.parentElement
+            , { close, tab } = this.getSetting().cssClasses;
+        if (el.className.includes(close) && parentEl && parentEl.lastChild && (parentEl.lastChild == el) && parentEl.className.includes(tab))
+            beforeClose.call(this.userProxy, e, id) && this.close(id);
+        else
+            beforeSelect.call(this.userProxy, e, id) && this.select(id);
+    }
+});
+export const apiProps = _apiProps;

src/com/react-dynamic-tabs/utils/api/api.js

@@ -1,4 +1,4 @@
-import { apiMethods, apiConstructor } from './api.factory';
+import { apiProps, apiConstructor } from './api.factory';
 import OptionManager from './optionManager/optionManager.js';
 import helper from '../helper';
 import ActivedTabsHistory from './activedTabsHistory';
@@ -12,7 +12,6 @@ const getDeps = function (options) {
     Pub_Sub.call(this);
     return { activedTabsHistoryIns, optionManagerIns, helper };
 };
-apiConstructor.prototype = Object.create(Object.assign({}, BaseApi.prototype, Tabs.prototype, Pub_Sub.prototype, apiMethods));
-apiConstructor.prototype.constructor = apiConstructor;
+helper.assingAll(apiConstructor.prototype, BaseApi.prototype, Tabs.prototype, Pub_Sub.prototype, apiProps).constructor = apiConstructor;
 export default apiConstructor.bind(null, getDeps);
 

src/com/react-dynamic-tabs/utils/api/baseApi.js

@@ -1,25 +1,30 @@
 import actions from '../stateManagement/actions';
+import Helper from '../helper.js';
 function BaseApi(helper) {
-    this.forceUpdateState = {};
     this._helper = helper;
     this._state = {};
-    this.stateRef = {};
     this._perviousState = {};
     this._dispatch = () => { };
+    helper.setNoneEnumProps(this, {
+        forceUpdateState: {},
+        stateRef: {}
+    });
 }
-BaseApi.prototype.updateReducer = function (state, dispatch) {
-    this.stateRef = state;
-    this._perviousState = this._helper.getCopyState(this._state);
-    this._state = this._helper.getCopyState(state);
-    this._perviousState = this._perviousState.hasOwnProperty('openTabIDs') ? this._perviousState :
-        this._helper.getCopyState(this._state);
-    this._dispatch = dispatch;
-};
 BaseApi.prototype._select = function (tabId) { this._dispatch({ type: actions.active, tabId }); };
 BaseApi.prototype._close = function (tabId) { this._dispatch({ type: actions.close, tabId }); };
 BaseApi.prototype._open = function (tabId) { this._dispatch({ type: actions.open, tabId }); };
 BaseApi.prototype._refresh = function () {
     this.forceUpdateState = {};
     this._dispatch({ type: actions.refresh });
 };
+Helper.setNoneEnumProps(BaseApi.prototype, {
+    updateReducer: function (state, dispatch) {
+        this.stateRef = state;
+        this._perviousState = this._helper.getCopyState(this._state);
+        this._state = this._helper.getCopyState(state);
+        this._perviousState = this._perviousState.hasOwnProperty('openTabIDs') ? this._perviousState :
+            this._helper.getCopyState(this._state);
+        this._dispatch = dispatch;
+    }
+});
 export default BaseApi;

src/com/react-dynamic-tabs/utils/helper.js

@@ -8,6 +8,25 @@ helper.getCopyState = function (state) {
         newState.openTabIDs = [...newState.openTabIDs];
     return newState;
 };
+helper.assingAll = function (targetObj, ...sourcObjs) {
+    // copy all enumerable and none-enumerable properties into target
+    sourcObjs.map(sourcObj => {
+        Object.getOwnPropertyNames(sourcObj).map(prop => {
+            targetObj[prop] = sourcObj[prop];
+        });
+    });
+    return targetObj;
+};
+helper.setNoneEnumProps = function (obj, props) {
+    const noneEnum = {};
+    Object.keys(props).map(prop => {
+        noneEnum[prop] = {
+            writable: true,
+            value: props[prop]
+        };
+    });
+    return Object.defineProperties(obj, noneEnum);
+};
 helper.getArraysDiff = function (arr1, arr2) {
     const arr1Copy = [...arr1], arr2Copy = [...arr2];
     arr1.map(item => {