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

ActionList performing check that assumes group id is still a number and not a string

    Details

    • Type: Bug Fix
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0.1.1
    • Component/s: Development
    • Labels:
      None
    • Rice Module:
      KEW

      Description

      See the following email on the subject from the rice collaboration list:

      We have been converting our 0.9.3 rice implementation to 1.0.0, and in the process of testing, we discovered a defect in the ActionListNew.jsp.

      The new version of rice now supports groupId as a String rather than a Long. However, line 356 of the ActionListNew.jsp (<c:when test="$

      {result.groupId != null && result.groupId != 0}

      ">) results in a data conversion exception. The !=0 is the culprit. Changing this line to
      <c:when test="$

      {result.groupId != null && result.groupId != ''}

      "> corrects the problem.

      I scanned the rice 1.0.1 branch to see if this was already fixed. It seems the ActionListNew.jsp no longer exists, but the ActionList.jsp contains the same defect.

        Attachments

          Activity

          Hide
          chagstrom Chad Hagstrom added a comment -

          Fixed the problem by replacing "$

          {result.groupId != null && result.groupId != 0}

          " with "$

          {!empty result.groupId}

          " instead.

          Show
          chagstrom Chad Hagstrom added a comment - Fixed the problem by replacing "$ {result.groupId != null && result.groupId != 0} " with "$ {!empty result.groupId} " instead.

            People

            • Assignee:
              chagstrom Chad Hagstrom
              Reporter:
              ewestfal Eric Westfall
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: