Uploaded image for project: 'Kuali Rice Development'
  1. Kuali Rice Development
  2. KULRICE-2451

Bad query generated when invoking KimRoleDaoOjb.getDelegationImplMapFromRoleIds

    Details

    • Type: Bug Fix
    • Status: Closed
    • Priority: Critical
    • Resolution: Complete
    • Affects Version/s: None
    • Fix Version/s: 1.0, KFS Release 3.0
    • Component/s: Development
    • Labels:
      None
    • Rice Module:
      KIM

      Description

      In the case that the list of roleIds that are passed in are empty, the following sql is generated:

      SELECT A0.DLGN_ID,A0.ROLE_ID,A0.ACTV_IND,A0.KIM_TYP_ID,A0.DLGN_TYP_CD,A0.VER_NBR,A0.OBJ_ID FROM KRIM_DLGN_T A0 WHERE (A0.ROLE_ID IN ()) AND A0.ACTV_IND = ?

      This causes a confusing error from MySQL which is hard to figure out (makes it look like an issue with ACTV_IND instead).

      Not sure if this is something that needs to be handled but wanted to be sure I recorded it in case it does.

        Attachments

          Issue Links

            Activity

            Hide
            ewestfal Eric Westfall added a comment -

            Ailish, if this is something which needs to be resolved, feel free to re-assign as needed.

            Show
            ewestfal Eric Westfall added a comment - Ailish, if this is something which needs to be resolved, feel free to re-assign as needed.
            Hide
            jkeller Jonathan Keller added a comment -

            I'm updating the role service to skip the call to the method if the role list is blank. This method is only called from 2 places and one already checks the list before passing it in.

            Show
            jkeller Jonathan Keller added a comment - I'm updating the role service to skip the call to the method if the role list is blank. This method is only called from 2 places and one already checks the list before passing it in.
            Hide
            jkeller Jonathan Keller added a comment -

            This is now committed to Rice. (revision 5515) It will get pulled into the KFS KIM branch with any subsequent changes.

            Show
            jkeller Jonathan Keller added a comment - This is now committed to Rice. (revision 5515) It will get pulled into the KFS KIM branch with any subsequent changes.
            Hide
            ewestfal Eric Westfall added a comment -

            Bulk change of all Rice 1.0 issues to closed after public release.

            Show
            ewestfal Eric Westfall added a comment - Bulk change of all Rice 1.0 issues to closed after public release.

              People

              • Assignee:
                jkeller Jonathan Keller
                Reporter:
                ewestfal Eric Westfall
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: