thomas/metube
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #876 from alexta69/copilot/fix-clear-completed-error

Browse Source 
Fix undefined access error when bulk deleting downloads
This commit is contained in:
Alex
2026-01-10 18:24:02 +02:00
committed by GitHub
parent 9be0781c7f 846c4f0e52
commit 3f1b89e04a
3 changed files with 11 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
.gitignore vendored
View File

@@ -7,6 +7,7 @@
# dependencies # dependencies
/ui/node_modules /ui/node_modules
/ui/package-lock.json
# profiling files # profiling files
chrome-profiler-events*.json chrome-profiler-events*.json

3
ui/package.json
View File

@@ -38,6 +38,7 @@
"@fortawesome/free-solid-svg-icons": "^7.1.0", "@fortawesome/free-solid-svg-icons": "^7.1.0",
"@ng-bootstrap/ng-bootstrap": "^20.0.0", "@ng-bootstrap/ng-bootstrap": "^20.0.0",
"@ng-select/ng-select": "^21.1.0", "@ng-select/ng-select": "^21.1.0",
"@popperjs/core": "^2.11.8",
"bootstrap": "^5.3.6", "bootstrap": "^5.3.6",
"ngx-cookie-service": "^21.1.0", "ngx-cookie-service": "^21.1.0",
"ngx-socket-io": "~4.9.3", "ngx-socket-io": "~4.9.3",
@@ -59,4 +60,4 @@
"typescript-eslint": "8.47.0", "typescript-eslint": "8.47.0",
"vitest": "^4.0.8" "vitest": "^4.0.8"
} }
} }

13
ui/src/app/services/downloads.service.ts
View File

@@ -118,12 +118,15 @@ export class DownloadsService {
} }
public delById(where: State, ids: string[]) { public delById(where: State, ids: string[]) {
ids.forEach(id => { const map = this[where];
const obj = this[where].get(id) if (map) {
if (obj) { for (const id of ids) {
obj.deleting = true const obj = map.get(id);
if (obj) {
obj.deleting = true;
}
} }
}); }
return this.http.post('delete', {where: where, ids: ids}); return this.http.post('delete', {where: where, ids: ids});
} }