]> 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:28:41 +0000 (15:28 -0400)
commit3ae18863abe7ac7490e599df24d7c9f2e78b5a4e
treeca81cc81dc092f067e29087dc2c9f91f05e73257
parent385c7af5f4f9f46f2beed8a1b60912750ca8517f
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