lat9 / emailfor404 Goto Github PK

Could this not be auto.loading and so not need the auto_loader config file?

AFAICS I don't see the bad url (that was not found) in any of the info contained in the email!

define('SEND_404_ERRORS_EMAILS_TO', '[email protected]');     // This is the email address that the 404 emails are sent to
define('SEND_404_ERRORS_EMAIL_TO_STATUS', '1');             // Set to '1' to enable the email sending
define('SEND_404_ERRORS_EMAIL_ARCHIVE', '0');               // Set to '0' for no archiving, any other value to write to email_archive in database

nitpick>..I would have expected these to be "elsewhere" and unique: since they get duplicated when in a language file...</nitpick

Update class-file constructor names to __constructor for compliance with PHP 7.0

Just trying out php 8 and am paying the price...ZC158, Xampp 8.0.1-1.
This is not reported on php 7.3.

[03-Feb-2021 16:45:08 Europe/Madrid]
Navigation: [0][not set] [1][not set] [2][not set]
Request URI: /tienda-158/
IP address: 127.0.0.1
#1 define() called at [CATALOG/includes/auto_loaders/config.emailFor404.php:17]
#2 require(CATALOG/includes/auto_loaders/config.emailFor404.php) called at [CATALOG/includes/classes/InitSystem.php:219]
#3 Zencart/InitSystem/InitSystem->getLoadersFromFilelist() called at [CATALOG/includes/classes/InitSystem.php:171]
#4 Zencart/InitSystem/InitSystem->loadAutoLoadersFromSystem() called at [CATALOG/includes/classes/InitSystem.php:33]
#5 Zencart/InitSystem/InitSystem->loadAutoLoaders() called at [CATALOG/includes/application_top.php:248]
#6 require(CATALOG/includes/application_top.php) called at [CATALOG/index.php:25]
--> PHP Warning: Constant SEND_404_ERRORS_EMAILS_TO already defined in ...\includes\auto_loaders\config.emailFor404.php on line 17.

....and for all the subsequent constants.
I imagine there is double loading from those new InitSystem...