Course List should be limited to user's working location

[Beth Willis]

In Course Materials, users can add and manage courses on the Course List page.  Currently, the Course List includes course created by users from other working locations.  It does not appear that users can add materials to courses not owned by their working location, but they are able to remove items associated with courses not owned by their working location.

Staff users should be able to access only courses related to their own working locations.

EG 3-6-0

[Jane Sandberg (sandbergja)]

I will be working on this on behalf of BC Libraries Cooperative. My plan is to add an org family filter (with the Ancestors/Descendants checkboxes) to the course list. It will default to the workstation org unit, but still allow, say, somebody at a main campus to view courses at a branch library. It will only show orgs where the user has the MANAGE_RESERVES permission, however.

[Jane Sandberg (sandbergja)]

Here is a branch: user/sandbergja/lp1905068_add_org_family_selector_to_course_list

Here is a link: https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/sandbergja/lp1905068_add_org_family_selector_to_course_list

And here are the testing notes from the commit message:

1) Apply this patch
2) Admin > Local admin > Course reserves list
3) Check that the list defaults to courses available at your workstation org unit
4) Check that, when you select ancestors, descendants, or a different org unit, the course list updates accordingly
5) Check that the list only displays org units for which you have the MANAGE_RESERVES permission

[Christine Burns (christine-burns)]

I have tested this code and consent to signing off on it with my name, Christine Burns and my email address, <email address hidden>

[Galen Charlton (gmc)]

There's a problem with this line:

  defaultOuId = 1;

There's no guarantee that the root of the OU tree has ID = 1, and I know of at least one production Evergreen system where that's not the case. Importing OrgService and getting the default from org.root() would resolve this.

[Jane Sandberg (sandbergja)]

Thanks, Galen. I made the correction you recommended, amended my commit, and force-pushed it to the same branch.

[Galen Charlton (gmc)]

Pushed on down to rel_3_6. Thanks, Jane and Christine. (Also, apologies, Christine, I forgot to add your signoff to the commit).