[smarcet] - #13227

* tweaks for second breakout speakers email

Author: smarcet

sengrid_webmailer/code/SentEmailSendGrid.php

@@ -14,7 +14,7 @@
  **/
 class SentEmailSendGrid extends DataObject
 {
-    private static $db = [
+    static $db = [
       'To'            => 'Varchar(255)',
       'From'          => 'Varchar(255)',
       'Subject'       => 'Varchar(255)',

summit/_config/injector.yml

@@ -155,10 +155,9 @@ Injector:
         0: '%$BatchTaskRepository'
         1: '%$BatchTaskFactory'
         2: '%$SpeakerRepository'
-        3: '%$SpeakerSelectionAnnouncementSenderFactory'
-        4: '%$SpeakerSecondBreakOutSenderFactory'
-        5: '%$SpeakerSummitRegistrationPromoCodeRepository'
-        6: '%$TransactionManager'
+        3: '%$SpeakerSecondBreakOutSenderFactory'
+        4: '%$SpeakerSummitRegistrationPromoCodeRepository'
+        5: '%$TransactionManager'
   EventbriteEventRepository:
     class: SapphireEventbriteEventRepository
   EventbriteEventFactory:

summit/code/infrastructure/factories/SpeakerSecondBreakOutSenderFactory.php

@@ -35,7 +35,7 @@ public function build(ISummit $summit, IPresentationSpeaker $speaker)
 
         if($speaker->hasConfirmedAssistanceFor($summit->getIdentifier())){
             // send reminder without code
-            return new PresentationSpeakerSummitReminderEmailSender;
+            return new PresentationSpeakerSummitReminderEmailSender();
         }
 
         // send reminder with code

summit/code/models/managers/SpeakerSecondBreakoutAnnouncementSenderManager.php

@@ -37,11 +37,6 @@ final class SpeakerSecondBreakoutAnnouncementSenderManager
      */
     private $speaker_repository;
 
-    /**
-     * @var ISpeakerSelectionAnnouncementSenderFactory
-     */
-    private $sender_factory;
-
     /**
      * @var ISpeakerSecondBreakOutSenderFactory
      */
@@ -57,7 +52,6 @@ public function __construct
         IBatchTaskRepository $batch_repository,
         IBatchTaskFactory    $batch_task_factory,
         IEntityRepository    $speaker_repository,
-        ISpeakerSelectionAnnouncementSenderFactory $sender_factory,
         ISpeakerSecondBreakOutSenderFactory $sender_breakout_factory,
         ISpeakerSummitRegistrationPromoCodeRepository $promo_code_repository,
         ITransactionManager  $tx_manager
@@ -68,7 +62,6 @@ public function __construct
         $this->batch_task_factory      = $batch_task_factory;
         $this->speaker_repository      = $speaker_repository;
         $this->tx_manager              = $tx_manager;
-        $this->sender_factory          = $sender_factory;
         $this->sender_breakout_factory = $sender_breakout_factory;
         $this->promo_code_repository   = $promo_code_repository;
     }
@@ -138,7 +131,7 @@ public function send(ISummit $current_summit, $batch_size)
                         continue;
                     }
 
-                    $sender_service = $this->sender_factory->build($current_summit, $speaker);
+                    $sender_service = $this->sender_breakout_factory->build($current_summit, $speaker);
 
                     if (is_null($sender_service)) continue;