Course archiving and deleting functions should use open-ils.courses.detach_material
Bug #1940105 reported by
Jane Sandberg
This bug affects 4 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Evergreen |
Fix Released
|
Undecided
|
Unassigned | ||
3.8 |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
Thanks to Galen for pointing these out.
Currently, there are two ways of removing a particular material from a course:
* the open-ils.
* the resetItemFields function in the Angular course service, used when archiving or deleting an entire course
To keep these from growing further and further out of sync, we should have the course archive and delete functions use open-ils.
One benefit of doing so is that archiving and deleting courses could benefit from bug 1939730 as soon as it is merged.
Changed in evergreen: | |
assignee: | nobody → Kyle Huckins (khuckins) |
Changed in evergreen: | |
milestone: | none → 3.8.1 |
Changed in evergreen: | |
milestone: | 3.8.1 → none |
Changed in evergreen: | |
assignee: | nobody → Jane Sandberg (sandbergja) |
Changed in evergreen: | |
assignee: | Jane Sandberg (sandbergja) → nobody |
no longer affects: | evergreen/3.7 |
Changed in evergreen: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
If a course that is selected for deletion still has items associated with it, I think it would be preferable to provide an alert to the user and to require confirmation of this action. Otherwise, it is too easy to delete a course and reset the items inadvertently. I would expect to be able to archive a course with items associated with it but not to be able to delete it.