From a9654f00c8fbc9934289d761843ddd071025013f Mon Sep 17 00:00:00 2001
From: Jake Vanderwerf <get@jakevanderwerf.ca>
Date: Fri, 28 Aug 2026 17:20:28 +0000
Subject: [PATCH] =PHPStorm Code Inspection: fixed class reference errors (from getting rid of JVB() and replacing with direct class::getInstance; as well as general code sniffing, etc

---
 inc/forms/PostSelector.php |   11 ++++++-----
 1 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/inc/forms/PostSelector.php b/inc/forms/PostSelector.php
index 5fa2026..c68191d 100644
--- a/inc/forms/PostSelector.php
+++ b/inc/forms/PostSelector.php
@@ -2,6 +2,7 @@
 namespace JVBase\forms;
 
 use JVBase\managers\Cache;
+use JVBase\managers\Roles;
 use WP_REST_Request;
 use WP_REST_Response;
 use WP_Query;
@@ -23,7 +24,7 @@
 	public function __construct(string $post_type, array $config = [])
 	{
 		$this->post_type = $post_type;
-		$this->cache = Cache::for(jvbNoBase($post_type), WEEK_IN_SECONDS)->connect('post', true);
+		$this->cache = Cache::for(jvbNoBase($post_type), 'post',WEEK_IN_SECONDS);
 
 		$this->config = wp_parse_args($config, [
 			'multiple' => true,
@@ -184,8 +185,8 @@
 		}
 
 		// Check cache
-		$cache_key = $this->cache->generateKey($args);
-		$cached_result = $this->cache->get($cache_key);
+
+		$cached_result = $this->cache->get($args);
 		if ($cached_result) {
 			return new WP_REST_Response($cached_result);
 		}
@@ -223,7 +224,7 @@
 		];
 
 		// Cache the result
-		$this->cache->set($cache_key, $result, 300); // 5 minute cache
+		$this->cache->set($args, $result, 300); // 5 minute cache
 
 		return new WP_REST_Response($result);
 	}
@@ -237,7 +238,7 @@
 			'methods' => 'GET',
 			'callback' => [new self('post'), 'searchPosts'],
 			'permission_callback' => function($request) {
-				return JVB()->roles()->checkRole(get_current_user(), $request->get_param('post_type'));
+				return Roles::getInstance()()->checkRole(get_current_user(), $request->get_param('post_type'));
 			},
 			'args' => [
 				'post_type' => [

--
Gitblit v1.10.0