Merge pull request #610 from SlateFoundation/fixes/task-context-defau…

…lt-titles fix(student tasks): use ->getTitle() for dynamic section titles
SlateFoundation · Oct 12, 2021 · 585706c · 585706c
2 parents ffc977e + 4b36c04
commit 585706c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/php-classes/Slate/CBL/Tasks/StudentTask.php b/php-classes/Slate/CBL/Tasks/StudentTask.php
@@ -174,8 +174,8 @@ public function getOrCreateDemonstration()
         if (!$this->Demonstration) {
             $this->Demonstration = ExperienceDemonstration::create([
                 'StudentID' => $this->StudentID,
-                'PerformanceType' => $this->Task->Title,
-                'Context' => $this->Task->Section->Title,
+                'PerformanceType' => $this->Task->getTitle(),
+                'Context' => $this->Task->Section->getTitle(),
                 'ExperienceType' => $this->Task->ExperienceType
             ]);
         }

diff --git a/php-migrations/Slate/CBL/Tasks/20161007000000_student-task-ratings.php b/php-migrations/Slate/CBL/Tasks/20161007000000_student-task-ratings.php
@@ -65,8 +65,8 @@
             'StudentID' => $StudentTask->StudentID,
             'Demonstrated' => $StudentTask->Submitted ?: null,
             'ExperienceType' => $StudentTask->ExperienceType,
-            'PerformanceType' => $StudentTask->Task->Title,
-            'Context' => $StudentTask->Section->Title
+            'PerformanceType' => $StudentTask->Task->getTitle(),
+            'Context' => $StudentTask->Section->getTitle()
         ]);
         $Demonstration->save(false);