Skip to content

Commit 8c274d7

Browse files
ISSUE-141254: list-view.component - extend logging, do not allow empty or null ids for multiselect mode
1 parent a02b784 commit 8c274d7

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

scripts/dxcomponents/components/containers/templates/listview/list-view.component.js

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,10 @@ export class ListViewComponent extends BaseComponent {
113113
// By default, pyGUID is used for Data classes and pyID is for Work classes as row-id/key
114114
const defaultRowID = this.configProps$?.referenceType === "Case" ? "pyID" : "pyGUID";
115115
this.compositeKeys = this.configProps$?.compositeKeys ?? this.payload.compositeKeys;
116+
console.debug(TAG, `Using compositeKeys: ${JSON.stringify(this.compositeKeys)}`);
117+
116118
this.rowID = this.compositeKeys && this.compositeKeys?.length === 1 ? this.compositeKeys[0] : defaultRowID;
119+
console.debug(TAG, `Using rowID: ${this.rowID}`);
117120

118121
this.componentValue = this.configProps$?.value;
119122
this.contextPage = this.configProps$?.contextPage;
@@ -186,7 +189,11 @@ export class ListViewComponent extends BaseComponent {
186189
}
187190

188191
#updateSelectedItemsMulti() {
189-
const readonlyIds = new Set(this.configProps$.readonlyContextList.map(element => element[this.rowID]));
192+
const readonlyIds = new Set(
193+
this.configProps$.readonlyContextList
194+
.map(element => element[this.rowID])
195+
.filter(id => id !== "" && id != null)
196+
);
190197
this.listViewItems.forEach((item) => {
191198
item.selected = readonlyIds.has(item[this.rowID]);
192199
});

0 commit comments

Comments
 (0)