pryley / blackbar

plugin/Modules/Actions.php

Indentation +146 added lines, -146 removed lines

@@ -6,162 +6,162 @@
 
 class Actions implements Module
 {
-    /**
-     * @var Application
-     */
-    protected $app;
-    /**
-     * @var array
-     */
-    protected $entries;
-    /**
-     * @var array
-     */
-    protected $hooks;
-    /**
-     * @var int
-     */
-    protected $totalActions;
-    /**
-     * @var float
-     */
-    protected $totalTime;
+	/**
+	 * @var Application
+	 */
+	protected $app;
+	/**
+	 * @var array
+	 */
+	protected $entries;
+	/**
+	 * @var array
+	 */
+	protected $hooks;
+	/**
+	 * @var int
+	 */
+	protected $totalActions;
+	/**
+	 * @var float
+	 */
+	protected $totalTime;
 
-    public function __construct(Application $app)
-    {
-        $this->app = $app;
-        $this->entries = [];
-        $this->hooks = [];
-        $this->totalActions = 0;
-        $this->totalTime = (float) 0;
-    }
+	public function __construct(Application $app)
+	{
+		$this->app = $app;
+		$this->entries = [];
+		$this->hooks = [];
+		$this->totalActions = 0;
+		$this->totalTime = (float) 0;
+	}
 
-    public function callbacksForHook(string $action): void
-    {
-        global $wp_filter;
-        if (!array_key_exists($action, $this->hooks)) {
-            return;
-        }
-        $this->hooks[$action]['callbacks_count'] = 0;
-        foreach ($wp_filter[$action] as $priority => $callbacks) {
-            if (!array_key_exists($priority, $this->hooks[$action]['callbacks'])) {
-                $this->hooks[$action]['callbacks'][$priority] = [];
-            }
-            foreach ($callbacks as $callback) {
-                if (is_array($callback['function']) && 2 === count($callback['function'])) {
-                    list($object, $method) = $callback['function'];
-                    if (is_object($object)) {
-                        $object = get_class($object);
-                        // $reflection = new \ReflectionClass($object);
-                        // if (str_starts_with($reflection->getNamespaceName(), 'GeminiLabs\BlackBar')) {
-                        //     continue; // skip blackbar callbacks
-                        // }
-                    }
-                    $this->hooks[$action]['callbacks'][$priority][] = sprintf('%s::%s', $object, $method);
-                } elseif (is_object($callback['function'])) {
-                    $this->hooks[$action]['callbacks'][$priority][] = get_class($callback['function']);
-                } else {
-                    $this->hooks[$action]['callbacks'][$priority][] = $callback['function'];
-                }
-                ++$this->hooks[$action]['callbacks_count'];
-            }
-        }
-    }
+	public function callbacksForHook(string $action): void
+	{
+		global $wp_filter;
+		if (!array_key_exists($action, $this->hooks)) {
+			return;
+		}
+		$this->hooks[$action]['callbacks_count'] = 0;
+		foreach ($wp_filter[$action] as $priority => $callbacks) {
+			if (!array_key_exists($priority, $this->hooks[$action]['callbacks'])) {
+				$this->hooks[$action]['callbacks'][$priority] = [];
+			}
+			foreach ($callbacks as $callback) {
+				if (is_array($callback['function']) && 2 === count($callback['function'])) {
+					list($object, $method) = $callback['function'];
+					if (is_object($object)) {
+						$object = get_class($object);
+						// $reflection = new \ReflectionClass($object);
+						// if (str_starts_with($reflection->getNamespaceName(), 'GeminiLabs\BlackBar')) {
+						//     continue; // skip blackbar callbacks
+						// }
+					}
+					$this->hooks[$action]['callbacks'][$priority][] = sprintf('%s::%s', $object, $method);
+				} elseif (is_object($callback['function'])) {
+					$this->hooks[$action]['callbacks'][$priority][] = get_class($callback['function']);
+				} else {
+					$this->hooks[$action]['callbacks'][$priority][] = $callback['function'];
+				}
+				++$this->hooks[$action]['callbacks_count'];
+			}
+		}
+	}
 
-    public function entries(): array
-    {
-        if (class_exists('Debug_Bar_Slow_Actions')) {
-            return [];
-        }
-        if (!empty($this->entries) || empty($this->hooks)) {
-            return $this->entries;
-        }
-        foreach ($this->hooks as $action => $data) {
-            $total = $this->totalTimeForHook($data);
-            $this->hooks[$action]['total'] = $total;
-            $this->totalTime += $total;
-            $this->totalActions += $data['count'];
-            $this->callbacksForHook($action);
-        }
-        uasort($this->hooks, [$this, 'sortByTime']);
-        $this->entries = array_slice($this->hooks, 0, 50); // return the 50 slowest actions
-        return $this->entries;
-    }
+	public function entries(): array
+	{
+		if (class_exists('Debug_Bar_Slow_Actions')) {
+			return [];
+		}
+		if (!empty($this->entries) || empty($this->hooks)) {
+			return $this->entries;
+		}
+		foreach ($this->hooks as $action => $data) {
+			$total = $this->totalTimeForHook($data);
+			$this->hooks[$action]['total'] = $total;
+			$this->totalTime += $total;
+			$this->totalActions += $data['count'];
+			$this->callbacksForHook($action);
+		}
+		uasort($this->hooks, [$this, 'sortByTime']);
+		$this->entries = array_slice($this->hooks, 0, 50); // return the 50 slowest actions
+		return $this->entries;
+	}
 
-    public function hasEntries(): bool
-    {
-        return !empty($this->entries());
-    }
+	public function hasEntries(): bool
+	{
+		return !empty($this->entries());
+	}
 
-    public function id(): string
-    {
-        return 'glbb-actions';
-    }
+	public function id(): string
+	{
+		return 'glbb-actions';
+	}
 
-    public function isVisible(): bool
-    {
-        return true;
-    }
+	public function isVisible(): bool
+	{
+		return true;
+	}
 
-    public function label(): string
-    {
-        $label = __('Hooks', 'blackbar');
-        if (class_exists('Debug_Bar_Slow_Actions')) {
-            return $label;
-        }
-        $this->entries(); // calculates the totalTime
-        if ($this->totalTime > 0) {
-            $label = sprintf('%s (<span class="glbb-actions-time">%.2f</span> %s)', $label, $this->totalTime, __('ms', 'blackbar'));
-        }
-        return $label;
-    }
+	public function label(): string
+	{
+		$label = __('Hooks', 'blackbar');
+		if (class_exists('Debug_Bar_Slow_Actions')) {
+			return $label;
+		}
+		$this->entries(); // calculates the totalTime
+		if ($this->totalTime > 0) {
+			$label = sprintf('%s (<span class="glbb-actions-time">%.2f</span> %s)', $label, $this->totalTime, __('ms', 'blackbar'));
+		}
+		return $label;
+	}
 
-    public function render(): void
-    {
-        $this->app->render('panels/actions', ['actions' => $this]);
-    }
+	public function render(): void
+	{
+		$this->app->render('panels/actions', ['actions' => $this]);
+	}
 
-    public function startTimer(): void
-    {
-        if (!isset($this->hooks[current_filter()])) {
-            $this->hooks[current_filter()] = [
-                'callbacks' => [],
-                'count' => 0,
-                'stack' => [],
-                'time' => [],
-            ];
-            add_action(current_filter(), [$this, 'stopTimer'], 9999); // @phpstan-ignore-line
-        }
-        ++$this->hooks[current_filter()]['count'];
-        array_push($this->hooks[current_filter()]['stack'], ['start' => microtime(true)]);
-    }
+	public function startTimer(): void
+	{
+		if (!isset($this->hooks[current_filter()])) {
+			$this->hooks[current_filter()] = [
+				'callbacks' => [],
+				'count' => 0,
+				'stack' => [],
+				'time' => [],
+			];
+			add_action(current_filter(), [$this, 'stopTimer'], 9999); // @phpstan-ignore-line
+		}
+		++$this->hooks[current_filter()]['count'];
+		array_push($this->hooks[current_filter()]['stack'], ['start' => microtime(true)]);
+	}
 
-    /**
-     * @param mixed $filteredValue
-     * @return mixed
-     */
-    public function stopTimer($filteredValue = null)
-    {
-        $time = array_pop($this->hooks[current_filter()]['stack']);
-        $time['stop'] = microtime(true);
-        array_push($this->hooks[current_filter()]['time'], $time);
-        return $filteredValue; // In case this was a filter.
-    }
+	/**
+	 * @param mixed $filteredValue
+	 * @return mixed
+	 */
+	public function stopTimer($filteredValue = null)
+	{
+		$time = array_pop($this->hooks[current_filter()]['stack']);
+		$time['stop'] = microtime(true);
+		array_push($this->hooks[current_filter()]['time'], $time);
+		return $filteredValue; // In case this was a filter.
+	}
 
-    public function totalTimeForHook(array $data): float
-    {
-        $total = 0;
-        foreach ($data['time'] as $time) {
-            $total += ($time['stop'] - $time['start']) * 1000;
-        }
-        return (float) $total;
-    }
+	public function totalTimeForHook(array $data): float
+	{
+		$total = 0;
+		foreach ($data['time'] as $time) {
+			$total += ($time['stop'] - $time['start']) * 1000;
+		}
+		return (float) $total;
+	}
 
-    protected function sortByTime(array $a, array $b): int
-    {
-        if ($a['total'] !== $b['total']) {
-            return ($a['total'] > $b['total']) ? -1 : 1;
-        }
-        return 0;
-    }
+	protected function sortByTime(array $a, array $b): int
+	{
+		if ($a['total'] !== $b['total']) {
+			return ($a['total'] > $b['total']) ? -1 : 1;
+		}
+		return 0;
+	}
 }

