Add ability to forward logs to classes implementing ArrayAccess (#233)

modxcms · Sep 26, 2023 · 25ed9e0 · 25ed9e0
1 parent 4544d16
commit 25ed9e0
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/src/xPDO/xPDO.php b/src/xPDO/xPDO.php
@@ -2099,9 +2099,17 @@ protected function _log($level, $msg, $target= '', $def= '', $file= '', $line= '
                 $filename = isset($targetOptions['filename']) ? $targetOptions['filename'] : 'error.log';
                 $filepath = isset($targetOptions['filepath']) ? $targetOptions['filepath'] : $this->getCachePath() . Cache\xPDOCacheManager::LOG_DIR;
                 $this->cacheManager->writeFile($filepath . $filename, $content, 'a');
-            } elseif ($target=='ARRAY' && isset($targetOptions['var']) && is_array($targetOptions['var'])) {
+            } elseif (
+                $target === 'ARRAY' &&
+                isset($targetOptions['var']) &&
+                (is_array($targetOptions['var']) || $targetOptions['var'] instanceof \ArrayAccess)
+            ) {
                 $targetOptions['var'][] = $content;
-            } elseif ($target=='ARRAY_EXTENDED' && isset($targetOptions['var']) && is_array($targetOptions['var'])) {
+            } elseif (
+                $target === 'ARRAY_EXTENDED' &&
+                isset($targetOptions['var']) &&
+                (is_array($targetOptions['var']) || $targetOptions['var'] instanceof \ArrayAccess)
+            ) {
                 $targetOptions['var'][] = array(
                     'content' => $content,
                     'level' => $this->_getLogLevel($level),