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

… errors
yiisoft · Oct 3, 2017 · bd85fa4 · bd85fa4
1 parent 6eae7fb
commit bd85fa4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/framework/CHANGELOG.md b/framework/CHANGELOG.md
@@ -4,6 +4,7 @@ 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 #14129: Fixed console help to properly work with tricky camelcased controller names (samdark, silverfire)
 - Enh #14126: Added variadic parameters support to DI container (SamMousa)
 - Enh #14087: Added `yii\web\View::registerCsrfMetaTags()` method that registers CSRF tags dynamically ensuring that caching doesn't interfere (RobinKamps)

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;
         }