👌 IMPROVE: General code cleanup

Author: robertdevore

benchpress.php

@@ -241,7 +241,7 @@ function benchpress_render_settings_page() {
     if ( version_compare( PHP_VERSION, '8.0', '>=' ) ) {
         echo '<tr><th>' . esc_html__( 'Enable Switch vs Match Benchmark', 'benchpress' ) . '</th>';
         echo '<td><input type="checkbox" name="benchpress_enable_switch_vs_match" ' . checked( 1, $enable_switch_vs_match, false ) . ' /></td></tr>';
-    }    
+    }
     echo '<tr><th>' . esc_html__( 'Enable Transient vs Direct Query Benchmark', 'benchpress' ) . '</th>';
     echo '<td><input type="checkbox" name="benchpress_enable_transient_vs_query" ' . checked( 1, get_option( 'benchpress_enable_transient_vs_query', 1 ), false ) . ' /></td></tr>';
     echo '<tr><th>' . esc_html__( 'Enable Post Meta Access Benchmark', 'benchpress' ) . '</th>';
@@ -463,10 +463,9 @@ function benchpress_ajax_snapshot() {
  * This function verifies the AJAX nonce, deletes the specified snapshot 
  * from the database, and returns a success or error message based on the result.
  *
- * @since 1.0.0
- *
  * @global wpdb $wpdb WordPress database abstraction object.
  *
+ * @since  1.0.0
  * @return void
  */
 function benchpress_delete_snapshot() {

classes/BenchPress_Snapshots_Table.php

@@ -16,6 +16,9 @@ public function __construct() {
 
     /**
      * Define columns for the Snapshots table.
+     * 
+     * @since  1.0.0
+     * @return array
      */
     public function get_columns() {
         return [
@@ -27,6 +30,9 @@ public function get_columns() {
 
     /**
      * Define sortable columns.
+     * 
+     * @since  1.0.0
+     * @return array
      */
     public function get_sortable_columns() {
         return [
@@ -37,6 +43,9 @@ public function get_sortable_columns() {
 
     /**
      * Prepare items for display in the Snapshots table.
+     * 
+     * @since  1.0.0
+     * @return void
      */
     public function prepare_items() {
         global $wpdb;
@@ -61,15 +70,18 @@ public function prepare_items() {
         );
 
         $total_items = $wpdb->get_var( "SELECT COUNT(id) FROM $table_name" );
-        $this->set_pagination_args([
+        $this->set_pagination_args( [
             'total_items' => $total_items,
             'per_page'    => $per_page,
             'total_pages' => ceil( $total_items / $per_page ),
-        ]);
+        ] );
     }
 
     /**
      * Default column renderer for displaying data.
+     * 
+     * @since  1.0.0
+     * @return mixed
      */
     protected function column_default( $item, $column_name ) {
         switch ( $column_name ) {

classes/BenchPress_Table.php

@@ -23,7 +23,8 @@ public function __construct() {
 
     /**
      * Define table columns.
-     *
+     * 
+     * @since  1.0.0
      * @return array List of columns.
      */
     public function get_columns() {
@@ -36,6 +37,9 @@ public function get_columns() {
 
     /**
      * Prepare items for the table.
+     * 
+     * @since  1.0.0
+     * @return void
      */
     public function prepare_items() {
         $benchmarks  = benchpress_run_all_benchmarks();
@@ -50,6 +54,8 @@ public function prepare_items() {
      *
      * @param array  $item        Item data.
      * @param string $column_name Column name.
+     * 
+     * @since  1.0.0
      * @return mixed Column output.
      */
     public function column_default( $item, $column_name ) {
@@ -60,6 +66,9 @@ public function column_default( $item, $column_name ) {
      * Override the parent display_tablenav method to remove empty divs.
      *
      * @param string $which The position of the tablenav (top or bottom).
+     * 
+     * @since  1.0.0
+     * @return void
      */
     public function display_tablenav( $which ) {
         // Do nothing.
@@ -68,6 +77,8 @@ public function display_tablenav( $which ) {
     /**
      * Override the parent get_bulk_actions method to remove bulk actions.
      *
+     * 
+     * @since  1.0.0
      * @return array An empty array of bulk actions.
      */
     public function get_bulk_actions() {
@@ -77,6 +88,8 @@ public function get_bulk_actions() {
     /**
      * Override pagination to prevent showing pagination controls.
      *
+     * 
+     * @since  1.0.0
      * @return void
      */
     public function pagination( $which ) {

includes/helper-functions.php

@@ -12,16 +12,16 @@
  * @return array Benchmark data for WP_Query by ID.
  */
 function benchpress_benchmark_wp_query_by_id() {
-    // Retrieve settings
-    $query_type   = get_option( 'benchpress_query_type', 'single' );
-    $post_id      = get_option( 'benchpress_post_id', [] );
-    $post_type    = get_option( 'benchpress_post_type', 'post' );
-    $post_count   = get_option( 'benchpress_post_count', 5 );
-    $taxonomy     = get_option( 'benchpress_taxonomy', '' );
-    $tax_terms    = get_option( 'benchpress_tax_terms', '' );
-    $orderby      = get_option( 'benchpress_orderby', 'date' );
-    $order        = get_option( 'benchpress_order', 'ASC' );
-    $iterations   = get_option( 'benchpress_query_iterations', 1 );
+    // Retrieve settings.
+    $query_type = get_option( 'benchpress_query_type', 'single' );
+    $post_id    = get_option( 'benchpress_post_id', [] );
+    $post_type  = get_option( 'benchpress_post_type', 'post' );
+    $post_count = get_option( 'benchpress_post_count', 5 );
+    $taxonomy   = get_option( 'benchpress_taxonomy', '' );
+    $tax_terms  = get_option( 'benchpress_tax_terms', '' );
+    $orderby    = get_option( 'benchpress_orderby', 'date' );
+    $order      = get_option( 'benchpress_order', 'ASC' );
+    $iterations = get_option( 'benchpress_query_iterations', 1 );
 
     $args = [
         'post_type'      => $post_type,
@@ -90,7 +90,7 @@ function benchpress_benchmark_wp_query_by_id() {
 /**
  * Benchmark Array Merge Methods.
  *
- * @since 1.0.0
+ * @since  1.0.0
  * @return array Benchmark data for array merge.
  */
 function benchpress_benchmark_array_merge() {
@@ -226,10 +226,10 @@ function benchpress_benchmark_transient_vs_direct_query() {
         $loop_count
     );
 
-    // Measure execution time for direct query..
+    // Measure execution time for direct query.
     $start_direct = microtime( true );
     $wpdb->get_results( $query );
-    $end_direct = microtime( true );
+    $end_direct        = microtime( true );
     $direct_query_time = $end_direct - $start_direct;
 
     // Measure execution time for transient caching.

includes/php-8.0-functions.php

@@ -53,9 +53,9 @@ function benchpress_benchmark_switch_vs_match() {
     $faster_or_slower = $difference > 0 ? 'slower' : 'faster';
 
     return [
-        'name'          => esc_html__( 'Switch vs Match', 'benchpress' ),
-        'execution_time'=> round( abs( $difference ), 5 ),
-        'description'   => sprintf(
+        'name'           => esc_html__( 'Switch vs Match', 'benchpress' ),
+        'execution_time' => round( abs( $difference ), 5 ),
+        'description'    => sprintf(
             esc_html__( 'The switch statement is %s by %s seconds compared to match.', 'benchpress' ),
             $faster_or_slower,
             round( abs( $difference ), 5 )