Merge pull request #228 from bmlt-enabled/optimizations

optimization and logging

Author: otrok7

admin/class-bmltenabled-admin.php

@@ -37,8 +37,8 @@ function admin_menu_link()
         $icon = apply_filters("BmltEnabled_IconSVG", 'dashicons-location-alt');
         $slug = $slugs[0];
         add_menu_page(
-            'Meeting List',
-            'Meeting List',
+            'Meeting Lists',
+            'Meeting Lists',
             $cap,
             $slug,
             '',

admin/class-bread-admin.php

@@ -264,6 +264,12 @@ function download_settings()
             $this->download_settings_inner();
         }
     }
+    function download_mpdf_log()
+    {
+        if ($this->bread->exportingLogFile()) {
+            $this->download_log_file();
+        }
+    }
     private function download_settings_inner()
     {
         $this->bread->getConfigurationForSettingId($this->bread->getRequestedSetting());
@@ -287,6 +293,33 @@ private function download_settings_inner()
         file_put_contents('php://output', $json_file);
         exit;
     }
+    function download_log_file()
+    {
+        if (!isset($_REQUEST['export-mpdf-log'])) {
+            exit;
+        }
+        foreach (Bread::get_log_files() as $log) {
+            if ($log['name'] === $_REQUEST['export-mpdf-log']) {
+                $this->exportLogFile($log['path']);
+            }
+        }
+        exit;
+    }
+    function exportLogFile($file)
+    {
+        ignore_user_abort(true);
+        header('Content-Description: File Transfer');
+        header('Content-Type: text/html; charset=utf-8');
+        header('Content-Disposition: attachment; filename="'.basename($file).'"');
+        header('Expires: 0');
+        header('Cache-Control: must-revalidate');
+        header('Pragma: public');
+        header('Content-Length: ' . filesize($file));
+        ob_end_flush();
+        readfile($file);// phpcs:ignore
+        ob_end_flush();
+        exit;
+    }
     function current_user_can_modify()
     {
         if (! current_user_can('manage_bread')) {
@@ -416,8 +449,8 @@ function admin_submenu_link($parent_slug)
         global $my_admin_page;
         $my_admin_page = add_submenu_page(
             $parent_slug,
-            'Printable Meeting List',
-            'Printable Meeting List',
+            'Printable Meeting Lists',
+            'Printable Meeting Lists',
             'manage_bread',
             basename(__FILE__),
             array(&$this, 'admin_options_page'),
@@ -555,6 +588,9 @@ function process_customize_form()
             }
             $this->bread->getConfigurationForSettingId($this->bread->getRequestedSetting());
             $this->bread->setOption('bread_version', sanitize_text_field($_POST['bread_version']));
+            $this->bread->setOption('logging', isset($_POST['logging']));
+            $this->bread->setOption('simpleTables', isset($_POST['simpleTables']));
+            $this->bread->setOption('packTabledata', isset($_POST['packTabledata']));
             $this->bread->setOption('front_page_content', wp_kses_post($_POST['front_page_content']));
             $this->bread->setOption('front_page_line_height', $_POST['front_page_line_height']);
             $this->bread->setOption('front_page_font_size', floatval($_POST['front_page_font_size']));
@@ -669,7 +705,7 @@ function process_customize_form()
             $this->bread->setOption('extra_meetings', array());
             if (isset($_POST['extra_meetings'])) {
                 foreach ($_POST['extra_meetings'] as $extra) {
-                    $this->bread->setOption('extra_meetings', wp_kses_post($extra));
+                    $this->bread->appendOption('extra_meetings', wp_kses_post($extra));
                 }
             }
             $authors = isset($_POST['authors_select']) ? $_POST['authors_select'] : [];

admin/partials/_bmlt_server_setup.php

@@ -133,6 +133,32 @@
                     </select>
                 </div>
             </div>
+            <div id="meetinglistloggingdiv" class="postbox">
+                <h3 class="hndle">Optimize/Debug mPDF<span title='<p>If there are problens during meeting list generation, you may enable debugging to help locate the error.</p><p><i>Please disable if not actively involved in locating problems.</i></p>' class="my-tooltip"><span class="tooltipster-icon">(?)</span></span></h3>
+                <div class="inside">
+                    <input type="checkbox" name="logging" id="logging" <?php echo $this->bread->emptyOption('logging') ? '' : 'checked'; ?>>
+                    <label for="logging">Enable Logging</label>
+                    <br/>
+                    <input type="checkbox" name="simpleTables" id="simpleTables" <?php echo $this->bread->emptyOption('simpleTables') ? '' : 'checked'; ?>>
+                    <label for="simpleTables">Enable SimpleTables</label>
+                    <br/>
+                    <input type="checkbox" name="packTabledata" id="packTabledata" <?php echo $this->bread->emptyOption('packTabledata') ? '' : 'checked'; ?>>
+                    <label for="packTabledata">Pack Table Data</label>
+                    <?php
+                    $logs = Bread::get_log_files();
+                    if (!empty($logs)) {?>
+                        <br/><h4>Download Log Files</h4>
+                        <?php
+                        foreach ($logs as $log) {
+                            ?>
+                            <a href="<?php echo esc_url(home_url().'/?export-mpdf-log='.$log['name']);?>"><?php echo esc_html($log['name']);?></a>
+                            <?php
+                        }
+                    }
+                    ?>
+                    <br/>
+                </div>
+            </div>
             <div id="meetinglistcachediv" class="postbox">
                 <h3 class="hndle">Meeting List Cache<span title='<p>Meeting List data is cached (as database transient) to generate a Meeting List faster.</p><p><i>CACHE is DELETED when you Save Changes.</i></p><p><b>The meeting list will not reflect changes to BMLT until the cache expires or is deleted.</b></p>' class="my-tooltip"><span class="tooltipster-icon">(?)</span></span></h3>
                 <div class="inside">

bmlt-meeting-list.php

@@ -11,7 +11,7 @@
  * Plugin Name:       Bread
  * Plugin URI:        https://bmlt.app
  * Description:       Maintains and generates PDF Meeting Lists from BMLT.
- * Version:           2.8.11
+ * Version:           2.9.0
  * Author:            bmlt-enabled
  * Author URI:        https://bmlt.app/
  * License:           GPL-2.0+

composer.json

@@ -10,6 +10,7 @@
   ],
   "require": {
     "mpdf/mpdf": "8.2.3",
+    "monolog/monolog": "3.8.1",
     "mpdf/qrcode": "^1.0",
     "myclabs/deep-copy": "1.11.1"
   },