From d3c77b5422a8979bba95af27b96dfdca02b60086 Mon Sep 17 00:00:00 2001 From: Jane Sandberg Date: Sat, 22 Oct 2022 15:43:10 -0700 Subject: [PATCH] LP1993922: Course material delete should not delete other courses' materials To test: * Apply this patch * Login to BR1 workstation * Go to Admin - Local Admin - Course Reserves List * Create two courses, Course 1 and Course 2 * Associate materials to both courses * Archive Course 1 * Go to Course materials on Course 2 and note that it still has all of its materials attached. Signed-off-by: Jane Sandberg Signed-off-by: Beth Willis Signed-off-by: Michele Morgan --- Open-ILS/src/eg2/src/app/staff/share/course.service.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Open-ILS/src/eg2/src/app/staff/share/course.service.ts b/Open-ILS/src/eg2/src/app/staff/share/course.service.ts index 12a030382d..df1afc43ed 100644 --- a/Open-ILS/src/eg2/src/app/staff/share/course.service.ts +++ b/Open-ILS/src/eg2/src/app/staff/share/course.service.ts @@ -171,7 +171,7 @@ export class CourseService { course_library_hash[course.id()] = course.owning_lib(); }); - this.pcrud.retrieveAll('acmcm', {course: course_ids}).subscribe(material => { + this.pcrud.search('acmcm', {course: course_ids}).subscribe(material => { deleteRequest$.push(this.net.request( 'open-ils.courses', 'open-ils.courses.detach_material', this.auth.token(), material.id())); -- 2.11.0