View Issue Details

IDProjectCategoryView StatusLast Update
0007147OXID eShop (all versions)4.07. Source code, Testpublic2020-06-09 07:55
Reporterhenrik.steffen 
PrioritylowSeverityminorReproducibilityalways
Status closedResolutionno change required 
Product Version6.2.1 
Target VersionFixed in Version 
Summary0007147: OXID default error_reporting includes E_WARNING
DescriptionWe think Warnings are a little bit too much logging for productive systems.

Would be nice if this could be removed from ShopControl.php in _getErrorReportingLevel

change
$errorReporting = E_ALL ^ E_NOTICE;

to
$errorReporting = E_ALL ^ E_NOTICE ^ E_WARNING;


Or maybe E_WARNING can be active in debug mode, or while not in productive mode?

Why is this configured in ShopControl at all, where we can't override it?
And at the same time we have error_reporting(E_ALL & ~E_DEPRECATED & ~E_NOTICE); in the bootstrap.php


TagsNo tags attached.
ThemeNot defined
BrowserNot defined
PHP VersionNot defined
Database VersionNot defined

Activities

QA

2020-06-08 08:49

administrator   ~0013251

Have you tried to set the PSR3 log level in config.inc.php to
$this->sLogLevel = 'error';

See: https://github.com/OXID-eSales/oxideshop_ce/blob/master/source/config.inc.php.dist#L39

henrik.steffen

2020-06-08 14:02

reporter   ~0013252

sorry, obviously newer versions of OXID eShop solved this issue with PSR-3 logger.