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

Travel Account Maintenance (New) account number validation broken and validation message nulled

    Details

    • Type: Bug Fix Bug Fix
    • Status: Closed Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 2.1.1
    • Fix Version/s: 2.1.3
    • Component/s: Development
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Similar issues:
      KULRICE-7191No validation on account numbers of Travel Maint document
      KULRICE-8544Travel Account Maintenance (New) Document: Account Number issues
      KULRICE-12116Fill AFT Per-Screen Item Gap: KRAD Demo: Travel Account Maintenance (New)
      KULRICE-12713 Validation exception on Editing a Travel Account
      KULRICE-5477Rice Dev: KRAD Travel Account Maintenance (new) error
      KULRICE-8545Travel Account Maintenance (New) Document: Fiscal Officer Accounts Section Issue
      KULRICE-3356"Travel Account" field on Travel Request creation page is not working properly
      KULRICE-6855Travel Account Maintenance document routes to exception
      KULRICE-11548KRAD Sampleapp Demo Travel Account Maintenance New Travel Account Type Code lookup doesn't select checkbox on value return
      KULRICE-13768Create AFT for Travel Account Bulk Update
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required

      Description

      Log into the sampleapp as admin
      Click on the KRAD tab
      Click on any Travel Account Maintenance (New) link
      In the Travel Account Number field type 1234567890
      Expecting 1234567890 to be a valid 10 digits.
      Also notice the error message "validation.null"

        Issue Links

          Activity

          Hide
          Erik Meade added a comment -

          Same behavior with the Fiscal Officer Accounts section's Travel Account Number too.

          Show
          Erik Meade added a comment - Same behavior with the Fiscal Officer Accounts section's Travel Account Number too.
          Hide
          Erik Meade added a comment -

          And the Note Text validation (typed in text Test note) in the Notes and Attachments section.

          Show
          Erik Meade added a comment - And the Note Text validation (typed in text Test note) in the Notes and Attachments section.
          Hide
          Brian Smith (Inactive) added a comment -

          These are the same issue, Martin will be rolling back the problematic code and the issue should go away.

          Show
          Brian Smith (Inactive) added a comment - These are the same issue, Martin will be rolling back the problematic code and the issue should go away.
          Hide
          Erik Meade added a comment -

          Smoke Tests edu.samplu.krad.travelview.MaintenanceAddDeleteFiscalOfficerIT.testVerifyAddDeleteFiscalOfficer
          edu.samplu.krad.travelview.MaintenanceAddDeleteNoteIT.testVerifyAddDeleteNote
          edu.samplu.krad.travelview.MaintenanceSaveIT.testVerifySave

          Show
          Erik Meade added a comment - Smoke Tests edu.samplu.krad.travelview.MaintenanceAddDeleteFiscalOfficerIT.testVerifyAddDeleteFiscalOfficer edu.samplu.krad.travelview.MaintenanceAddDeleteNoteIT.testVerifyAddDeleteNote edu.samplu.krad.travelview.MaintenanceSaveIT.testVerifySave
          Hide
          Peter Giles (Inactive) added a comment -

          Martin, can you verify that this issue is fixed in 2.1.2 and then resolve? Thanks

          Show
          Peter Giles (Inactive) added a comment - Martin, can you verify that this issue is fixed in 2.1.2 and then resolve? Thanks
          Hide
          Martin Taylor (Inactive) added a comment -

          Properly handling the required check but not validating for 10 digits.

          Show
          Martin Taylor (Inactive) added a comment - Properly handling the required check but not validating for 10 digits.
          Hide
          Martin Taylor (Inactive) added a comment - - edited

          TravelAccount.xml refers to constraintText 'must be 10 digits' but isn't clear on any validation besides it being required. In that regard it is working properly and the validation errors are being displayed properly (Required). Believe the validationPattern should be removed from TravelAccount.xml and replaced with validCharacterConstraint -> numericPatternConstraint.

          Show
          Martin Taylor (Inactive) added a comment - - edited TravelAccount.xml refers to constraintText 'must be 10 digits' but isn't clear on any validation besides it being required. In that regard it is working properly and the validation errors are being displayed properly (Required). Believe the validationPattern should be removed from TravelAccount.xml and replaced with validCharacterConstraint -> numericPatternConstraint.
          Hide
          Martin Taylor (Inactive) added a comment -

          Earlier fix corrected, updated to numeric character constraint

          Show
          Martin Taylor (Inactive) added a comment - Earlier fix corrected, updated to numeric character constraint

            People

            • Assignee:
              Martin Taylor (Inactive)
              Reporter:
              Erik Meade
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 1 hour
                1h

                  Structure Helper Panel