@steroidsjs/core 3.0.69 → 3.0.71

package/actions/list.d.ts

@@ -111,10 +111,9 @@ export declare const LIST_TOGGLE_ALL = "@list/toggle_all";
 export declare const LIST_SET_LAYOUT = "@list/set_layout";
 export declare const LIST_CHANGE_ACTION = "@list/change_action";
 export declare const LIST_SELECT_ITEM = "@list/select_item";
-export declare const httpFetchHandler: (list: IList, query: any, { http, httpCancelToken }: {
+export declare const httpFetchHandler: (list: IList, query: any, { http }: {
     http: any;
-    httpCancelToken: any;
-}) => any;
+}, options?: any) => any;
 export declare const localFetchHandler: (list: IList, query: Record<string, unknown>) => {
     items: any[];
     total: number;

package/actions/list.js

@@ -62,17 +62,16 @@ var createList = function (listId, props) { return ({
     sortAttribute: (0, get_1["default"])(props, '_sort.attribute') || null,
     layoutAttribute: (0, get_1["default"])(props, '_layout.attribute') || null
 }); };
-var httpFetchHandler = function (list, query, _a) {
-    var http = _a.http, httpCancelToken = _a.httpCancelToken;
+var httpFetchHandler = function (list, query, _a, options) {
+    var http = _a.http;
+    if (options === void 0) { options = {}; }
     var url = list.action;
     if (list.scope) {
-        url
-            += (url.indexOf('?') !== -1 ? '&' : '?') + 'scope=' + list.scope.join(',');
+        url += (url.indexOf('?') !== -1 ? '&' : '?') + 'scope=' + list.scope.join(',');
     }
+    // options поддерживает { cancelToken } или { signal } (если использовать AbortController)
     return http
-        .send(list.actionMethod, url || window.location.pathname, query, {
-        cancelToken: httpCancelToken
-    })
+        .send(list.actionMethod, url || window.location.pathname, query, options)
         .then(function (response) { return response.data; })["catch"](function (error) {
         if (typeof list.onError === 'function') {
             list.onError(error);
@@ -155,7 +154,6 @@ var listChangeAction = function (listId, action) { return ({
     action: action
 }); };
 exports.listChangeAction = listChangeAction;
-var _cancelSourceByList = new Map();
 /**
  * Update query values and send request
  * @param listId
@@ -180,77 +178,91 @@ var listFetch = function (listId, query) {
             formValues[key] = query[key];
             toDispatch.push((0, form_2.formChange)(list.formId, key, query[key]));
         });
+        // Отменяем предыдущий запрос для этого listId (если есть)
+        components.http._promises = components.http._promises || {};
+        var prev = components.http._promises[listId];
+        if (prev && typeof prev.cancel === 'function') {
+            // для axios.CancelToken.source()
+            prev.cancel('Cancelled by new request for list ' + listId);
+        }
+        else if (prev && typeof prev.abort === 'function') {
+            // для AbortController
+            prev.abort();
+        }
+        // Создаем новый источник отмены
+        var source = axios_1["default"].CancelToken.source();
+        components.http._promises[listId] = source;
+        var options = { cancelToken: source.token };
+        // Set `Loading...`
         if (list.isRemote) {
-            // Set `Loading...`
             toDispatch.push({
                 listId: listId,
                 type: exports.LIST_BEFORE_FETCH
             });
         }
-        // Axios cancel logic: cancel previous request for this listId
-        var prevSource = _cancelSourceByList.get(listId);
-        if (prevSource) {
-            // отменяем предыдущий активный запрос (если он ещё выполняется)
-            try {
-                prevSource.cancel('Cancelled by new request');
-            }
-            catch (e) {
-                /* ignore */
-            }
-        }
-        var cancelSource = axios_1["default"].CancelToken.source();
-        _cancelSourceByList.set(listId, cancelSource);
-        // Send request
-        toDispatch.push(Promise.resolve(onFetch(list, formValues, __assign(__assign({}, components), { httpCancelToken: cancelSource.token }))).then(function (data) {
-            var _a;
-            // cleanup: текущий источник уже отработал
-            _cancelSourceByList["delete"](listId);
-            // Skip on empty
-            if (!data) {
+        toDispatch.push([
+            Promise.resolve(onFetch(list, formValues, components, options))
+                .then(function (data) {
+                var _a;
+                // Skip on empty
+                if (!data) {
+                    return [];
+                }
+                // Check list is not destroy
+                if (!getState().list.lists[listId]) {
+                    return [];
+                }
+                if ((0, isArray_1["default"])(data)) {
+                    data = {
+                        items: data,
+                        total: data.length,
+                        meta: null
+                    };
+                }
+                var items = data.items || [];
+                var total = data.total || items.length || null;
+                var page = formValues[list.pageAttribute];
+                var pageSize = formValues[list.pageSizeAttribute];
+                var totalPages = Math.ceil(((list === null || list === void 0 ? void 0 : list.total) || 0) / (pageSize || 1));
+                var hasNextPage = (_a = data === null || data === void 0 ? void 0 : data.hasNextPage) !== null && _a !== void 0 ? _a : (page !== totalPages || null);
+                return [
+                    (0, form_2.formSetErrors)(list.formId, data.errors || null),
+                    {
+                        items: items,
+                        total: total,
+                        hasNextPage: hasNextPage,
+                        meta: data.meta || null,
+                        page: page,
+                        pageSize: pageSize,
+                        listId: listId,
+                        defaultPageValue: list.defaultPageValue,
+                        type: exports.LIST_AFTER_FETCH
+                    },
+                ];
+            })["catch"](function (error) {
+                // Если это отмена — просто игнорируем ошибку
+                if (axios_1["default"].isCancel && axios_1["default"].isCancel(error)) {
+                    return [];
+                }
+                // Handle cancellation quietly
+                // axios v0.x: axios.isCancel(error)
+                var isAxiosCancel = typeof axios_1["default"].isCancel === 'function' && axios_1["default"].isCancel(error);
+                if (isAxiosCancel) {
+                    // игнорируем, это отмена запроса
+                    return [];
+                }
+                // прочие ошибки — можно пробросить или обработать
+                if (typeof list.onError === 'function') {
+                    list.onError(error);
+                }
                 return [];
-            }
-            // Check list is not destroy
-            if (!getState().list.lists[listId]) {
-                return [];
-            }
-            if ((0, isArray_1["default"])(data)) {
-                data = {
-                    items: data,
-                    total: data.length,
-                    meta: null
-                };
-            }
-            var items = data.items || [];
-            var total = data.total || items.length || null;
-            var page = formValues[list.pageAttribute];
-            var pageSize = formValues[list.pageSizeAttribute];
-            var totalPages = Math.ceil(((list === null || list === void 0 ? void 0 : list.total) || 0) / (pageSize || 1));
-            var hasNextPage = (_a = data === null || data === void 0 ? void 0 : data.hasNextPage) !== null && _a !== void 0 ? _a : (page !== totalPages || null);
-            return [
-                (0, form_2.formSetErrors)(list.formId, data.errors || null),
-                {
-                    items: items,
-                    total: total,
-                    hasNextPage: hasNextPage,
-                    meta: data.meta || null,
-                    page: page,
-                    pageSize: pageSize,
-                    listId: listId,
-                    defaultPageValue: list.defaultPageValue,
-                    type: exports.LIST_AFTER_FETCH
-                },
-            ];
-        })["catch"](function (err) {
-            // Если это отмена через axios, просто игнорируем
-            _cancelSourceByList["delete"](listId);
-            // Если нужно — можно выставить ошибку формы или диспатчить событие
-            if (!axios_1["default"].isCancel(err)) {
-                // передаём ошибку дальше — если нужна стандартная обработка
-                throw err;
-            }
-            // при отмене просто возвращаем пустой массив действий
-            return [];
-        }));
+            })["finally"](function () {
+                // убрать сохранённый источник если он тот же
+                if (components.http._promises[listId] === source) {
+                    delete components.http._promises[listId];
+                }
+            }),
+        ]);
         return dispatch(toDispatch);
     };
 };

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@steroidsjs/core",
-    "version": "3.0.69",
+    "version": "3.0.71",
     "description": "",
     "author": "Vladimir Kozhin <hello@kozhindev.com>",
     "repository": {

package/ui/form/FieldList/FieldList.js

@@ -53,6 +53,9 @@ var range_1 = __importDefault(require("lodash-es/range"));
 var isEmpty_1 = __importDefault(require("lodash-es/isEmpty"));
 var concat_1 = __importDefault(require("lodash-es/concat"));
 var get_1 = __importDefault(require("lodash-es/get"));
+var omit_1 = __importDefault(require("lodash-es/omit"));
+var values_1 = __importDefault(require("lodash-es/values"));
+var zipObject_1 = __importDefault(require("lodash-es/zipObject"));
 var react_use_1 = require("react-use");
 var hooks_1 = require("../../../hooks");
 var Form_1 = require("../../form/Form/Form");
@@ -66,7 +69,7 @@ function FieldList(props) {
     var context = (0, react_1.useContext)(Form_1.FormContext);
     // Resolve model
     var modelAttributes = (_a = components.meta.getModel(props.model)) === null || _a === void 0 ? void 0 : _a.attributes;
-    var isWithReduxForm = (0, hooks_1.useSelector)(function (state) { return (0, get_1["default"])(state, ['form', context.formId]) || null; });
+    var reduxFormValues = (0, hooks_1.useSelector)(function (state) { return (0, get_1["default"])(state, ['form', context.formId]) || null; });
     var dispatch = context.provider.useDispatch();
     // Mapper for preserving the correct sequence of rows on the UI
     var _b = (0, react_1.useState)((0, range_1["default"])(props.input.value) || []), storeToRowIndexMap = _b[0], setStoreToRowIndexMap = _b[1];
@@ -86,9 +89,26 @@ function FieldList(props) {
         dispatch((0, form_1.formArrayAdd)(context.formId, props.input.name, rowsCount, props.initialValues));
     }, [addRowIndexes, context.formId, dispatch, props.initialValues, props.input.name]);
     var onRemove = (0, react_1.useCallback)(function (rowIndex) {
+        var _a;
         removeRowIndex(rowIndex);
-        dispatch((0, form_1.formArrayRemove)(context.formId, props.input.name, rowIndex));
-    }, [context.formId, dispatch, props.input.name, removeRowIndex]);
+        var actions = [
+            (0, form_1.formArrayRemove)(context.formId, props.input.name, rowIndex),
+        ];
+        var listErrors = (0, get_1["default"])(reduxFormValues === null || reduxFormValues === void 0 ? void 0 : reduxFormValues.errors, props.input.name);
+        if (!(0, isEmpty_1["default"])(listErrors)) {
+            /**
+             * После удаления строки, удаляет соответствующую этой строке ошибку
+             * и обновляет индексы для последующих элементов
+             */
+            var errorsWithoutRemoved = (0, omit_1["default"])(listErrors, String(rowIndex));
+            var errorsArray = (0, values_1["default"])(errorsWithoutRemoved);
+            var errorsForList = (0, zipObject_1["default"])((0, range_1["default"])(errorsArray.length), errorsArray);
+            var newTotalErrors = __assign(__assign({}, reduxFormValues.errors), (_a = {}, _a[props.input.name] = errorsForList, _a));
+            // @ts-ignore
+            actions.push((0, form_1.formSetErrors)(context.formId, newTotalErrors));
+        }
+        dispatch(actions);
+    }, [context.formId, dispatch, reduxFormValues === null || reduxFormValues === void 0 ? void 0 : reduxFormValues.errors, props.input.name, removeRowIndex]);
     (0, react_use_1.useMount)(function () {
         // Add initial rows
         if (!props.input.value) {
@@ -140,7 +160,7 @@ function FieldList(props) {
     var FieldListView = props.view || components.ui.getView('form.FieldListView');
     var FieldListItemView = props.itemView || components.ui.getView('form.FieldListItemView');
     return (react_1["default"].createElement(Form_1.FormContext.Provider, { value: contextValue },
-        react_1["default"].createElement(FieldListView, __assign({}, viewProps), !(0, isEmpty_1["default"])(storeToRowIndexMap) && (0, range_1["default"])(props.input.value || 0).map(function (index) { return (react_1["default"].createElement(FieldListItemView, __assign({}, itemViewProps, { key: isWithReduxForm ? storeToRowIndexMap[index] : index, prefix: props.input.name + '.' + index, rowIndex: index }))); }))));
+        react_1["default"].createElement(FieldListView, __assign({}, viewProps), !(0, isEmpty_1["default"])(storeToRowIndexMap) && (0, range_1["default"])(props.input.value || 0).map(function (index) { return (react_1["default"].createElement(FieldListItemView, __assign({}, itemViewProps, { key: reduxFormValues ? storeToRowIndexMap[index] : index, prefix: props.input.name + '.' + index, rowIndex: index }))); }))));
 }
 FieldList.defaultProps = {
     initialValues: null,