Table: improve tile mode

There may not always be a tile for a row. Therefore, implement
Table.createTiles more lenient in order to avoid null pointer
exceptions. In addition, the TableTileGridMediator ensures that null
values removed from its list of tiles.
The list of tiles of the TableTileGridMediator needs to be unique as the
tiles will be rendered one after another. If there are duplicates in the
list of tiles an 'Already rendered'-error will be thrown if the second
occurrence is rendered. Therefore, the TableTileGridMediator removes all
duplicates from its list of tiles.

379008

Author: fschinkel

eclipse-scout-core/src/table/Table.ts

@@ -5209,8 +5209,10 @@ export class Table extends Widget implements TableModel, Filterable<TableRow> {
   createTiles(rows: TableRow[]): Tile[] {
     return rows.map(row => {
       let tile = this.createTileForRow(row);
-      this._adaptTile(tile);
-      tile.rowId = row.id;
+      if (tile) {
+        this._adaptTile(tile);
+        tile.rowId = row.id;
+      }
       return tile;
     });
   }

eclipse-scout-core/src/table/TableTileGridMediator.ts

@@ -190,6 +190,8 @@ export class TableTileGridMediator extends Widget implements TableTileGridMediat
   }
 
   setTiles(tiles: ObjectOrChildModel<Tile>[]) {
+    tiles = arrays.ensure(tiles)
+      .filter((value, index, array) => value && array.indexOf(value) === index); // remove duplicates
     this.setProperty('tiles', tiles);
   }
 
@@ -435,7 +437,7 @@ export class TableTileGridMediator extends Widget implements TableTileGridMediat
     this.table.loadingSupport.renderLoading(true);
   }
 
-  override destroy() {
+  protected override _destroy() {
     // destroy tiles manually since owner is the mediator thus the tileGrid can't destroy them
     this.tiles.forEach(tile => tile.destroy());
     this.tileAccordion.destroy();
@@ -528,7 +530,8 @@ export class TableTileGridMediator extends Widget implements TableTileGridMediat
     if (!this.table.tileMode || $.isEmptyObject(this.tilesMap)) {
       return;
     }
-    this.tiles = this.table.rows.map(row => this.tilesMap[row.id]);
+    this.tiles = this.table.rows.map(row => this.tilesMap[row.id])
+      .filter((value, index, array) => value && array.indexOf(value) === index); // remove duplicates
     this.tileAccordion.setTiles(this.tiles);
   }