=A few more phpStorm suggestions, mainly in PHPDocs

Jake Vanderwerf

2026-08-28 5b50afdcafd150186ea5c7b0e1cad98065590b35

 inc/managers/queue/Queue.php

@@ -4,21 +4,32 @@
   exit;
}

use JVBase\managers\CustomTable;
use Exception;
use JVBase\Instance;
use JVBase\managers\AdminPages;
use JVBase\managers\Errors;
use JVBase\rest\routes\QueueRoutes;
use Throwable;
use WP_Error;
use WP_REST_Request;
use WP_REST_Response;

class Queue
{
   use Instance;
   private Storage $storage;
   private Processor $processor;
   private TypeRegistry $registry;
   public Executor $executor;
   private Locker $locker;

   public function __construct()
   private QueueRoutes $routes;

   private function __construct()
   {
      //Load routes
      $this->routes = new QueueRoutes();

      $this->storage = new Storage();
      $this->registry = new TypeRegistry();
      $this->locker = new Locker();
@@ -66,13 +77,15 @@
    * @param int $userId User ID
    * @param array $data Request data
    * @param array $options {
    *     @type string $priority     'low', 'normal', 'high'
    *     @type int $delay           Seconds to delay processing
    *     @type string $scheduled    Specific datetime to process
    *     @type array|string $depends_on  Operation IDs this depends on
    *     @type string|array $chunk_key   Key(s) to chunk (overrides registry)
    *     @type int $chunk_size      Chunk size (overrides registry)
    * @type string $priority 'low', 'normal', 'high'
    * @type int $delay Seconds to delay processing
    * @type string $scheduled Specific datetime to process
    * @type array|string $depends_on Operation IDs this depends on
    * @type string|array $chunk_key Key(s) to chunk (overrides registry)
    * @type int $chunk_size Chunk size (overrides registry)
    * }
    * @throws Throwable
    * @throws Throwable
    */
   public function add(string $type, int $userId, array $data, array $options = []): array|WP_Error
   {
@@ -95,8 +108,8 @@
            'updated_existing' => false,
         ];

      } catch (\Exception $e) {
         JVB()->error()->log('queue', $e->getMessage(), $data, 'high');
      } catch (Exception $e) {
         Errors::getInstance()->log('queue', $e->getMessage(), $data, 'high');
         return new WP_Error('queue_failed', $e->getMessage());
      }
   }
@@ -104,7 +117,7 @@
   /**
    * Try to merge incoming operation into an existing pending/scheduled one.
    * Returns result array if merged, null if not.
    * @throws \Throwable
    * @throws Throwable
    */
   private function tryMerge(Operation $incoming): ?array
   {
@@ -173,7 +186,7 @@
   public function maintenance(): void
   {
      $this->locker->withLock(function () {
         $this->storage->resetStuckOperations(30);
         $this->storage->resetStuckOperations();
      });
      $this->runQueueOnShutdown();
   }
@@ -196,7 +209,7 @@
   /**
    * Get a specific value from an operation - backwards compatibility
    */
   public function getOperationValue(string $id, string $column, bool $decodeJson = true): mixed
   public function getOperationValue(string $id, string $column, bool $decodeJson = true): \static|string|int|array|null
   {
      $op = $this->get($id);
      if (!$op) {
@@ -343,7 +356,7 @@
      // Use provided operation_id or generate one
      $op->id = !empty($options['operation_id'])
         ? $options['operation_id']
         : 'u' . $userId . '_' . uniqid('op_');
         : self::generateID($userId);

      $op->type = $type;
      $op->userId = $userId;
@@ -422,7 +435,7 @@

   public function registerAdminAction():void
   {
      $admin = JVB()->admin();
      $admin = AdminPages::getInstance();
      $admin->registerAction(
         'Restart Stuck Operations',
         'restart-stuck-operations',
@@ -436,11 +449,12 @@
         'infinity'
      );
   }

   /**
    * @param WP_REST_Response $response
    * @param WP_REST_Request $request
    * @param string $action
    *
    * @return bool|WP_REST_Response
    * @return WP_REST_Response|bool
    */
   public function adminActionFilter(WP_REST_Response $response, WP_REST_Request $request, string $action):WP_REST_Response|bool
   {
@@ -463,4 +477,9 @@
            return $response;
      }
   }

   public static function generateId(int $userID):string
   {
      return 'u' . $userID . '_' . uniqid('op_');
   }
}