From a24a06002081ad71a78ffeff9072725ba39cf121 Mon Sep 17 00:00:00 2001
From: Jake Vanderwerf <get@jakevanderwerf.ca>
Date: Tue, 17 Feb 2026 20:05:31 +0000
Subject: [PATCH] =minor changes, particularly around the JVB_CHILD_URL pattern
---
inc/managers/queue/executors/ContentExecutor.php | 45 +++++++++++++++++----------------------------
1 files changed, 17 insertions(+), 28 deletions(-)
diff --git a/inc/managers/queue/executors/ContentExecutor.php b/inc/managers/queue/executors/ContentExecutor.php
index 7d4db57..c6d99af 100644
--- a/inc/managers/queue/executors/ContentExecutor.php
+++ b/inc/managers/queue/executors/ContentExecutor.php
@@ -76,9 +76,11 @@
$results = [];
$errors = [];
+ $success = [];
$timelineParents = [];
$timelineStatus = [];
$timelineSharedFields = [];
+ $newPostsMap = [];
foreach ($posts as $id => $postData) {
try {
@@ -94,16 +96,14 @@
]);
if (!$newId || is_wp_error($newId)) {
+ $errors[$id] = 'Could not create post';
$progress->failItem($id, 'Could not create post');
continue;
}
+ $newPostsMap[$id] = $newId;
$this->savePostFields($newId, $postData);
- $results[$id] = [
- 'success' => true,
- 'new_id' => $newId,
- 'processed_fields' => array_keys($postData)
- ];
+ $success[$newId] = array_keys($postData);
$progress->advance();
continue;
}
@@ -117,7 +117,6 @@
$this->savePostFields((int)$id, $postData);
-
if (Features::forContent($content)->has('is_timeline')) {
$post = get_post((int)$id);
$parentId = $post->post_parent > 0 ? $post->post_parent : $post->ID;
@@ -142,19 +141,12 @@
}
}
}
- $results[$id] = [
- 'success' => true,
- 'processed_fields' => array_keys($postData)
- ];
- $progress->advance();
+ $success[$id] = array_keys($postData);
+ $progress->advance();
} catch (Exception $e) {
$progress->failItem($id, $e->getMessage());
$errors[$id] = $e->getMessage();
- $results[$id] = [
- 'success' => false,
- 'error' => $e->getMessage()
- ];
}
}
@@ -188,15 +180,13 @@
$outcome = count($errors) === count($posts) ? 'failed' : 'partial';
}
-
-
-
return new Result(
outcome: $outcome,
result: [
- 'posts' => $results,
+ 'posts' => $success,
'errors' => $errors,
- 'updated_count' => count(array_filter($results, fn($r) => $r['success'] ?? false)),
+ 'new_posts' => $newPostsMap,
+ 'updated_count' => count($success),
'failed_count' => count($errors)
]
);
@@ -215,9 +205,9 @@
return true;
}
- $meta = Meta::forPost($postId);
- $meta->setAll($allowedFields);
- return true;
+ return Meta::forPost($postId)
+ ->setAll($allowedFields)
+ ->save();
}
// ─────────────────────────────────────────────────────────────
@@ -318,7 +308,7 @@
foreach ($posts as $index => $post) {
$meta = Meta::forPost($post->ID);
if ($index === 0) {
- $meta->set('timeline', '', false);
+ $meta->set('timeline', '');
$previousPost = $post;
continue; // Parent has no timeline
}
@@ -329,7 +319,7 @@
if ($timeline) {
$termId = $this->getOrCreateTerm($timeline, 'timeline');
if ($termId) {
- $success = $meta->set('timeline', $termId, false);
+ $success = $meta->set('timeline', $termId);
}
}
}
@@ -337,7 +327,7 @@
if ($lastKey === $index) {
$latestTimestamp = strtotime($post->post_date);
}
-
+ $meta->save();
$previousPost = $post;
}
@@ -458,8 +448,7 @@
}
foreach ($children as $child) {
- $childMeta = Meta::forPost($child);
- $result = $childMeta->setAll($values, false);
+ Meta::forPost($child)->setAll($values)->save(false);
}
}
}
--
Gitblit v1.10.0