PISHPS-387: "Starting Subscription Renewal Reminder from Scheduled Ta…

…sks" log info message now only appears when subscriptions are enabled (#893)
mollie · Nov 14, 2024 · b4961c2 · b4961c2
1 parent b011aa0
commit b4961c2
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/src/Components/Subscription/Actions/RemindAction.php b/src/Components/Subscription/Actions/RemindAction.php
@@ -37,6 +37,11 @@ class RemindAction extends BaseAction
      */
     private $reminderValidator;
 
+    /**
+     * @var LoggerInterface
+     */
+    protected $log;
+
 
     /**
      * @param SettingsService $pluginSettings
@@ -96,6 +101,11 @@ public function remindSubscriptionRenewal(Context $context): int
                 continue;
             }
 
+            $message = 'Starting Subscription Renewal Reminder from Scheduled Tasks for SalesChannel "%s".';
+            $message = sprintf($message, $salesChannel->getName());
+
+            $this->log->info($message);
+
             $daysOffset = $settings->getSubscriptionsReminderDays();
 
             $availableSubscriptions = $this->getRepository()->findByReminderRangeReached($salesChannel->getId(), $context);

diff --git a/src/ScheduledTask/Subscription/RenewalReminder/AbstractRenewalReminderTaskHandler.php b/src/ScheduledTask/Subscription/RenewalReminder/AbstractRenewalReminderTaskHandler.php
@@ -53,8 +53,6 @@ public static function getHandledMessages(): iterable
     public function run(): void
     {
         try {
-            $this->logger->info('Starting Subscription Renewal Reminder from Scheduled Tasks.');
-
             $context = new Context(new SystemSource());
 
             $remindedCount = $this->subscriptionManager->remindSubscriptionRenewal($context);