react-context-store - npm Package Compare versions

Comparing version 1.0.0-beta.4 to 1.0.0-beta.5

changelog.md

@@ -10,2 +10,6 @@ # Changelog
 
+## [1.0.0-beta.5] - 2021-02-28
+
+- Fixed createOne overwriting existing indexes for arrays instead of inserting
+
 ## [1.0.0-beta.4] - 2021-02-23
@@ -12,0 +16,0 @@

dist/context-store--indexable/modifiers/get-create-one-context-data.js

@@ -49,8 +49,6 @@ "use strict";
 };
-var __spreadArrays = (this && this.__spreadArrays) || function () {
-    for (var s = 0, i = 0, il = arguments.length; i < il; i++) s += arguments[i].length;
-    for (var r = Array(s), k = 0, i = 0; i < il; i++)
-        for (var a = arguments[i], j = 0, jl = a.length; j < jl; j++, k++)
-            r[k] = a[j];
-    return r;
+var __spreadArray = (this && this.__spreadArray) || function (to, from) {
+    for (var i = 0, il = from.length, j = to.length; i < il; i++, j++)
+        to[j] = from[i];
+    return to;
 };
@@ -63,17 +61,19 @@ Object.defineProperty(exports, "__esModule", { value: true });
     return __awaiter(this, void 0, void 0, function () {
-        var action, error, getIndex, preload, value, e_1, _c;
-        return __generator(this, function (_d) {
-            switch (_d.label) {
+        var action, error, getIndex, preload, value, e_1;
+        return __generator(this, function (_c) {
+            switch (_c.label) {
                 case 0:
                     action = dataHandlers.action, error = dataHandlers.error, getIndex = dataHandlers.getIndex, preload = dataHandlers.preload;
                     value = null;
-                    _d.label = 1;
+                    _c.label = 1;
                 case 1:
-                    _d.trys.push([1, 4, , 9]);
+                    _c.trys.push([1, 4, , 5]);
                     return [4, setContextDataForCreateOne(contextData, setContextData, params, getIndex, shared_1.statefulStates.loading, preload)];
                 case 2:
-                    value = (_a = (_d.sent())) !== null && _a !== void 0 ? _a : value;
+                    value =
+                        (_a = (_c.sent())) !== null && _a !== void 0 ? _a : value;
                     return [4, setContextDataForCreateOne(contextData, setContextData, params, getIndex, shared_1.statefulStates.success, action)];
                 case 3:
-                    value = (_b = (_d.sent())) !== null && _b !== void 0 ? _b : value;
+                    value =
+                        (_b = (_c.sent())) !== null && _b !== void 0 ? _b : value;
                     if (value == null) {
@@ -85,24 +85,20 @@ return [2, Promise.reject(shared_1.errorMessages.actionReturnedNull)];
                     }
-                    return [3, 9];
+                    return [3, 5];
                 case 4:
-                    e_1 = _d.sent();
-                    _d.label = 5;
-                case 5:
-                    _d.trys.push([5, 7, , 8]);
-                    return [4, setContextDataForCreateOne(contextData, setContextData, params, getIndex, shared_1.statefulStates.error, error)];
-                case 6:
-                    value = _d.sent();
-                    if (value == null) {
-                        return [2, Promise.reject(shared_1.errorMessages.indexNotFound)];
+                    e_1 = _c.sent();
+                    try {
+                        setContextData(__assign(__assign({}, contextData), { state: shared_1.statefulStates.error }));
+                        if (typeof e_1 === "string") {
+                            return [2, Promise.reject(e_1)];
+                        }
+                        else {
+                            return [2, Promise.reject(e_1.message || shared_1.errorMessages.unknownPreloadOrActionReject)];
+                        }
                     }
-                    else {
-                        return [2, Promise.resolve(value)];
+                    catch (_d) {
+                        setContextData(__assign(__assign({}, contextData), { state: shared_1.statefulStates.error }));
+                        return [2, Promise.reject(shared_1.errorMessages.errorCallbackRejected)];
                     }
-                    return [3, 8];
-                case 7:
-                    _c = _d.sent();
-                    setContextData(__assign(__assign({}, contextData), { state: shared_1.statefulStates.error }));
-                    return [2, Promise.reject(shared_1.errorMessages.errorCallbackRejected)];
-                case 8: return [3, 9];
-                case 9: return [2];
+                    return [3, 5];
+                case 5: return [2];
             }
@@ -145,4 +141,4 @@ });
     if (Array.isArray(data)) {
-        var newData = __spreadArrays(data);
-        newData.splice(index, 1, newValue);
+        var newData = __spreadArray([], data);
+        newData.splice(index, 0, newValue);
         return __assign(__assign({}, store), { data: newData, state: state });
@@ -149,0 +145,0 @@ }

dist/context-store--indexable/modifiers/get-delete-one-context-data.d.ts

@@ -10,3 +10,3 @@ /// <reference types="react" />
 }): Promise<IndexableContextStoreValue<TContextStore>>;
-export declare function setContextDataForDeleteOne<Params, TContextStore extends IndexableContextStore<unknown>>(contextData: TContextStore, setContextData: React.Dispatch<React.SetStateAction<TContextStore>>, params: Params, getIndex: (params: Params) => IndexableContextStoreKey<TContextStore>, state: Stateful["state"], action?: (params: Params) => Promise<IndexableContextStoreValue<TContextStore> | null>, deleteIfNull?: boolean): Promise<Record<string | number | symbol, unknown>[IndexableContextStoreKey<TContextStore>]>;
+export declare function setContextDataForDeleteOne<Params, TContextStore extends IndexableContextStore<unknown>>(contextData: TContextStore, setContextData: React.Dispatch<React.SetStateAction<TContextStore>>, params: Params, getIndex: (params: Params) => IndexableContextStoreKey<TContextStore>, state: Stateful["state"], action?: (params: Params) => Promise<IndexableContextStoreValue<TContextStore> | null>, deleteIfNull?: boolean): Promise<import("../interfaces").IndexableContextStoreData<unknown>[IndexableContextStoreKey<TContextStore>]>;
 export declare function getUpdatedContextDataForDeleteOne<TContextStore extends IndexableContextStore<unknown>>(store: TContextStore, index: IndexableContextStoreKey<TContextStore>, value: null | IndexableContextStoreValue<TContextStore>, state: keyof typeof statefulStates): TContextStore;

dist/context-store--indexable/modifiers/get-delete-one-context-data.js

@@ -49,8 +49,6 @@ "use strict";
 };
-var __spreadArrays = (this && this.__spreadArrays) || function () {
-    for (var s = 0, i = 0, il = arguments.length; i < il; i++) s += arguments[i].length;
-    for (var r = Array(s), k = 0, i = 0; i < il; i++)
-        for (var a = arguments[i], j = 0, jl = a.length; j < jl; j++, k++)
-            r[k] = a[j];
-    return r;
+var __spreadArray = (this && this.__spreadArray) || function (to, from) {
+    for (var i = 0, il = from.length, j = to.length; i < il; i++, j++)
+        to[j] = from[i];
+    return to;
 };
@@ -75,6 +73,8 @@ Object.defineProperty(exports, "__esModule", { value: true });
                 case 2:
-                    value = (_a = (_d.sent())) !== null && _a !== void 0 ? _a : value;
+                    value =
+                        (_a = (_d.sent())) !== null && _a !== void 0 ? _a : value;
                     return [4, setContextDataForDeleteOne(contextData, setContextData, params, getIndex, shared_1.statefulStates.success, action)];
                 case 3:
-                    value = (_b = (_d.sent())) !== null && _b !== void 0 ? _b : value;
+                    value =
+                        (_b = (_d.sent())) !== null && _b !== void 0 ? _b : value;
                     if (value == null) {
@@ -150,3 +150,3 @@ return [2, Promise.reject(shared_1.errorMessages.indexNotFound)];
     if (Array.isArray(data)) {
-        var newData = __spreadArrays(data);
+        var newData = __spreadArray([], data);
         newData.splice(index, 1);
@@ -153,0 +153,0 @@ return __assign(__assign({}, store), { data: newData, state: state });

dist/context-store--indexable/modifiers/get-update-one-context-data.d.ts

@@ -10,3 +10,3 @@ /// <reference types="react" />
 }): Promise<IndexableContextStoreValue<TContextStore>>;
-export declare function setContextDataForUpdateOne<Params, TContextStore extends IndexableContextStore<any>>(contextData: TContextStore, setContextData: React.Dispatch<React.SetStateAction<TContextStore>>, params: Params, getIndex: (params: Params) => IndexableContextStoreKey<TContextStore>, state: Stateful["state"], action?: (params: Params) => Promise<Partial<IndexableContextStoreValue<TContextStore>> | null>): Promise<Record<string | number | symbol, any>[IndexableContextStoreKey<TContextStore>]>;
+export declare function setContextDataForUpdateOne<Params, TContextStore extends IndexableContextStore<any>>(contextData: TContextStore, setContextData: React.Dispatch<React.SetStateAction<TContextStore>>, params: Params, getIndex: (params: Params) => IndexableContextStoreKey<TContextStore>, state: Stateful["state"], action?: (params: Params) => Promise<Partial<IndexableContextStoreValue<TContextStore>> | null>): Promise<import("../interfaces").IndexableContextStoreData<any>[IndexableContextStoreKey<TContextStore>]>;
 export declare function getUpdatedContextDataForUpdateOne<TContextStore extends IndexableContextStore<any>>(store: TContextStore, index: IndexableContextStoreKey<TContextStore>, value: null | Partial<IndexableContextStoreValue<TContextStore>>, state: keyof typeof statefulStates): TContextStore;

dist/context-store--indexable/modifiers/get-update-one-context-data.js

@@ -49,8 +49,6 @@ "use strict";
 };
-var __spreadArrays = (this && this.__spreadArrays) || function () {
-    for (var s = 0, i = 0, il = arguments.length; i < il; i++) s += arguments[i].length;
-    for (var r = Array(s), k = 0, i = 0; i < il; i++)
-        for (var a = arguments[i], j = 0, jl = a.length; j < jl; j++, k++)
-            r[k] = a[j];
-    return r;
+var __spreadArray = (this && this.__spreadArray) || function (to, from) {
+    for (var i = 0, il = from.length, j = to.length; i < il; i++, j++)
+        to[j] = from[i];
+    return to;
 };
@@ -74,6 +72,8 @@ Object.defineProperty(exports, "__esModule", { value: true });
                 case 2:
-                    value = (_a = (_d.sent())) !== null && _a !== void 0 ? _a : value;
+                    value =
+                        (_a = (_d.sent())) !== null && _a !== void 0 ? _a : value;
                     return [4, setContextDataForUpdateOne(contextData, setContextData, params, getIndex, shared_1.statefulStates.success, action)];
                 case 3:
-                    value = (_b = (_d.sent())) !== null && _b !== void 0 ? _b : value;
+                    value =
+                        (_b = (_d.sent())) !== null && _b !== void 0 ? _b : value;
                     if (value == null) {
@@ -149,3 +149,3 @@ return [2, Promise.reject(shared_1.errorMessages.actionReturnedNull)];
     if (Array.isArray(data)) {
-        var newData = __spreadArrays(data);
+        var newData = __spreadArray([], data);
         newData.splice(index, 1, newValue);
@@ -152,0 +152,0 @@ return __assign(__assign({}, store), { data: newData, state: state });

dist/context-store--stateful-indexable/modifiers/get-create-one-context-data.js

@@ -49,8 +49,6 @@ "use strict";
 };
-var __spreadArrays = (this && this.__spreadArrays) || function () {
-    for (var s = 0, i = 0, il = arguments.length; i < il; i++) s += arguments[i].length;
-    for (var r = Array(s), k = 0, i = 0; i < il; i++)
-        for (var a = arguments[i], j = 0, jl = a.length; j < jl; j++, k++)
-            r[k] = a[j];
-    return r;
+var __spreadArray = (this && this.__spreadArray) || function (to, from) {
+    for (var i = 0, il = from.length, j = to.length; i < il; i++, j++)
+        to[j] = from[i];
+    return to;
 };
@@ -71,9 +69,11 @@ Object.defineProperty(exports, "__esModule", { value: true });
                 case 1:
-                    _c.trys.push([1, 4, , 5]);
+                    _c.trys.push([1, 4, , 8]);
                     return [4, setContextDataForCreateOne(contextData, setContextData, params, getIndex, shared_1.statefulStates.loading, preload)];
                 case 2:
-                    value = (_a = (_c.sent())) !== null && _a !== void 0 ? _a : value;
+                    value =
+                        (_a = (_c.sent())) !== null && _a !== void 0 ? _a : value;
                     return [4, setContextDataForCreateOne(contextData, setContextData, params, getIndex, shared_1.statefulStates.success, action)];
                 case 3:
-                    value = (_b = (_c.sent())) !== null && _b !== void 0 ? _b : value;
+                    value =
+                        (_b = (_c.sent())) !== null && _b !== void 0 ? _b : value;
                     if (value == null) {
@@ -85,6 +85,14 @@ return [2, Promise.reject(shared_1.errorMessages.actionReturnedNull)];
                     }
-                    return [3, 5];
+                    return [3, 8];
                 case 4:
                     e_1 = _c.sent();
+                    if (!error) return [3, 6];
+                    return [4, setContextDataForCreateOne(contextData, setContextData, params, getIndex, shared_1.statefulStates.error, error)];
+                case 5:
+                    _c.sent();
+                    return [3, 7];
+                case 6:
                     setContextData(__assign(__assign({}, contextData), { state: shared_1.statefulStates.success }));
+                    _c.label = 7;
+                case 7:
                     if (typeof e_1 === "string") {
@@ -96,4 +104,4 @@ return [2, Promise.reject(e_1)];
                     }
-                    return [3, 5];
-                case 5: return [2];
+                    return [3, 8];
+                case 8: return [2];
             }
@@ -139,4 +147,4 @@ });
     if (Array.isArray(data)) {
-        var newData = __spreadArrays(data);
-        newData.splice(index, 1, newValue);
+        var newData = __spreadArray([], data);
+        newData.splice(index, 0, newValue);
         return __assign(__assign({}, store), { data: newData });
@@ -143,0 +151,0 @@ }

dist/context-store--stateful-indexable/modifiers/get-delete-one-context-data.js

@@ -49,8 +49,6 @@ "use strict";
 };
-var __spreadArrays = (this && this.__spreadArrays) || function () {
-    for (var s = 0, i = 0, il = arguments.length; i < il; i++) s += arguments[i].length;
-    for (var r = Array(s), k = 0, i = 0; i < il; i++)
-        for (var a = arguments[i], j = 0, jl = a.length; j < jl; j++, k++)
-            r[k] = a[j];
-    return r;
+var __spreadArray = (this && this.__spreadArray) || function (to, from) {
+    for (var i = 0, il = from.length, j = to.length; i < il; i++, j++)
+        to[j] = from[i];
+    return to;
 };
@@ -75,6 +73,8 @@ Object.defineProperty(exports, "__esModule", { value: true });
                 case 2:
-                    value = (_a = (_c.sent())) !== null && _a !== void 0 ? _a : value;
+                    value =
+                        (_a = (_c.sent())) !== null && _a !== void 0 ? _a : value;
                     return [4, setContextDataForDeleteOne(contextData, setContextData, params, getIndex, shared_1.statefulStates.success, action)];
                 case 3:
-                    value = (_b = (_c.sent())) !== null && _b !== void 0 ? _b : value;
+                    value =
+                        (_b = (_c.sent())) !== null && _b !== void 0 ? _b : value;
                     if (value == null) {
@@ -162,3 +162,3 @@ return [2, Promise.reject(shared_1.errorMessages.actionReturnedNull)];
     if (Array.isArray(data)) {
-        var newData = __spreadArrays(data);
+        var newData = __spreadArray([], data);
         newData.splice(index, 1);
@@ -165,0 +165,0 @@ return __assign(__assign({}, store), { data: newData });

dist/context-store--stateful-indexable/modifiers/get-update-one-context-data.js

@@ -49,8 +49,6 @@ "use strict";
 };
-var __spreadArrays = (this && this.__spreadArrays) || function () {
-    for (var s = 0, i = 0, il = arguments.length; i < il; i++) s += arguments[i].length;
-    for (var r = Array(s), k = 0, i = 0; i < il; i++)
-        for (var a = arguments[i], j = 0, jl = a.length; j < jl; j++, k++)
-            r[k] = a[j];
-    return r;
+var __spreadArray = (this && this.__spreadArray) || function (to, from) {
+    for (var i = 0, il = from.length, j = to.length; i < il; i++, j++)
+        to[j] = from[i];
+    return to;
 };
@@ -74,6 +72,8 @@ Object.defineProperty(exports, "__esModule", { value: true });
                 case 2:
-                    value = (_a = (_c.sent())) !== null && _a !== void 0 ? _a : value;
+                    value =
+                        (_a = (_c.sent())) !== null && _a !== void 0 ? _a : value;
                     return [4, setContextDataForUpdateOne(contextData, setContextData, params, getIndex, shared_1.statefulStates.success, action)];
                 case 3:
-                    value = (_b = (_c.sent())) !== null && _b !== void 0 ? _b : value;
+                    value =
+                        (_b = (_c.sent())) !== null && _b !== void 0 ? _b : value;
                     if (value == null) {
@@ -159,3 +159,3 @@ return [2, Promise.reject(shared_1.errorMessages.actionReturnedNull)];
     if (Array.isArray(data)) {
-        var newData = __spreadArrays(data);
+        var newData = __spreadArray([], data);
         newData.splice(index, 1, newValue);
@@ -162,0 +162,0 @@ return __assign(__assign({}, store), { data: newData });

package.json

 {
   "author": "Snowcoders",
   "description": "A two way binding solution that uses React context as a storage mechanism.",
-  "dependencies": {},
   "devDependencies": {
@@ -23,5 +22,5 @@ "@snowcoders/renovate-config": "2.0.2",
     "rimraf": "3.0.2",
-    "ts-jest": "26.5.1",
+    "ts-jest": "26.5.2",
     "tslib": "2.1.0",
-    "typescript": "4.1.5"
+    "typescript": "4.2.2"
   },
@@ -47,3 +46,3 @@ "license": "MIT",
   "sideEffects": false,
-  "version": "1.0.0-beta.4"
+  "version": "1.0.0-beta.5"
 }

No alert changes

Improved metrics

Lines of code

1627

increased by0.31%

Worsened metrics

Total package byte prevSize

103033

decreased by-0.16%

No dependency changes