Logs not being written after upgrade to 1.5.5e/f

Carrying this discussion over from the One-Page Checkout support thread.

jpda:

Have checked it again.
Removed both local directories and put both local configure.php files in their 'normal' directories (/includes/ and /admin/includes/, w/o explicit defines for logs. Thus forcing reading of these configure files.
Result: both debug logs were written.

Restored both local directories, put both **local **configure.php files in those local directories (/includes/local/ and /admin/includes/local/), w/o explicit defines for logs
And:
Restored both original configure.php files from the live site into their 'normal' directories (/includes/ and /admin/includes/).
Result: no log files!

Extra check: when I remove all configure.php files from /includes/ and /admin/includes/, leaving the local versions w/o explicit defines for logs, then debug logs will be written!

Conclusion? :unsure::unsure:

Cheers,

jpda

conclusion:

the definition in the local configure is WRONG.

the local gets loaded before the straight includes. look at the following two defines:

define('DIR_FS_LOGS', DIR_FS_CATALOG . '/zc_logs');
define('DIR_FS_LOGS', '/var/www/test_site/zc_logs');

if the 1st line is in the local configure, at that point DIR_FS_CATALOG is NOT defined. so your log file directory is not defined.

in the 2nd line, by explicitly defining the directory, the logs get correctly written.

best.

@jpda was kind enough to send me copies of his /includes/configure.php and /includes/local/configure.php. Turns out that the issue was that the /includes/configure.php (since he's doing his upgrade locally) hadn't been updated from the Zen Cart 1.5.4 format, so it included all the sub-definitions that (as of Zen Cart 1.5.5) have been moved to the /includes/defined_paths.php script ... like DIR_FS_LOGS.

Commenting-out those definitions to put the /includes/configure.php "in line" with ZC1.5.5+ usage corrected the issue.

Indeed, my local configure did have the 1.5.5 format, the /includes/configure.php did not as @lat9 kindly figured out.

jpda