@sumaris-net/ngx-components 1.23.2 → 1.23.5

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

@@ -26087,14 +26087,13 @@
                 delete: { get: function () { return _super_1.prototype.delete; } }
             });
             return __awaiter(this, void 0, void 0, function () {
-                var data, rowsById, rowNotDeleted, err_1;
+                var data, rowNotDeleted, err_1;
                 var _this = this;
                 return __generator(this, function (_b) {
                     switch (_b.label) {
                         case 0:
                             this.markAsLoading();
                             data = this.getDataFromRows(rows);
-                            rowsById = Beans.collectById(rows);
                             _b.label = 1;
                         case 1:
                             _b.trys.push([1, 3, 4, 5]);
@@ -26103,18 +26102,11 @@
                         case 2:
                             // Call service deletion
                             _b.sent();
-                            rowNotDeleted = Object.getOwnPropertyNames(rowsById)
-                                .reduce(function (res, id) {
-                                var row = rowsById[id];
-                                var present = _this.getRow(+id) === row;
-                                return present ? res.concat(row) : res;
-                            }, []);
-                            //this.getRows().filter(row => rows.includes(row));
-                            // Apply missing deletion
+                            rowNotDeleted = this.getRows().filter(rows.includes);
                             if (isNotEmptyArray(rowNotDeleted)) {
-                                console.warn("[table-datasource] Force deletion of " + rowNotDeleted.length + " rows (Is service update the cache ?)");
+                                console.warn("[table-datasource] Force deletion of " + rowNotDeleted.length + " rows! Please check that data service update the cache, after deletion");
                                 rowNotDeleted
-                                    // Start from the end
+                                    // Start at the end
                                     .sort(function (a, b) { return a.id > b.id ? -1 : 1; })
                                     .forEach(function (r) { return _super.delete.call(_this, r.id); });
                             }
@@ -26688,7 +26680,7 @@
                 return rxjs.of(undefined); // Continue
             }))
                 .subscribe(function (res) { return _this.updateView(res); }));
-            // Listen dataSource events
+            // Listen dataSource loading events
             if (this._dataSource)
                 this.listenDatasourceLoading(this._dataSource);
         };
@@ -28042,8 +28034,9 @@
                 if (this.debug)
                     console.debug('[table] Many call to listenDatasource(): Cleaning previous subscriptions...');
                 this._dataSourceLoadingSubscription.unsubscribe();
-                this._subscription.remove(this._dataSourceLoadingSubscription);
+                this.unregisterSubscription(this._dataSourceLoadingSubscription);
             }
+            // Propage loading to table
             this._dataSourceLoadingSubscription = this._dataSource.loadingSubject
                 .pipe(operators.distinctUntilChanged(), 
             // If changed to True: propagate as soon as possible
@@ -28051,7 +28044,7 @@
             // If changed to False: wait 250ms before propagate (to make sure the spinner has been displayed)
             operators.debounceTime(250), operators.tap(function (loading) { return !loading && _this.setLoading(false); }))
                 .subscribe();
-            this._subscription.add(this._dataSourceLoadingSubscription);
+            this.registerSubscription(this._dataSourceLoadingSubscription);
         };
         AppTable.prototype.startCellValueChanges = function (name, row) {
             var def = this._cellValueChangesDefs[name];