From 5b50afdcafd150186ea5c7b0e1cad98065590b35 Mon Sep 17 00:00:00 2001
From: Jake Vanderwerf <get@jakevanderwerf.ca>
Date: Fri, 28 Aug 2026 18:44:28 +0000
Subject: [PATCH] =A few more phpStorm suggestions, mainly in PHPDocs
---
inc/managers/queue/executors/ContentExecutor.php | 31 ++++++++++++++++++-------------
1 files changed, 18 insertions(+), 13 deletions(-)
diff --git a/inc/managers/queue/executors/ContentExecutor.php b/inc/managers/queue/executors/ContentExecutor.php
index 6b6aa1a..562a004 100644
--- a/inc/managers/queue/executors/ContentExecutor.php
+++ b/inc/managers/queue/executors/ContentExecutor.php
@@ -1,10 +1,14 @@
<?php
namespace JVBase\managers\queue\executors;
-use JVBase\managers\queue\{Executor, Operation, Progress, Result, Storage};
+use DateTime;
+use JVBase\managers\queue\{Executor, Operation, Progress, Result};
+use JVBase\managers\Errors;
+use JVBase\managers\Roles;
use JVBase\meta\Meta;
use JVBase\registrar\Registrar;
use Exception;
+use WP_Post;
if (!defined('ABSPATH')) {
exit;
@@ -22,8 +26,6 @@
];
private int $userId;
- private string $postType;
- private array $fields = [];
public function execute(Operation $operation, Progress $progress): Result
{
@@ -43,7 +45,7 @@
};
} catch (Exception $e) {
- JVB()->error()->log(
+ Errors::getInstance()->log(
'[ContentExecutor]:execute',
$e->getMessage(),
[
@@ -125,7 +127,7 @@
// Send notification
// if (jvbSiteHasNotifications()) {
-// JVB()->notification()->addNotification(
+// NotificationManager::getInstance()->addNotification(
// $this->userId,
// 'content_update_complete',
// null,
@@ -134,8 +136,8 @@
// }
$outcome = 'success';
- if (!empty($errors)) {
- $outcome = count($errors) === count($posts) ? 'failed' : 'partial';
+ if (!empty($results['errors'])) {
+ $outcome = count($results['success']) === count($posts) ? 'failed' : 'partial';
}
return new Result(
@@ -360,7 +362,10 @@
}
}
- private function calculateTimeline(\WP_POST $previous, \WP_POST $current): ?string
+ /**
+ * @throws \DateMalformedStringException
+ */
+ private function calculateTimeline(WP_POST $previous, WP_POST $current): ?string
{
$previousDate = strtotime($previous->post_date);
$currentDate = strtotime($current->post_date);
@@ -378,8 +383,8 @@
return $weeks === 1 ? '1 Week' : $weeks . ' Weeks';
}
- $previousDateTime = new \DateTime($previous->post_date);
- $currentDateTime = new \DateTime($current->post_date);
+ $previousDateTime = new DateTime($previous->post_date);
+ $currentDateTime = new DateTime($current->post_date);
$interval = $previousDateTime->diff($currentDateTime);
$months = ($interval->y * 12) + $interval->m;
@@ -391,7 +396,7 @@
return ($months === 1) ? '1 Month' : $months . ' Months';
}
- private function swapTimelineParent(\WP_Post $oldParent, \WP_Post $newParent, array $allPosts): void
+ private function swapTimelineParent(WP_Post $oldParent, WP_Post $newParent, array $allPosts): void
{
// Swap titles and content
$originalTitle = $oldParent->post_title;
@@ -415,7 +420,7 @@
]);
// Clear timeline taxonomy from new parent
- wp_set_object_terms($newParent->ID, [], BASE . 'timeline', false);
+ wp_set_object_terms($newParent->ID, [], BASE . 'timeline');
// Update all other posts to new parent
foreach ($allPosts as $index => $post) {
@@ -596,7 +601,7 @@
{
error_log('Handling term '.$ID.' with data: '.print_r($data, true));
//Existing term update
- if ($registrar->hasFeature('is_ownable') && (!JVB()->roles()->isOwner($this->userId, $ID) && !JVB()->roles()->isManager($this->userId, $ID))) {
+ if ($registrar->hasFeature('is_ownable') && (!Roles::getInstance()()->isOwner($this->userId, $ID) && !Roles::getInstance()()->isManager($this->userId, $ID))) {
error_log('Term is ownable. User does not own this term.');
$progress->failItem($ID, 'No permission to modify this term');
$results['errors'][$ID] = 'No permission';
--
Gitblit v1.10.0