Merge pull request #41 from indiana-university/LMSA-9502

LMSA-9502 - add a different display for non-SIS courses
indiana-university · Dec 3, 2024 · 55d52ee · 55d52ee
2 parents 81580f0 + 5ccb0d9
commit 55d52ee
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 1 deletion.
diff --git a/src/main/java/edu/iu/uits/lms/crosslist/controller/DecrosslistController.java b/src/main/java/edu/iu/uits/lms/crosslist/controller/DecrosslistController.java
@@ -123,6 +123,7 @@ public String lookupSearchBySisId(@ModelAttribute FindParentModel findParentMode
             model.addAttribute("availableToDecrosslistSectionsList", availableToDecrosslistSectionsList);
 
             model.addAttribute("findParentResult", findParentResult);
+
             // add canvasCourseId to be used in audit log purposes later
             model.addAttribute("canvasCourseId", findParentResult.getCanvasCourseId());
 
@@ -203,6 +204,7 @@ public String decrossListSections(@RequestParam("section-checkboxes") List<Strin
 
         if (findParentResult != null) {
             model.addAttribute("findParentResult", findParentResult);
+
             // add canvasCourseId to be used in audit log purposes later
             model.addAttribute("canvasCourseId", findParentResult.getCanvasCourseId());
 

diff --git a/src/main/java/edu/iu/uits/lms/crosslist/model/FindParentResult.java b/src/main/java/edu/iu/uits/lms/crosslist/model/FindParentResult.java
@@ -49,4 +49,8 @@ public class FindParentResult {
     private String statusIconName;
     private String canvasCourseId;
     private List<Section> sectionList;
+
+    public String displaySisCourseId() {
+        return sisCourseId == null ? "non-SIS course" : sisCourseId;
+    }
 }
diff --git a/src/main/resources/templates/findParentCourse.html b/src/main/resources/templates/findParentCourse.html
@@ -94,7 +94,7 @@ <h1 id="tool-title" class="rvt-p-top-sm rvt-ts-36">De-cross-listing sections</h1
         <form th:object="${findParentModel}" id="decrosslist-form" th:action="@{|/app/decrosslist-sections|}" method="post">
             <div class="rvt-flex rvt-items-center">
                 <rvt-icon th:name="${findParentResult.statusIconName}" class="rvt-border-radius-circle rvt-p-all-xs" th:classappend="${findParentResult.statusIconCssClasses}"></rvt-icon>
-                <h2 id="results-message" tabindex="-1" th:text="|Parent course found: ${findParentResult.sisCourseId}|" class="rvt-ts-18 rvt-text-bold rvt-p-left-xs">Parent course found</h2>
+                <h2 id="results-message" tabindex="-1" th:text="|Parent course found: ${findParentResult.displaySisCourseId}|" class="rvt-ts-18 rvt-text-bold rvt-p-left-xs">Parent course found</h2>
             </div>
             <div>
                 <div class="rvt-p-top-sm">