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

Quote:

---

Originally Posted by 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,

Code:

---

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.

Quote:

---

Originally Posted by 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:

Code:

---

        <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:

Code:

---

        <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.