]> git.evergreen-ils.org Git - Evergreen.git/commit
LP#1939338: fix IDL mistake that could prevent admin of carousels
authorGalen Charlton <gmc@equinoxOLI.org>
Fri, 11 Feb 2022 15:45:51 +0000 (10:45 -0500)
committerMike Rylander <mrylander@gmail.com>
Thu, 24 Mar 2022 19:27:19 +0000 (15:27 -0400)
commitb0f56f96df65fd28d8ec47491a413967145b8476
treeb7687d1df4bbff839376133cb43dd86f2e619108
parent853485a16b6b2ade8923c4d403292b942c231420
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>
Signed-off-by: Andrea Buntz Neiman <abneiman@equinoxOLI.org>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Open-ILS/examples/fm_IDL.xml