Spacing +53 added lines, -53 removed lines

@@ -27,70 +27,70 @@ 
      */
     protected $totalTime;
 
-    public function __construct(Application $app)
+    public function __construct( Application $app )
     {
         $this->app = $app;
-        $this->entries = [];
-        $this->hooks = [];
+        $this->entries = [ ];
+        $this->hooks = [ ];
         $this->totalActions = 0;
         $this->totalTime = (float) 0;
     }
 
-    public function callbacksForHook(string $action): void
+    public function callbacksForHook( string $action ): void
     {
         global $wp_filter;
-        if (!array_key_exists($action, $this->hooks)) {
+        if( !array_key_exists( $action, $this->hooks ) ) {
             return;
         }
-        $this->hooks[$action]['callbacks_count'] = 0;
-        foreach ($wp_filter[$action] as $priority => $callbacks) {
-            if (!array_key_exists($priority, $this->hooks[$action]['callbacks'])) {
-                $this->hooks[$action]['callbacks'][$priority] = [];
+        $this->hooks[ $action ][ 'callbacks_count' ] = 0;
+        foreach( $wp_filter[ $action ] as $priority => $callbacks ) {
+            if( !array_key_exists( $priority, $this->hooks[ $action ][ 'callbacks' ] ) ) {
+                $this->hooks[ $action ][ 'callbacks' ][ $priority ] = [ ];
             }
-            foreach ($callbacks as $callback) {
-                if (is_array($callback['function']) && 2 === count($callback['function'])) {
-                    list($object, $method) = $callback['function'];
-                    if (is_object($object)) {
-                        $object = get_class($object);
+            foreach( $callbacks as $callback ) {
+                if( is_array( $callback[ 'function' ] ) && 2 === count( $callback[ 'function' ] ) ) {
+                    list( $object, $method ) = $callback[ 'function' ];
+                    if( is_object( $object ) ) {
+                        $object = get_class( $object );
                         // $reflection = new \ReflectionClass($object);
                         // if (str_starts_with($reflection->getNamespaceName(), 'GeminiLabs\BlackBar')) {
                         //     continue; // skip blackbar callbacks
                         // }
                     }
-                    $this->hooks[$action]['callbacks'][$priority][] = sprintf('%s::%s', $object, $method);
-                } elseif (is_object($callback['function'])) {
-                    $this->hooks[$action]['callbacks'][$priority][] = get_class($callback['function']);
+                    $this->hooks[ $action ][ 'callbacks' ][ $priority ][ ] = sprintf( '%s::%s', $object, $method );
+                } elseif( is_object( $callback[ 'function' ] ) ) {
+                    $this->hooks[ $action ][ 'callbacks' ][ $priority ][ ] = get_class( $callback[ 'function' ] );
                 } else {
-                    $this->hooks[$action]['callbacks'][$priority][] = $callback['function'];
+                    $this->hooks[ $action ][ 'callbacks' ][ $priority ][ ] = $callback[ 'function' ];
                 }
-                ++$this->hooks[$action]['callbacks_count'];
+                ++$this->hooks[ $action ][ 'callbacks_count' ];
             }
         }
     }
 
     public function entries(): array
     {
-        if (class_exists('Debug_Bar_Slow_Actions')) {
-            return [];
+        if( class_exists( 'Debug_Bar_Slow_Actions' ) ) {
+            return [ ];
         }
-        if (!empty($this->entries) || empty($this->hooks)) {
+        if( !empty( $this->entries ) || empty( $this->hooks ) ) {
             return $this->entries;
         }
-        foreach ($this->hooks as $action => $data) {
-            $total = $this->totalTimeForHook($data);
-            $this->hooks[$action]['total'] = $total;
+        foreach( $this->hooks as $action => $data ) {
+            $total = $this->totalTimeForHook( $data );
+            $this->hooks[ $action ][ 'total' ] = $total;
             $this->totalTime += $total;
-            $this->totalActions += $data['count'];
-            $this->callbacksForHook($action);
+            $this->totalActions += $data[ 'count' ];
+            $this->callbacksForHook( $action );
         }
-        uasort($this->hooks, [$this, 'sortByTime']);
-        $this->entries = array_slice($this->hooks, 0, 50); // return the 50 slowest actions
+        uasort( $this->hooks, [ $this, 'sortByTime' ] );
+        $this->entries = array_slice( $this->hooks, 0, 50 ); // return the 50 slowest actions
         return $this->entries;
     }
 
     public function hasEntries(): bool
     {
-        return !empty($this->entries());
+        return !empty( $this->entries() );
     }
 
     public function id(): string
@@ -105,62 +105,62 @@ 
 
     public function label(): string
     {
-        $label = __('Hooks', 'blackbar');
-        if (class_exists('Debug_Bar_Slow_Actions')) {
+        $label = __( 'Hooks', 'blackbar' );
+        if( class_exists( 'Debug_Bar_Slow_Actions' ) ) {
             return $label;
         }
         $this->entries(); // calculates the totalTime
-        if ($this->totalTime > 0) {
-            $label = sprintf('%s (<span class="glbb-actions-time">%.2f</span> %s)', $label, $this->totalTime, __('ms', 'blackbar'));
+        if( $this->totalTime > 0 ) {
+            $label = sprintf( '%s (<span class="glbb-actions-time">%.2f</span> %s)', $label, $this->totalTime, __( 'ms', 'blackbar' ) );
         }
         return $label;
     }
 
     public function render(): void
     {
-        $this->app->render('panels/actions', ['actions' => $this]);
+        $this->app->render( 'panels/actions', [ 'actions' => $this ] );
     }
 
     public function startTimer(): void
     {
-        if (!isset($this->hooks[current_filter()])) {
-            $this->hooks[current_filter()] = [
-                'callbacks' => [],
+        if( !isset( $this->hooks[ current_filter() ] ) ) {
+            $this->hooks[ current_filter() ] = [
+                'callbacks' => [ ],
                 'count' => 0,
-                'stack' => [],
-                'time' => [],
+                'stack' => [ ],
+                'time' => [ ],
             ];
-            add_action(current_filter(), [$this, 'stopTimer'], 9999); // @phpstan-ignore-line
+            add_action( current_filter(), [ $this, 'stopTimer' ], 9999 ); // @phpstan-ignore-line
         }
-        ++$this->hooks[current_filter()]['count'];
-        array_push($this->hooks[current_filter()]['stack'], ['start' => microtime(true)]);
+        ++$this->hooks[ current_filter() ][ 'count' ];
+        array_push( $this->hooks[ current_filter() ][ 'stack' ], [ 'start' => microtime( true ) ] );
     }
 
     /**
      * @param mixed $filteredValue
      * @return mixed
      */
-    public function stopTimer($filteredValue = null)
+    public function stopTimer( $filteredValue = null )
     {
-        $time = array_pop($this->hooks[current_filter()]['stack']);
-        $time['stop'] = microtime(true);
-        array_push($this->hooks[current_filter()]['time'], $time);
+        $time = array_pop( $this->hooks[ current_filter() ][ 'stack' ] );
+        $time[ 'stop' ] = microtime( true );
+        array_push( $this->hooks[ current_filter() ][ 'time' ], $time );
         return $filteredValue; // In case this was a filter.
     }
 
-    public function totalTimeForHook(array $data): float
+    public function totalTimeForHook( array $data ): float
     {
         $total = 0;
-        foreach ($data['time'] as $time) {
-            $total += ($time['stop'] - $time['start']) * 1000;
+        foreach( $data[ 'time' ] as $time ) {
+            $total += ( $time[ 'stop' ] - $time[ 'start' ] ) * 1000;
         }
         return (float) $total;
     }
 
-    protected function sortByTime(array $a, array $b): int
+    protected function sortByTime( array $a, array $b ): int
     {
-        if ($a['total'] !== $b['total']) {
-            return ($a['total'] > $b['total']) ? -1 : 1;
+        if( $a[ 'total' ] !== $b[ 'total' ] ) {
+            return ( $a[ 'total' ] > $b[ 'total' ] ) ? -1 : 1;
         }
         return 0;
     }

views/panels/queries.php

Spacing +11 added lines, -11 removed lines

@@ -1,28 +1,28 @@
-<?php defined('WPINC') || die; ?>
+<?php defined( 'WPINC' ) || die; ?>
 
-<?php if (!defined('SAVEQUERIES') || !SAVEQUERIES) : ?>
+<?php if( !defined( 'SAVEQUERIES' ) || !SAVEQUERIES ) : ?>
     <table>
         <tbody>
             <tr>
-                <td><a href="https://wordpress.org/documentation/article/debugging-in-wordpress/#savequeries">SAVEQUERIES</a> <?= _x('must be enabled to view SQL queries', 'SAVEQUERIES', 'blackbar'); ?>.</td>
+                <td><a href="https://wordpress.org/documentation/article/debugging-in-wordpress/#savequeries">SAVEQUERIES</a> <?= _x( 'must be enabled to view SQL queries', 'SAVEQUERIES', 'blackbar' ); ?>.</td>
             </tr>
         </tbody>
     </table>
 <?php else : ?>
     <form method="get" class="glbb-queries-filter">
-        <input class="glbb-input" type="text" name="glbb_query_filter" id="glbb_query_filter" placeholder="<?= __('Find Queries Containing', 'blackbar'); ?>">
-        <input class="glbb-input glbb-input-small" type="text" name="glbb_query_min_time" id="glbb_query_min_time" placeholder="<?= __('Minimum Execution Time', 'blackbar'); ?>">
+        <input class="glbb-input" type="text" name="glbb_query_filter" id="glbb_query_filter" placeholder="<?= __( 'Find Queries Containing', 'blackbar' ); ?>">
+        <input class="glbb-input glbb-input-small" type="text" name="glbb_query_min_time" id="glbb_query_min_time" placeholder="<?= __( 'Minimum Execution Time', 'blackbar' ); ?>">
         <select class="glbb-input" name="glbb_query_sort" id="glbb_query_sort">
-            <option value><?= __('Sort by execution order', 'blackbar'); ?></option>
-            <option value="time"><?= __('Sort by execution time', 'blackbar'); ?></option>
+            <option value><?= __( 'Sort by execution order', 'blackbar' ); ?></option>
+            <option value="time"><?= __( 'Sort by execution time', 'blackbar' ); ?></option>
         </select>
     </form>
     <table class="glbb-queries-table">
         <tbody>
-            <?php foreach ($queries->entries() as $index => $entry) : ?>
-                <tr data-index="<?= esc_attr($index); ?>" data-time="<?= esc_attr($entry['time']); ?>">
-                    <td class="glbb-small"><?= esc_html($entry['time']); ?></td>
-                    <td><pre><code class="language-sql"><?= esc_html($entry['sql']); ?></code></pre></td>
+            <?php foreach( $queries->entries() as $index => $entry ) : ?>
+                <tr data-index="<?= esc_attr( $index ); ?>" data-time="<?= esc_attr( $entry[ 'time' ] ); ?>">
+                    <td class="glbb-small"><?= esc_html( $entry[ 'time' ] ); ?></td>
+                    <td><pre><code class="language-sql"><?= esc_html( $entry[ 'sql' ] ); ?></code></pre></td>
                 </tr>
             <?php endforeach; ?>
         </tbody>

Braces +5 added lines, -2 removed lines

@@ -8,9 +8,12 @@
             </tr>
         </tbody>
     </table>
-<?php else : ?>
+<?php else {
+	: ?>
     <form method="get" class="glbb-queries-filter">
-        <input class="glbb-input" type="text" name="glbb_query_filter" id="glbb_query_filter" placeholder="<?= __('Find Queries Containing', 'blackbar'); ?>">
+        <input class="glbb-input" type="text" name="glbb_query_filter" id="glbb_query_filter" placeholder="<?= __('Find Queries Containing', 'blackbar');
+}
+?>">
         <input class="glbb-input glbb-input-small" type="text" name="glbb_query_min_time" id="glbb_query_min_time" placeholder="<?= __('Minimum Execution Time', 'blackbar'); ?>">
         <select class="glbb-input" name="glbb_query_sort" id="glbb_query_sort">
             <option value><?= __('Sort by execution order', 'blackbar'); ?></option>