KS Enrollment
  1. KS Enrollment
  2. KSENROLL-3318

Manage Delivery Logistics :: Implement additional validations when creating RDLs

    Details

    • Type: Bug Fix Bug Fix
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: M5
    • Fix Version/s: M5
    • Component/s: PDT - Green
    • Security Level: Public (Public: Anyone can view)
    • Labels:
    • KS ENR Functional Area:
      Course Offering

      Description

      Implement additional validations which occur after clicking the "Add" button when revising delivery logistics ...
      o All non-TBA RDLs require all fields are required and must be valid. Note: This validation is only um valid for M5. Later this validation will be relaxed to accommodate "open ended" requests as seen in https://wiki.kuali.org/pages/viewpage.action?pageId=314716311 item 28. Please think about this when implementing and provide good comments.
      o TBA RDLs must be missing at least one field (else they aren't really TBAs).
      o Add additional condition to building/room code validation. If room code is present a building code is required.

        Issue Links

          Activity

          Hide
          Glenn Sudduth (Inactive) added a comment - - edited

          The "non-TBA RDLs require all fields" validation will prevent KSENROLL-3304 from happening, but when that validation is relaxed it will probably resurface, so it will be backlogged.

          Show
          Glenn Sudduth (Inactive) added a comment - - edited The "non-TBA RDLs require all fields" validation will prevent KSENROLL-3304 from happening, but when that validation is relaxed it will probably resurface, so it will be backlogged.
          Hide
          Cassy Beekman (Inactive) added a comment -

          Adam and Taryn: let's use this jira for QA for all of these related issues from KSENROLL-3315, KSENROLL-3304, and this jira. Please make sure these issues are covered:

          FROM KSENROLL-3315:
          1. If TBA selected and none other fields entered, we get stacktrace
          2. If only starttime entered and starttime am/pm left blank, stacktrace
          3. if room is entered, building is a required field
          4. If starttime/endtime AP/PM selected, then start/end time is required
          5. Editing a schedule request, doesn't copy the TBA for edit

          FROM KSENROLL-3304:
          Adding a Requested Delivery Logistic with all fields empty (except Features) results in a stacktrace.
          Reproduction Steps:
          o Navigate to the "Revise" Delivery Logistics form.
          o Click the Add button without filling in any scheduling info. This will result in an RDL with only a feature code.
          o Click Save and Process. This results in an NPE in ActivityOfferingMaintainableImpl.loadScheduleActuals().

          Show
          Cassy Beekman (Inactive) added a comment - Adam and Taryn: let's use this jira for QA for all of these related issues from KSENROLL-3315 , KSENROLL-3304 , and this jira. Please make sure these issues are covered: FROM KSENROLL-3315 : 1. If TBA selected and none other fields entered, we get stacktrace 2. If only starttime entered and starttime am/pm left blank, stacktrace 3. if room is entered, building is a required field 4. If starttime/endtime AP/PM selected, then start/end time is required 5. Editing a schedule request, doesn't copy the TBA for edit FROM KSENROLL-3304 : Adding a Requested Delivery Logistic with all fields empty (except Features) results in a stacktrace. Reproduction Steps: o Navigate to the "Revise" Delivery Logistics form. o Click the Add button without filling in any scheduling info. This will result in an RDL with only a feature code. o Click Save and Process. This results in an NPE in ActivityOfferingMaintainableImpl.loadScheduleActuals().
          Hide
          Taryn Pedigo (Inactive) added a comment -

          Behaves as expected.

          Show
          Taryn Pedigo (Inactive) added a comment - Behaves as expected.

            People

            • Assignee:
              Andrew Lubbers (Inactive)
              Reporter:
              Glenn Sudduth (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Structure Helper Panel