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

CharsetPatternConstraint does not always validate strings of characters correctly

    Details

    • Type: Bug Fix
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.4
    • Fix Version/s: 2.4
    • Component/s: Development
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Rice Module:
      KRAD
    • Sprint:
      2.4.0-rc1 Sprint 8
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required
    • Code Review Status:
      Not Required
    • Include in Release Notes?:
      Yes

      Description

      in krad component library, validation → CharsetPattern Constraint example, the valid chars string is p:validChars="AB#0-9" and the text is "Only A, B pound characters, and numbers between 0 and 9 allowed". Instead of allowing numbers from 0 - 9, it is allowing only 0, - , and 9.

      As a side note, there is a missing comma in the desc between B and pound characters.

        Attachments

          Issue Links

            Activity

            Hide
            cniesen Claus Niesen added a comment -
            p:validChars="AB#0-9"

            This looks way to much like a range. I suggest changing the order of where the dash is shown.

            Show
            cniesen Claus Niesen added a comment - p:validChars= "AB#0-9" This looks way to much like a range. I suggest changing the order of where the dash is shown.
            Hide
            shahess Shannon Hess added a comment -

            Claus, Is the text incorrect then? It says it should allow numbers between 0 and 9. Should it actually say "Only A, B, pound characters, dash, 0 and 9 allowed"?

            Show
            shahess Shannon Hess added a comment - Claus, Is the text incorrect then? It says it should allow numbers between 0 and 9. Should it actually say "Only A, B, pound characters, dash, 0 and 9 allowed"?
            Hide
            cniesen Claus Niesen added a comment -

            Yes, it should say "Only A, B, pound characters, dash, 0 and 9 allowed".

            Show
            cniesen Claus Niesen added a comment - Yes, it should say "Only A, B, pound characters, dash, 0 and 9 allowed".

              People

              • Assignee:
                jruch Jeff Ruch
                Reporter:
                shahess Shannon Hess
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - 15 minutes
                  15m
                  Remaining:
                  Remaining Estimate - 15 minutes
                  15m
                  Logged:
                  Time Spent - Not Specified
                  Not Specified