Permissions Architecture: Courses error when trying to delete location in BugFest

Ah, okay. Thanks for clarifying .

the consensus was that it's better to make it possible to break Course Reserves than to require it to be a part of the FOLIO Core

In that case, I will mark this closed > done

the behaviour you are describing is precisely what happens when no check is made for locations in use by Course Reserves. It's the bug I fixed in https://github.com/folio-org/ui-tenant-settings/commit/3f3727a9cd86ac64a3fe498a9e7da9ab0e9b5d40

But making this test causes the side-effects described in the present issue, and the consensus was that it's better to make it possible to break Course Reserves than to require it to be a part of the FOLIO Core.

, I tested this in snapshot and I no longer get an error when deleting a location as a limited-permission user. However, I did notice that, when the location has been selected in the Courses app and you try to delete it, it just deletes. Is that intended? For other types of records (e.g. Inventory records), FOLIO won't let you delete locations that are "in use". This is the popup you get if you try to delete such a location:

Maybe it's okay if a location that's in use in Courses is deleted. I mean, Location doesn't seem to be a required field in the Courses record. I did notice, though, that after I deleted the in-use location in Courses, it left behind a weird UUID reference in the Courses UI:

Also, I tested this same workflow in Honeysuckle BugFest and, in that environment, FOLIO did prevent deletion of the location when in use in Courses:

So this is a change in behavior. I just wanted to make sure everyone was aware and okay with that.

Steps to repro:

1. Go to Locations and add a new location

2. Go to Courses and create a new course and select new location as the Location

3. Go back to Locations and delete the new location

Outcome in Honeysuckle: FOLIO disallows deletion

Outcome in Snapshot: FOLIO allows deletion and the Courses record is left with a reference to a UUID

FYI (Courses PO) and (Locations PO)

I reserve the right be unhelpfully grumpy

This should be part of the programmers bill of rights.

Yes; but I reserve the right be unhelpfully grumpy about it.