Kuali Rice Development
  1. Kuali Rice Development
  2. KULRICE-6687

Person lookup isn't case-insensitive in Rice 2.0

    Details

    • Type: Bug Fix Bug Fix
    • Status: Closed Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 2.0, 2.1
    • Fix Version/s: 2.0.2
    • Component/s: Development
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Similar issues:
      KULRICE-5249Case Insensitive flag ignored in KNS
      KULRICE-6817Document search by document type doesn't allow wildcards or case insensitive searches
      KULRICE-6556Rice Dev: Principal ID search on Person lookup is case sensitive
      KULRICE-11712Suggest queries should be case insensitive
      KULRICE-4165Search on document description should be case insensitive
      KULRICE-85implement case-insensitive searching
      KULRICE-7060KIM Person Maintenance screen allows a new user to be created with mixed-case or uppercase letters in Principal Name field
      KULRICE-7624Results from person lookup should be bounded by default like it is in Rice 1.x
      KULRICE-5694Under some circumstances lookups can return the same row multiple times
      KULRICE-2407finish Person Lookup and Inquiry
    • Rice Module:
      KIM
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required

      Description

      We are seeing a couple of things w.r.t Person Lookup:

      1. Lookup using principalName - principalName fieldValue is always converted to lowercase before the actual search (which is case-sensitive always). So, this lookup only works for database values that are in lowercase. If we have principal names with mixed case or uppercase, the search fails.

      2. Lookup using other fields - These are case-sensitive now.

      When I looked at the code, the difference I see is that the actual search happens thru CriteriaLookupDao which does not have the case-insensitive logic like LookupDao does.

        Issue Links

          Activity

          Hide
          Matt Sargent added a comment -

          Chitra, which version of rice 2.0 are you using?

          -Matt

          Show
          Matt Sargent added a comment - Chitra, which version of rice 2.0 are you using? -Matt
          Hide
          Chitra Chandran added a comment -

          Hi Matt, I was testing using RC1 release

          Show
          Chitra Chandran added a comment - Hi Matt, I was testing using RC1 release
          Show
          Jeremy Hanson added a comment - https://jira.kuali.org/browse/KULRICE-6556
          Hide
          Jeremy Hanson added a comment -

          I thought we started enforcing lower case principal names in 1.0.x?

          Show
          Jeremy Hanson added a comment - I thought we started enforcing lower case principal names in 1.0.x?
          Hide
          Jeremy Hanson added a comment -

          so questions.

          KC has principalNames that are mixed case? We have enforced lower case since 1.0.1. I believe our documentation says that you need to override the identity service to implement it with mixed-case. Are you doing this?

          If you add wildcards to the search, it won't be case-sensitive. Is that a problem, or is there a reason case-insensitivity is needed?

          Show
          Jeremy Hanson added a comment - so questions. KC has principalNames that are mixed case? We have enforced lower case since 1.0.1. I believe our documentation says that you need to override the identity service to implement it with mixed-case. Are you doing this? If you add wildcards to the search, it won't be case-sensitive. Is that a problem, or is there a reason case-insensitivity is needed?
          Hide
          Jessica Coltrin (Inactive) added a comment -

          This seems to not be a bug, so I'm resolving. Please reopen if there is a need.

          Show
          Jessica Coltrin (Inactive) added a comment - This seems to not be a bug, so I'm resolving. Please reopen if there is a need.
          Hide
          Jessica Coltrin (Inactive) added a comment -

          Closing since these items are now in the release notes.

          Show
          Jessica Coltrin (Inactive) added a comment - Closing since these items are now in the release notes.
          Hide
          Ken Geis added a comment -

          I don't think this should have been closed. Case-sensitive search on person names (first and last, not principal) make search difficult. This is a real implementation problem for us since usability-wise, we've found search to be a weak point in Rice. I would like for my users to be able to search people in a case-insensitive way without needing to use wildcards.

          Show
          Ken Geis added a comment - I don't think this should have been closed. Case-sensitive search on person names (first and last, not principal) make search difficult. This is a real implementation problem for us since usability-wise, we've found search to be a weak point in Rice. I would like for my users to be able to search people in a case-insensitive way without needing to use wildcards.
          Hide
          Matt Sargent added a comment -

          Eric was going to bring this up to the KTI, but after some research here's what we've found...

          In 1.0.3.3 the name fields (first, middle, last) are case insensitive. So if you search first name for Eric, ERIC, eric, erIC, etc. you get the same results. In 2.0 you only get the expected result if you search for Eric.

          So there is a bug that needs fixed on those name fields.

          Show
          Matt Sargent added a comment - Eric was going to bring this up to the KTI, but after some research here's what we've found... In 1.0.3.3 the name fields (first, middle, last) are case insensitive. So if you search first name for Eric, ERIC, eric, erIC, etc. you get the same results. In 2.0 you only get the expected result if you search for Eric. So there is a bug that needs fixed on those name fields.
          Hide
          Eric Westfall added a comment -

          And in terms of Jeremy's earlier comment. This is in regards to first name, last name, etc. not the principal name.

          Show
          Eric Westfall added a comment - And in terms of Jeremy's earlier comment. This is in regards to first name, last name, etc. not the principal name.
          Hide
          Kristina Taylor (Inactive) added a comment -

          This is now working for KC as of our latest Rice upgrade.

          Show
          Kristina Taylor (Inactive) added a comment - This is now working for KC as of our latest Rice upgrade.

            People

            • Assignee:
              Jeremy Hanson
              Reporter:
              Chitra Chandran
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Structure Helper Panel