Fix logLine function and add deprecation notice (#67)

tpay-com · Dec 12, 2023 · 78132ec · 78132ec
1 parent b96f98e
commit 78132ec
Show file tree

Hide file tree

Showing 5 changed files with 47 additions and 2 deletions.
diff --git a/src/Utilities/FileLogger.php b/src/Utilities/FileLogger.php
@@ -18,6 +18,19 @@ public function log($level, $message, array $context = [])
         $this->info($message);
     }
 
+    /**
+     * @param string $text
+     *
+     * @throws TException
+     *
+     * @deprecated 3.0.0
+     */
+    public function logLine($text)
+    {
+        $this->checkLogFile();
+        file_put_contents($this->getLogPath(), $text.PHP_EOL, FILE_APPEND);
+    }
+
     /**
      * @param string $message
      *

diff --git a/src/Utilities/Logger.php b/src/Utilities/Logger.php
@@ -77,9 +77,15 @@ public static function log($title, $text, $logLevel = 'info')
         self::getLogger()->log($logLevel, json_encode($content));
     }
 
-    /** @param string $text */
+    /**
+     * @param string $text
+     *
+     * @throws TException
+     *
+     * @deprecated 3.0.0
+     */
     public static function logLine($text)
     {
-        self::$logger->info((string) $text);
+        self::getLogger()->logLine($text);
     }
 }
diff --git a/src/Utilities/Util.php b/src/Utilities/Util.php
@@ -92,6 +92,10 @@ public static function log($title, $text)
      * Save one line to log file
      *
      * @param string $text text to save
+     *
+     * @throws TException
+     *
+     * @deprecated 3.0.0
      */
     public static function logLine($text)
     {

diff --git a/tests/Utilities/LoggerTest.php b/tests/Utilities/LoggerTest.php
@@ -19,6 +19,16 @@ public function testLogged()
         $this->deleteTestLog();
     }
 
+    public function testLineLogged()
+    {
+        Logger::logLine('test');
+
+        $this->assertFileExists($this->logFilename());
+
+        // tearDown
+        $this->deleteTestLog();
+    }
+
     public function testDisableLogging()
     {
         Logger::disableLogging();

diff --git a/tests/Utilities/UtilTest.php b/tests/Utilities/UtilTest.php
@@ -19,6 +19,18 @@ public function testLogged()
         $this->deleteTestLog();
     }
 
+    public function testLineLogged()
+    {
+        Util::$loggingEnabled = true;
+        Util::$customLogPatch = null;
+
+        Util::logLine('test');
+
+        $this->assertFileExists($this->logFilename());
+
+        $this->deleteTestLog();
+    }
+
     public function testDisableLogging()
     {
         Util::$loggingEnabled = false;