If using the function "set_error_handler" even the possible errors containing @ are registered #2526

Index: Renderer.php

=========================================================

--- Renderer.php (rvision 4955)
+++ Renderer.php (copie de travail)
@@ -52,7 +52,10 @@
*/
protected static function renderHeader()
{

• @header('Content-Type: text/html; charset=utf-8');
• if (!headers_sent())
• {
• header('Content-Type: text/html; charset=utf-8');

•         }
  }
  
  /**

Keywords: core DataTable Renderer.php

Attachment:
patch.diff

Is this necessary? ErrorHandler checks if error_reporting() returns 0 (indicating the use of @).

Even so, what are you debugging that you needed this patch?

I record all types of errors, I made changes on my version, but instead of using the @ to hide the error, the patch will make it easier not to use it

There are many places we use silent fail @ and better not change just one, your error handler shouldd check for the silent fail instead