adds a check before ancestors are accessed

learningequality · Feb 6, 2025 · 406f138 · 406f138
1 parent 89a99e8
commit 406f138
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 4 deletions.
diff --git a/...e/sidePanels/LessonResourceSelection/subPages/PreviewSelectedResources/PreviewContent.vue b/...e/sidePanels/LessonResourceSelection/subPages/PreviewSelectedResources/PreviewContent.vue
@@ -30,7 +30,8 @@
     </div>
 
     <ResourceSelectionBreadcrumbs
-      :ancestors="ancestors"
+      v-if="ancestors.length"
+      :ancestors="[...ancestors, content]"
       :channelsLink="channelsLink"
       :topicsLink="topicsLink"
       class="align-breadcrumbs"
@@ -40,7 +41,7 @@
       <h5>
         <KLabeledIcon :label="content.kind">
           <template #icon>
-            <LearningActivityIcon :kind="content.learning_activities" />
+            <LearningActivityIcon :kind="learningActivities" />
           </template>
           <template>
             {{ content.title }}
@@ -147,6 +148,11 @@
         type: Boolean,
         required: true,
       },
+      questions: {
+        type: Array,
+        required: false,
+        default: () => [],
+      },
     },
     data() {
       return {
@@ -182,6 +188,12 @@
 
         return undefined;
       },
+      learningActivities() {
+        if (this.content.learning_activities) {
+          return this.content.learning_activities;
+        }
+        return [];
+      },
     },
     methods: {
       topicsLink(topicId) {

diff --git a/...ummaryPage/sidePanels/LessonResourceSelection/subPages/PreviewSelectedResources/index.vue b/...ummaryPage/sidePanels/LessonResourceSelection/subPages/PreviewSelectedResources/index.vue
@@ -64,8 +64,8 @@
     },
     computed: {
       isSelected() {
-        if (this.selectedResources) {
-          return this.selectedResources.find(resource => resource.id === this.contentId);
+        if (this.selectedResources && this.contentNode) {
+          return this.selectedResources.some(resource => resource.id === this.contentNode.id);
         }
         return false;
       },