Merge pull request #34 from EscolaLMS/feature/STAR-40

Repair bug after change value in consultations
EscolaLMS · Mar 24, 2022 · 622dcac · 622dcac
2 parents e53009a + 7d14d39
commit 622dcac
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 5 deletions.
diff --git a/src/Consultations/ApprovedTermVariables.php b/src/Consultations/ApprovedTermVariables.php
@@ -14,7 +14,7 @@ public static function defaultSectionsContent(): array
                 'user_name' => self::VAR_USER_NAME,
                 'consultation' => self::VAR_CONSULTATION_TITLE,
                 'proposed_term' => self::VAR_CONSULTATION_PROPOSED_TERM
-            ]),),
+            ])),
         ];
     }
 }
diff --git a/src/Consultations/CommonConsultationVariables.php b/src/Consultations/CommonConsultationVariables.php
@@ -13,15 +13,13 @@ abstract class CommonConsultationVariables extends EmailVariables
     const VAR_CONSULTATION_TITLE    = '@VarConsultationTitle';
     const VAR_CONSULTATION_PROPOSED_TERM    = '@VarConsultationProposedTerm';
 
-
-
     public static function mockedVariables(?User $user = null): array
     {
         $faker = \Faker\Factory::create();
         return array_merge(parent::mockedVariables(), [
             self::VAR_USER_NAME       => $faker->name(),
             self::VAR_CONSULTATION_TITLE    => $faker->word(),
-
+            self::VAR_CONSULTATION_PROPOSED_TERM => $faker->dateTime(),
         ]);
     }
 

diff --git a/src/Consultations/ReportTermVariables.php b/src/Consultations/ReportTermVariables.php
@@ -28,7 +28,7 @@ public static function requiredVariablesInSection(string $sectionKey): array
     public static function variablesFromEvent(EventWrapper $event): array
     {
         return array_merge(parent::variablesFromEvent($event), [
-            self::VAR_CONSULTATION_BUYER_NAME => $event->getConsultationTerm()->orderItem->order->user->name,
+            self::VAR_CONSULTATION_BUYER_NAME => $event->getConsultationTerm()->user->name,
         ]);
     }