Skip to content

Commit 057a7d8

Browse files
Todor-adsilhan007
authored andcommitted
fix(ui5-view-settings-dialog): fix circular dependency JS error (#4844)
Fixes circular dependency JS error by removing self reference associatedItem property in initSortByItems getters FIXES: #4837
1 parent c09b0c4 commit 057a7d8

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

packages/fiori/src/ViewSettingsDialog.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -416,7 +416,6 @@ class ViewSettingsDialog extends UI5Element {
416416
return {
417417
text: item.text,
418418
selected: item.selected,
419-
associatedItem: item,
420419
index,
421420
};
422421
});
@@ -565,8 +564,7 @@ class ViewSettingsDialog extends UI5Element {
565564
sortDescending = !this._currentSettings.sortOrder[0].selected,
566565
sortBy = _currentSortBySelected && _currentSortBySelected.text,
567566
sortByElementIndex = _currentSortBySelected && _currentSortBySelected.index,
568-
initSortIByItem = this.initSortByItems.find((item, index) => index === sortByElementIndex),
569-
sortByItem = initSortIByItem && initSortIByItem.associatedItem;
567+
sortByItem = this.sortItems[sortByElementIndex];
570568
return {
571569
sortOrder,
572570
sortDescending,

0 commit comments

Comments
 (0)