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

                cloned from

                Bug Fix - A problem which impairs or prevents the functions of the product. KULRICE-8803 Inquiry bases logic on readonly for building inquiry or direct inquiry based on the readonly flag of the Inquiry widget, not the field it applies to

                • Critical - Crashes, loss of data, severe memory leak.
                • Closed
                is related to

                Task - A task that needs to be done. KULRICE-9226 AttributeDefinition forceUpperCase Doesn't Work

                • Minor - Minor loss of function, or other problem where easy workaround is present.
                • Closed

                  Activity

                  Ascending order - Click to sort in descending order
                  Hide
                  Permalink
                  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
                  Permalink
                  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
                  Permalink
                  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
                  Permalink
                  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
                  Permalink
                  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
                  Permalink
                  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: