Display Log Files [Support Thread] (v1.5.1-to-v1.5.6)

torvista:

This functionality (showing the existence of debug logs in the admin message stack) is included in 1.6, why not copy it from there and avoid an unnecessary upgrade merge.

Understood.. and when 1.6 is released, I'll deal with it during the upgrade.. It's a fairly minor change anyway.. And I've only installed this on ONE site.. (just discovered this mod)

I noticed a "display_logs_uninstall.sql" in the zipped file,

DELETE FROM admin_pages WHERE page_key='toolsDisplayLogs';

is there an install SQL to install in the admin pages?

Please ignore the above comment. As it auto registers. Sorry.

I've submitted v1.0.4 to the Plugins to correct a couple of issues:

BUGFIX: Unprintable characters result in blank file display.
BUGFIX: Refreshing page after file delete results in error.

This (and previous) versions of the plugin are Zen Cart v1.5.2-compatible.

I've submitted v1.0.5 to the plugins; somewhere along the line :huh:, the USPS shipping log files started starting with SHIP_usps instead of just usps.

lat9:

I've submitted v1.0.5 to the plugins; somewhere along the line :huh:, the USPS shipping log files started starting with SHIP_usps instead of just usps.

It wasn't really until I actively started using this plugin on a local computer while developing some functionality that I realized I had been missing something on my active site. Apparently there are "instructions" displayed at line 209 of ADMIN/display_logs.php, but only if accessing the logs from a non secure site. The line provides two pieces of information, 1) A message about not using a SSL connection and that viewing the contents of the logs could present a security risk. and 2) instructions about usage of the log viewer, but they are concatenated together inside of the https test. So, here is what I did/changed at and around that line. (Changes made/to make attempted to be highlighted in red.) As said, wouldn't even know you were missing something unless you were using a non-SSL server setting.

V1.0.5 of the code (lines 202-213) change from:

        <td><table border="0" width="100%" cellspacing="0" cellpadding="0">
          <tr>
            <td class="pageHeading"><?php echo HEADING_TITLE; ?></td>
            <td class="pageHeading" align="right"><?php echo zen_draw_separator('pixel_trans.gif', HEADING_IMAGE_WIDTH, HEADING_IMAGE_HEIGHT); ?></td>
          </tr>
<?php if (substr(HTTP_SERVER, 0, 5) != 'https') {  // display security warning about downloads if not SSL ?>
          <tr>
            <td class="main"><?php echo WARNING_NOT_SECURE . sprintf(TEXT_INSTRUCTIONS, MAX_LOG_FILE_READ_SIZE, ((isset($_GET) && isset($_GET['sort']) && $_GET['sort'] == 'a') ? TEXT_OLDEST : TEXT_MOST_RECENT), (($numLogFiles > MAX_LOG_FILES_TO_VIEW) ? MAX_LOG_FILES_TO_VIEW : $numLogFiles), $numLogFiles); ?></td>
            <td class="main" align="right"><?php echo zen_draw_separator('pixel_trans.gif', HEADING_IMAGE_WIDTH, HEADING_IMAGE_HEIGHT); ?></td>
          </tr>
<?php } ?>
        </table></td>

Change to:

        <td><table border="0" width="100%" cellspacing="0" cellpadding="0">
          <tr>
            <td class="pageHeading"><?php echo HEADING_TITLE; ?></td>
            <td class="pageHeading" align="right"><?php echo zen_draw_separator('pixel_trans.gif', HEADING_IMAGE_WIDTH, HEADING_IMAGE_HEIGHT); ?></td>
          </tr>
          <tr>
          <td class="main"><?php if (substr(HTTP_SERVER, 0, 5) != 'https') {  // display security warning about downloads if not SSL ?>
                      <?php echo WARNING_NOT_SECURE; 
} 
                     echo sprintf(TEXT_INSTRUCTIONS, MAX_LOG_FILE_READ_SIZE, ((isset($_GET) && isset($_GET['sort']) && $_GET['sort'] == 'a') ? TEXT_OLDEST : TEXT_MOST_RECENT), (($numLogFiles > MAX_LOG_FILES_TO_VIEW) ? MAX_LOG_FILES_TO_VIEW : $numLogFiles), $numLogFiles); ?></td>
            <td class="main" align="right"><?php echo zen_draw_separator('pixel_trans.gif', HEADING_IMAGE_WIDTH, HEADING_IMAGE_HEIGHT); ?></td>
          </tr>
        </table></td>

Oops! :ohmy: I'll get that change into the queue! Thanks for the report.

I've just uploaded v1.0.6 to the plugins to correct the missing instructions' display and to correct a problem where the "delete all" button didn't work if you also used the "CSS Buttons for Admin" plugin.

Note: The zip file identifies itself as v1.0.6a, since I'd messed up the versioning on my github repository.

v1.0.6 is now available from the plugins.

I note that 1.0.7 has three folders. YOUR_ADMIN, Zen Cart 1.5.1, and Zen Cart v1.5.3. I am assume (I know) that the YOUR_ADMIN folder is the basic and one would overwrite the YOUR_ADMIN/includes/header.php with the appropriate one for their version? If this is correct, which would be appropriate for 1.5.2.
Could not find mention of this in the install portion of the readme.

dbltoe:

If this is correct, which would be appropriate for 1.5.2.
Could not find mention of this in the install portion of the readme.
Yes, I am aware that 1.5.2 was released as beta BUT, so many ran it anyway that you will find a database upgrade step in 1.5.3 to upgrade from 1.5.2. Plus, you'll find a whatsnew_1.5.2.html and a changed_files-v1-5-2.html in the docs.
If it's an IDK, that's fine. Just wanting to dot the T's and cross the I's. :P

v1.0.7 introduced torvista's notification that log files are present via message to the admin's header. That's the difference between the Zen Cart v1.5.1 and Zen Cart v1.5.3 directories, just the different /YOUR_ADMIN/includes/header.php files to contain the update for that header message. For v1.5.2, use the v1.5.3 version if you want that feature.

If you just want the "Display Logs" functionality, then the files in the /YOUR_ADMIN directory are all you need.

SO... If I want Torvista's addition would you merge the two header.php files or just the one in the specific version of ZC?

dbltoe, I'll need to review what's going on with the header.php modification to give you informed instructions.

The changes to the /admin/includes/header.php file from ZC v1.5.1 to v1.5.3 appear to be downwardly compatible (there was no change from v1.5.2 to v1.5.3) and, it turns out, it's the v1.5.3 (modified) version of that file that's in the /YOUR_ADMIN/includes/header.php file that is part of this plugin's distribution.

I'll update the plugin's distribution file to identify that core-file overwrite in the readme.html and remove the redundant directories.

I've submitted v1.0.8 to the plugins; it makes a modification to the previous header-message change so that the message identifies the number of myDEBUG-*.log files and contains a link to the Display Logs page. The extraneous directories were also removed.

THANX for all you do for the ZC community:hug:

v1.0.8 is now available for download.

Typical work.
Translation -- Outstanding:clap::bigups::wow:

Will be playing with this to have it suggest our customers contact us immediately or add an auto e-mail notification.

dbltoe, thanks for your kind words (and smileys!).