MOL-1124: do not load settings without db

mollie · Oct 19, 2023 · 0d4fc8d · 0d4fc8d
1 parent 159513e
commit 0d4fc8d
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/src/Resources/config/services/services.xml b/src/Resources/config/services/services.xml
@@ -10,6 +10,7 @@
             <argument type="service" id="Kiener\MolliePayments\Service\SettingsService"/>
             <argument>%kernel.logs_dir%/mollie_%kernel.environment%.log</argument>
             <argument>14</argument>
+            <argument>%env(DATABASE_URL)%</argument>
         </service>
 
         <service id="mollie_payments.logger" class="Monolog\Logger">

diff --git a/src/Service/Logger/MollieLoggerFactory.php b/src/Service/Logger/MollieLoggerFactory.php
@@ -35,25 +35,34 @@ class MollieLoggerFactory
      * @var string
      */
     private $retentionDays;
+    private string $dsn;
 
 
     /**
      * @param SettingsService $settingsService
      * @param string $filename
      * @param string $retentionDays
+     * @param string $dsn
      */
-    public function __construct(SettingsService $settingsService, string $filename, string $retentionDays)
+    public function __construct(SettingsService $settingsService, string $filename, string $retentionDays, string $dsn)
     {
         $this->settingsService = $settingsService;
         $this->filename = $filename;
         $this->retentionDays = $retentionDays;
+        $this->dsn = $dsn;
     }
 
     /**
      * @return LoggerInterface
      */
     public function createLogger(): LoggerInterface
     {
+
+        if ($this->dsn === '' || $this->dsn === 'mysql://_placeholder.test') {
+            // deployment server without database
+            return new Logger(self::CHANNEL);
+        }
+
         $config = $this->settingsService->getSettings();
 
         # 100 = DEBUG, 200 = INFO