@sumaris-net/ngx-components 1.5.4 → 1.5.8

package/bundles/sumaris-net.ngx-components.umd.js

@@ -523,49 +523,56 @@
             return { data: [], total: 0 };
         }
         // Filter items
-        var target = items;
+        var filteredItems = items;
         if (isNotNilOrBlank(value)) {
             // If wildcard, search using regexp
             if (value.indexOf('*') !== -1) {
                 value = value.replace('*', '.*');
-                target = target.filter(function (v) { return keys.findIndex(function (key) { return matchUpperCase(getPropertyByPathAsString(v, key), value); }) !== -1; });
+                filteredItems = filteredItems.filter(function (v) { return keys.findIndex(function (key) { return matchUpperCase(getPropertyByPathAsString(v, key), value); }) !== -1; });
             }
             // Else (no wildcard), search using startsWith
             else {
-                target = target.filter(function (v) { return keys.findIndex(function (key) { return startsWithUpperCase(getPropertyByPathAsString(v, key), value); }) !== -1; });
+                filteredItems = filteredItems.filter(function (v) { return keys.findIndex(function (key) { return startsWithUpperCase(getPropertyByPathAsString(v, key), value); }) !== -1; });
             }
         }
         // Exclude ids
         if (isNotEmptyArray(opts === null || opts === void 0 ? void 0 : opts.excludedIds)) {
-            target = target.filter(function (v) { return !opts.excludedIds.includes(+v['id']); });
-        }
-        // Sort (copy if need, before sorting)
-        if (target === items)
-            target = target.slice();
-        target.sort(propertiesPathComparator(keys));
-        var total = items.length || 0;
-        var fetchMore;
-        // Truncate if need
+            filteredItems = filteredItems.filter(function (v) { return !opts.excludedIds.includes(+v['id']); });
+        }
+        // Sort
+        if (!opts || opts.skipSort !== true) {
+            // Make to copy input items, before sorting (if not yet already done)
+            if (filteredItems === items)
+                filteredItems = filteredItems.slice();
+            // Sort by keys
+            filteredItems.sort(propertiesPathComparator(keys));
+        }
+        var total = filteredItems.length || 0;
+        // Pagination (truncate if need)
+        var pageData = filteredItems;
         if (opts) {
-            if (isNil(opts.offset) && opts.offset > 0) {
-                items = items.slice(opts.offset);
+            if (isNotNil(opts.offset) && opts.offset > 0) {
+                pageData = pageData.slice(opts.offset);
             }
-            if (isNotNil(opts.size) && items.length > opts.size) {
-                items = items.slice(0, opts.size);
+            if (isNotNil(opts.size) && pageData.length > opts.size) {
+                pageData = pageData.slice(0, opts.size);
             }
         }
-        var res = {
-            data: items,
+        var result = {
+            data: pageData,
             total: total
         };
         // Add fetch more capability, if total was fetched
-        var nextOffset = (opts && opts.offset || 0) + items.length;
+        var nextOffset = (opts && opts.offset || 0) + pageData.length;
         if (nextOffset < total) {
-            res.fetchMore = function (_) { return __awaiter(_this, void 0, void 0, function () { return __generator(this, function (_a) {
-                return [2 /*return*/, suggestFromArray(items, value, Object.assign({ offset: nextOffset }, opts))];
-            }); }); };
+            result.fetchMore = function (_) { return __awaiter(_this, void 0, void 0, function () {
+                return __generator(this, function (_a) {
+                    return [2 /*return*/, suggestFromArray(filteredItems, undefined /*OPTIMISATION: already applied*/, Object.assign(Object.assign({}, opts), { offset: nextOffset, skipSort: true, excludedIds: undefined // OPTIMISATION: already filtered
+                        }))];
+                });
+            }); };
         }
-        return res;
+        return result;
     }
     function suggestFromStringArray(values, value, options) {
         value = (typeof value === 'string' && value !== '*') && value.toUpperCase() || undefined;
@@ -1146,11 +1153,11 @@
             });
         });
     }
-    function waitForTrue(predicate, opts) {
+    function waitForTrue(observable, opts) {
         return __awaiter(this, void 0, void 0, function () {
             var firstTrueObservable, $timeout;
             return __generator(this, function (_a) {
-                firstTrueObservable = predicate.pipe(operators.first(function (v) { return v === true; }), operators.map(function () { }) // convert to void
+                firstTrueObservable = observable.pipe(operators.first(function (v) { return v === true; }), operators.map(function () { }) // convert to void
                 );
                 // Timeout (+ dueTime)
                 if (opts && opts.timeout > 0) {