LP#1939338: fix IDL mistake that could prevent admin of carousels user/gmcharlt/lp1939338_fix_perm_checks
authorGalen Charlton <gmc@equinoxOLI.org>
Fri, 11 Feb 2022 15:45:51 +0000 (10:45 -0500)
committerGalen Charlton <gmc@equinoxOLI.org>
Fri, 11 Feb 2022 15:50:15 +0000 (10:50 -0500)
commit8ee1ca25a282f51593afae0f60d7df48259f507f
tree93e00f1dd02e28a35089f267f4c4b3df73718655
parent9582ea41ef57dbc6068a5c9eb6a4c9400f8c3c89
LP#1939338: fix IDL mistake that could prevent admin of carousels

This patch fixes a typo found in the IDL definition of the
classes for bib, call number, item, and user buckets. One effect
of the typo is that the admin interface for carousels could not
flesh out the record buckets unless the staff user was either
a global administrator or was the one who had created the carousel
and bucket originally.

This patch likely fixes other cases where one would expect to be
able to access buckets created by another user.

This patch is effectively a subset of a patch in the Simple Reporter
branch (bug 1945836 and working repository commit 36369719067) that
addresses several more errors in the IDL. However, I'm submitting this one
now in the hopes of resolving a long-standing bug with carousels sooner
rather than later.

To test
-------
[1] As a local admin, create a carousel definition.
[2] As a different local admin at the same OU, open the carousels
    admin interface. Note that attempting to retrieve carousel defs
    created in step 1 will result in a console error and the interface
    not being completely rendered.
[3] Apply the patch and repeat step 2. This time, the carousel definitions
    should all display.

Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>
Open-ILS/examples/fm_IDL.xml