Comment 12 for bug 1929588

Revision history for this message
Galen Charlton (gmc) wrote :

Tested and removing signoff:

[1] If more than one course is selected for duplication, the new courses take their title only from the last selected course. Either this behavior should be corrected or the action should be permitted only when exactly one course is selected.
[2] There's some ng lint noise.

Regarding the original discussion, I note that ALEPH does include an option to copy over the reading list items when you duplicate a course. I think adding an option to copy over materials would make the feature more useful, but would entail some changes to open-ils.courses.detach_material and friends to handle the case where an item is associated with more than one course.

I would be fine merging a don't-copy-over-materials branch, by the way, once #1 and #2 are addressed.