Skip to content

Commit a209a82

Browse files
authored
fix: on reset remove unnecessary fileList array (#2130)
## Description Removed unnecessary fileList array and directly assigned an empty array to fileObj.files. ## Motivation Reset needs to reset all files, thus no reason to pass the file list ## Tests included - [ ] Included for each change/fix? - [ ] Passing? (Merge will not be approved unless this is checked) ## Documentation - [ ] swagger documentation updated \[required\] - [ ] official documentation updated \[nice-to-have\] ### official documentation info If you have updated the official documentation, please provide PR # and URL of the pages where the updates are included ## Backend version - [ ] Does it require a specific version of the backend - which version of the backend is required: ## Summary by Sourcery Bug Fixes: - Ensure reset jobs no longer include previous datablock archive IDs by clearing the files list during reset.
2 parents 18c157f + ebeaae4 commit a209a82

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

src/app/datasets/admin-tab/admin-tab.component.ts

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -52,14 +52,7 @@ export class AdminTabComponent implements OnInit, OnDestroy {
5252
pid: "",
5353
files: [],
5454
};
55-
const fileList: string[] = [];
5655
fileObj.pid = this.dataset["pid"];
57-
if (this.dataset["datablocks"]) {
58-
this.dataset["datablocks"].map((d) => {
59-
fileList.push(d["archiveId"]);
60-
});
61-
}
62-
fileObj.files = fileList;
6356
const job: CreateJobDtoV3 = {
6457
emailJobInitiator: user.email,
6558
type: "reset",

0 commit comments

Comments
 (0)