Fixes yiisoft#15046: Throw an yii\web\HeadersAlreadySentException i…

…f headers were sent before web response

framework/CHANGELOG.md

@@ -4,6 +4,7 @@ Yii Framework 2 Change Log
 2.0.14 under development
 ------------------------
 
+- Bug #15046: Throw an `yii\web\HeadersAlreadySentException` if headers were sent before web response (dmirogin)
 - Enh #3087: Added `yii\helpers\BaseHtml::error()` "errorMethod" option to be able to customize errors display (yanggs07, developeruz)
 - Bug #15355: Fixed `yii\db\Query::from()` does not work with `yii\db\Expression` (vladis84)
 - Bug #8983: Only truncate the original log file for rotation (matthewyang, developeruz)

framework/web/HeadersAlreadySentException.php

@@ -0,0 +1,29 @@
+<?php
+/**
+ * @link http://www.yiiframework.com/
+ * @copyright Copyright (c) 2008 Yii Software LLC
+ * @license http://www.yiiframework.com/license/
+ */
+
+namespace yii\web;
+
+use yii\base\Exception;
+
+/**
+ * HeadersAlreadySentException represents an exception caused by
+ * any headers that were already sent before web response was sent.
+ *
+ * @author Dmitry Dorogin <[email protected]>
+ * @since 2.0.14
+ */
+class HeadersAlreadySentException extends Exception
+{
+    /**
+     * @inheritdoc
+     */
+    public function __construct($file, $line)
+    {
+        $message = YII_DEBUG ? "Headers already sent in {$file} on line {$line}." : 'Headers already sent.';
+        parent::__construct($message);
+    }
+}

framework/web/Response.php

@@ -362,8 +362,8 @@ public function clear()
      */
     protected function sendHeaders()
     {
-        if (headers_sent()) {
-            return;
+        if (headers_sent($file, $line)) {
+            throw new HeadersAlreadySentException($file, $line);
         }
         if ($this->_headers) {
             $headers = $this->getHeaders();