Fixes #9438: yii\web\DbSession now relies on error handler to displ…

…ay errors
yiisoft · Oct 6, 2017 · 66723d0 · 66723d0
1 parent be211f7
commit 66723d0
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/framework/CHANGELOG.md b/framework/CHANGELOG.md
@@ -3,6 +3,8 @@ Yii Framework 2 Change Log
 
 2.0.13 under development
 ------------------------
+
+- Enh #9438: `yii\web\DbSession` now relies on error handler to display errors (samdark)
 - Bug #6226: Fix fatal symlink error when publishing in multi threaded environments (dynasource)
 - Enh #13486: Use DI container to instantiate cookies in order to be able to set defaults (samdark)
 - Bug #14902: Fixed PHP notice in `yii\web\MultipartFormDataParser` (olimsaidov)

diff --git a/framework/web/DbSession.php b/framework/web/DbSession.php
@@ -183,13 +183,7 @@ public function writeSession($id, $data)
                     ->execute();
             }
         } catch (\Exception $e) {
-            $exception = ErrorHandler::convertExceptionToString($e);
-            // its too late to use Yii logging here
-            error_log($exception);
-            if (YII_DEBUG) {
-                echo $exception;
-            }
-
+            Yii::$app->errorHandler->handleException($e);
             return false;
         }