Skip to content

Commit 9dc8d84

Browse files
authored
Merge pull request #1509 from rocket-admin/backend_table_setting_ordering_cleaning
Refactor buildNewPersonalTableSettingsEntity function to handle undefined values instead of null
2 parents f6f553c + b92ae77 commit 9dc8d84

1 file changed

Lines changed: 10 additions & 10 deletions

File tree

backend/src/entities/table-settings/personal-table-settings/utils/build-new-personal-table-settings-entity.util.ts

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -2,15 +2,15 @@ import { PersonalTableSettingsData } from '../data-structures/create-personal-ta
22
import { PersonalTableSettingsEntity } from '../personal-table-settings.entity.js';
33

44
export function buildNewPersonalTableSettingsEntity(
5-
personalSettingsData: PersonalTableSettingsData,
5+
personalSettingsData: PersonalTableSettingsData,
66
): PersonalTableSettingsEntity {
7-
const newEntity = new PersonalTableSettingsEntity();
8-
Object.assign(newEntity, personalSettingsData);
9-
Object.keys(personalSettingsData).forEach((key) => {
10-
if (personalSettingsData[key as keyof PersonalTableSettingsData] === null) {
11-
// eslint-disable-next-line security/detect-object-injection
12-
delete (newEntity as any)[key];
13-
}
14-
});
15-
return newEntity;
7+
const newEntity = new PersonalTableSettingsEntity();
8+
Object.assign(newEntity, personalSettingsData);
9+
Object.keys(personalSettingsData).forEach((key) => {
10+
if (personalSettingsData[key as keyof PersonalTableSettingsData] === undefined) {
11+
// eslint-disable-next-line security/detect-object-injection
12+
delete (newEntity as any)[key];
13+
}
14+
});
15+
return newEntity;
1616
}

0 commit comments

Comments
 (0)