Skip to content

Commit 98daffc

Browse files
committed
[sc-12720] Resolve SonarCloud issue: Use for…of instead of .forEach(…)
1 parent a474698 commit 98daffc

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed

src/apifetch.ts

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -213,15 +213,13 @@ const executeApiFetch: ExecuteApiFetch = function (
213213

214214
// Add sortBy and sortOrder
215215
if (Array.isArray(settings?.paging.sortBy) && settings?.paging.sortBy.length > 1) {
216-
settings?.paging.sortBy.forEach(function (value, index) {
217-
queryParamsString =
218-
queryParamsString +
216+
for (const [index, value] of settings.paging.sortBy.entries()) {
217+
queryParamsString +=
219218
settingToQueryParam(value, 'sort') +
220-
settingToQueryParam(settings?.paging.sortOrder[index], 'order');
221-
});
219+
settingToQueryParam(settings.paging.sortOrder[index], 'order');
220+
}
222221
} else {
223-
queryParamsString =
224-
queryParamsString +
222+
queryParamsString +=
225223
settingToQueryParam(settings?.paging.sortBy, 'sort') +
226224
settingToQueryParam(settings?.paging.sortOrder, 'order');
227225
}

0 commit comments

Comments
 (0)