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

Uppercase implementation does not uppercase values

    Details

    • Type: Bug Fix
    • Status: Reopened
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: Backlog
    • Component/s: Development
    • Security Level: Public (Public: Anyone can view)
    • Labels:
    • Rice Module:
      KRAD
    • KRAD Feature Area:
      UIF Component
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required
    • Include in Release Notes?:
      Yes

      Description

      Currently when uppercaseValue is set to true on input field, the value is uppercased by using CSS. However, this only uppercases the value for display, it does not actually uppercase the value that is submitted. We need to change this or add functionality to uppercase the value as well

        Attachments

          Issue Links

            Activity

            Hide
            jkneal Jerry Neal (Inactive) added a comment -

            So that the value is uppercase for script calls (like ajax queries), we probably want to add script for the onchange event that will uppercase the value

            Show
            jkneal Jerry Neal (Inactive) added a comment - So that the value is uppercase for script calls (like ajax queries), we probably want to add script for the onchange event that will uppercase the value
            Hide
            sdm86 Steve Manning (Inactive) added a comment -

            Added functionality to krad.utility.js uppercaseValue function to actually convert the string to uppercase so that it will be persisted as an uppercase string

            Show
            sdm86 Steve Manning (Inactive) added a comment - Added functionality to krad.utility.js uppercaseValue function to actually convert the string to uppercase so that it will be persisted as an uppercase string
            Hide
            jkneal Jerry Neal (Inactive) added a comment -

            I think we should probably be checking this server side as well (see linked issue). When doing the binding we can read the uppercase information from the attribute definition and perform uppercase if needed.

            Show
            jkneal Jerry Neal (Inactive) added a comment - I think we should probably be checking this server side as well (see linked issue). When doing the binding we can read the uppercase information from the attribute definition and perform uppercase if needed.
            Hide
            cniesen Claus Niesen added a comment -

            I've noticed that uppercaseValue isn't set when forceUppercase in the data dictionaryis set.

            Show
            cniesen Claus Niesen added a comment - I've noticed that uppercaseValue isn't set when forceUppercase in the data dictionaryis set.
            Hide
            sdm86 Steve Manning (Inactive) added a comment -

            uppercaseValue works, the problem is still with the forceUpperCase on the attribute definition

            Show
            sdm86 Steve Manning (Inactive) added a comment - uppercaseValue works, the problem is still with the forceUpperCase on the attribute definition
            Hide
            jkneal Jerry Neal (Inactive) added a comment -

            This implementation is not quite what we need. The problem is the uppercasing is being done during the lifecycle, which happens after the controller executes. We need the data to be uppercased before the controller executes. We could do this in the binding (some place that checks for encryption, property editors.

            Show
            jkneal Jerry Neal (Inactive) added a comment - This implementation is not quite what we need. The problem is the uppercasing is being done during the lifecycle, which happens after the controller executes. We need the data to be uppercased before the controller executes. We could do this in the binding (some place that checks for encryption, property editors.

              People

              • Assignee:
                Unassigned
                Reporter:
                jkneal Jerry Neal (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated: