From ac444cba221832c012c0435fdc8339fe9f37febb Mon Sep 17 00:00:00 2001
From: Jake Vanderwerf <get@jakevanderwerf.ca>
Date: Mon, 11 May 2026 18:35:04 +0000
Subject: [PATCH] =Some changes to the CRUD.js editing, timeline post configuration

---
 inc/ui/CRUDSkeleton.php |  168 ++++++++++++++++++++++++++++++++++++-------------------
 1 files changed, 110 insertions(+), 58 deletions(-)

diff --git a/inc/ui/CRUDSkeleton.php b/inc/ui/CRUDSkeleton.php
index 481f0e1..b5e977a 100644
--- a/inc/ui/CRUDSkeleton.php
+++ b/inc/ui/CRUDSkeleton.php
@@ -1,8 +1,10 @@
 <?php
 namespace JVBase\ui;
 
+use JVBase\base\Site;
 use JVBase\managers\UserTermsManager;
 use JVBase\meta\Form;
+use JVBase\registrar\Registrar;
 use WP_User;
 
 if (!defined('ABSPATH')) {
@@ -117,7 +119,6 @@
 	protected ?array $uploaderConfig = null;
 
 	// Data
-	protected $dataSourceCallback = null;
 	protected array $templates = [];
 
 	// UI Options
@@ -128,6 +129,7 @@
 	protected array $customDateRanges = [];
 	protected array $additionalClasses = [];
 
+	protected ?Registrar $registrar;
 	public function __construct() {
 		$this->icon = jvbDefaultIcon();
 		$this->user = wp_get_current_user();
@@ -148,6 +150,12 @@
 	 */
 	public function content(string $type, string $singular, string $plural): self {
 		$this->dataType = $type;
+		$registrar = Registrar::getInstance($type);
+		if ($registrar) {
+			$this->registrar = Registrar::getInstance($type)??null;
+			$this->sections = $this->registrar->getSections();
+		}
+
 		$this->singular = $singular;
 		$this->plural = $plural;
 		return $this;
@@ -213,13 +221,17 @@
 	 */
 	public function addTaxonomyFilter(array $taxonomies, ?string $limit = null): self {
 		foreach($taxonomies as $taxonomy) {
-			$this->taxonomies[$taxonomy] = [
-				'type'	=> 'taxonomy',
-				'taxonomy'=> $taxonomy,
-				'limit'	=> $limit,
-				'label'	=> JVB_TAXONOMY[$taxonomy]['plural']??'',
-				'icon'	=> JVB_TAXONOMY[$taxonomy]['icon']??''
-			];
+			$registrar = Registrar::getInstance($taxonomy);
+
+			if ($registrar) {
+				$this->taxonomies[$taxonomy] = [
+					'type'	=> 'taxonomy',
+					'taxonomy'=> $taxonomy,
+					'limit'	=> $limit,
+					'label'	=> $registrar->getPlural(),
+					'icon'	=> $registrar->getIcon()
+				];
+			}
 		}
 
 		return $this;
@@ -227,8 +239,8 @@
 
 	protected function taxConfig(string $taxonomy, string $label = ''):array
 	{
-		$isVerified = jvbUserIsVerified();
-		$label = ($label === '') ? JVB_TAXONOMY[$taxonomy]['plural'] : $label;
+		$isVerified = $this->userIsVerified();
+		$label = ($label === '') ? Registrar::getInstance($taxonomy)->getPlural() : $label;
 		return [
 			'type'		=> 'taxonomy',
 			'label'		=> $label,
@@ -239,6 +251,13 @@
 		];
 	}
 
+	protected function userIsVerified():bool
+	{
+		$membership = Site::membership();
+
+		return !($membership && $membership->has('member_verified')) || current_user_can('skip_moderation');
+	}
+
 	public function addSearch():self
 	{
 		$this->hasSearch = true;
@@ -336,7 +355,7 @@
 		}
 
 		$this->timelineSharedFields = array_keys(array_filter($this->fields, function ($field) {
-			if (!array_key_exists('for_all', $field) || $field['for_all'] === false){
+			if (!array_key_exists('for_all', $field) || $field['for_all'] === false || is_null($field['for_all'])){
 				return true;
 			}
 			return false;
@@ -350,7 +369,6 @@
 			return false;
 		}));
 
-
 		$all = array_merge($this->timelineUniqueFields, $this->timelineSharedFields);
 		$this->nonTimelineFields = array_filter($this->fields, function ($field) use ($all) {
 			return !in_array($field, $all);
@@ -451,15 +469,6 @@
 	}
 
 	/**
-	 * Set the data source callback
-	 * Callback should accept filters and return array of items
-	 */
-	public function dataSource(callable $callback): self {
-		$this->dataSourceCallback = $callback;
-		return $this;
-	}
-
-	/**
 	 * Add a custom template
 	 */
 	public function addTemplate(string $name, string $template): self {
@@ -613,7 +622,7 @@
 		}
 		?>
 		<details class="all-filters col start" data-ignore>
-			<summary>Filters <button hidden data-action="clear-filters" data-ignore><?=jvbIcon('x')?><span>Clear Filters</span></span></button></summary>
+			<summary>Filters</summary>
 			<?php
 
 			$this->renderSearch();
@@ -627,6 +636,7 @@
 			?>
 			<button data-action="refresh" data-ignore><?=jvbIcon('arrows-clockwise')?><span>Hard Refresh</span></span></button>
 		</details>
+		<button hidden data-action="clear-filters" data-ignore hidden><?=jvbIcon('x')?><span>Clear Filters</span></span></button>
 		<?php
 	}
 
@@ -876,8 +886,8 @@
 			foreach ($terms as $term) {
 				$out .= sprintf(
 					'<option value="%s">%s</option>',
-					esc_attr($term['term_id']),
-					esc_html($term['name'])
+					esc_attr(is_object($term) ? $term->term_id : $term['term_id']),
+					esc_html(is_object($term) ? $term->name : $term['name'])
 				);
 			}
 			$out .= '</select></div>';
@@ -892,9 +902,9 @@
 	 */
 	protected function getCommonTerms(string $taxonomy, ?string $limit = null):array {
 		if ($limit) {
-			if ($limit === 'user') {
+			if (Site::has('membership') && $limit === 'user') {
 				$manager = new UserTermsManager();
-				return $manager->getUserTerms($this->user_id, $taxonomy);
+				return $manager->fetchUserTerms($this->user_id, $taxonomy);
 			} else {
 				$limit = (int)$limit;
 			}
@@ -973,9 +983,12 @@
 					<option value="<?=$control?>"<?=$disabled?>><?=$label?></option>
 					<?php
 				}
-				foreach ($this->taxonomies as $taxonomy => $config) {
+
+				foreach ($this->taxonomies as $taxonomy =>$config) {
+					$registrar = Registrar::getInstance($taxonomy);
+					if (!$registrar) continue;
 					?>
-					<option value="tax-<?=$taxonomy?>" data-type="selector" data-single="<?=JVB_TAXONOMY[$taxonomy]['singular']?>" data-plural="<?=JVB_TAXONOMY[$taxonomy]['plural']?>" data-taxonomy="<?=$taxonomy?>">Add to <?= JVB_TAXONOMY[$taxonomy]['singular']??$config['label'] ?></option>
+					<option value="tax-<?=$taxonomy?>" data-type="selector" data-single="<?=$registrar->getSingular()?>" data-plural="<?=$registrar->getPlural()?>" data-taxonomy="<?=$taxonomy?>">Add to <?= $registrar->getSingular() ?></option>
 					<?php
 				}
 				?>
@@ -1043,7 +1056,7 @@
 			}, ARRAY_FILTER_USE_KEY);
 
 			echo '<template class="timelineItem">';
-			echo Form::renderImagePreview(null, ['fields' => $temp]);
+			echo Form::renderImagePreview(null, $temp);
 			echo '</template>';
 		}
 		if (!array_key_exists('empty', $templates)) {
@@ -1543,19 +1556,15 @@
 			<input type="hidden" name="form-id" value="<?=uniqid('new-')?>" />
 			<input type="hidden" name="content" value="<?=$this->dataType?>" />
 			<div class="fields">
-				<div class="field-group radio-options row" data-field="post_status" data-field-type="radio">
-					<span>Status:</span>
-					<?php
-					$this->getApplicableStatuses('edit');
-					?>
-				</div>
-				<?php if (!$this->userCanPublish) { ?>
-					<p class="description">Your account needs to be verified before you can publish content.</p>
-				<?php }
+				<?php
+				if (!empty($this->statuses)) {
+					echo Form::render('post_status', '', $this->getStatusFieldConfig('edit-'));
+				}
 
 				if (!empty($this->sections)) {
 					$tabs = [];
-					foreach ($this->sections as $slug => $config) {
+					foreach ($this->sections as $config) {
+						$slug = $config['slug'];
 						$section = [];
 						if (array_key_exists('icon', $config)) {
 							$section = [
@@ -1567,10 +1576,6 @@
 							'content' => '',
 							'description' => $config['description']??'',
 						], $section);
-						$icon = jvbSectionIcon($slug);
-						if ($icon !== '') {
-							$tabs[$slug]['icon'] = $icon;
-						}
 					}
 				} else {
 					$tabs = false;
@@ -1593,13 +1598,15 @@
 				}
 
 				if ($this->isTimeline) {
-					$temp = array_filter($fields, function ($field) {
-						return in_array($field, $this->timelineUniqueFields);
+					$temp = array_filter($fields, function ($field) use ($fields) {
+						return in_array($field, $this->timelineUniqueFields) && (!array_key_exists('hidden', $fields[$field]) || $fields[$field]['hidden'] === false);
 					}, ARRAY_FILTER_USE_KEY);
 					$config = [
 						'type'		=> 'upload',
 						'subtype'	=> 'timeline',
-						'data'		=> 'timeline',
+						'multiple'	=> true,
+						'limit'		=> 0,
+						'data'		=> ['timeline'],
 						'label'		=> 'Progression',
 						'fields'	=> $temp
 					];
@@ -1609,24 +1616,31 @@
 							if (in_array($field['type'], ['taxonomy', 'selector'])) {
 								$field = array_merge($field, $this->taxConfig($field['taxonomy'], $field['label']));
 							}
-							$content .= Form::render($slug, '', $field);
+							if (!array_key_exists('hidden', $field) || $field['hidden'] === false) {
+								$content .= Form::render($slug, '', $field);
+							}
 						}
 					}
 
 
-					$content .= Form::render('timeline', '', $config);
+					$content .= Form::render('timeline_gallery', '', $config);
 
-					$tabs['progression']['content'] = $content;
+					if ($tabs) {
+						$tabs['progression']['content'] = $content;
+					} else {
+						echo $content;
+					}
+
 					$fields = $this->nonTimelineFields;
 				}
 				foreach ($fields as $n => $config) {
+					if (in_array($config['type'], ['taxonomy', 'selector'])) {
+						$config = array_merge($config, $this->taxConfig($config['taxonomy'], $config['label']));
+					}
 					if ($tabs) {
 						$section = (array_key_exists('section', $config)) ? $config['section'] : 'basic';
-						$tabs[$section]['content'] .= Form::render($n,'', $config);
+						$tabs[$section]['content'] .= Form::render($n, '', $config);
 					} else {
-						if (in_array($config['type'], ['taxonomy', 'selector'])) {
-							$config = array_merge($config, $this->taxConfig($config['taxonomy'], $config['label']));
-						}
 						echo Form::render($n, '', $config);
 					}
 				}
@@ -1662,12 +1676,9 @@
 			<p class="description">You can unselect items by clicking the image here.</p>
 			<p class="hint"><strong>IMPORTANT: </strong> Whatever changes you make here will be applied to all selected <?=$this->plural?>.</p>
 			<div class="fields">
-				<div class="field-group radio-options row">
-					<?php
-					$this->getApplicableStatuses('bulk-');
-					?>
-				</div>
 				<?php
+				echo Form::render('post_status', '', $this->getStatusFieldConfig('bulk-'));
+
 				if (!empty($this->taxonomies)) {
 					?>
 					<div class="taxonomies">
@@ -1704,7 +1715,46 @@
 		);
 	}
 
+	protected function getStatusFieldConfig(string $prefix): array
+	{
+		$options = [];
+		foreach ($this->statuses as $status) {
+			if ($status === 'all' || !array_key_exists($status, $this->allowedStatuses)) {
+				continue;
+			}
+			$config = $this->allowedStatuses[$status];
+
+			if (in_array($status, ['future', 'past'])) {
+				if ($status === 'future') {
+					$status = 'publish';
+					$config = ['icon' => 'eye', 'label' => 'Live'];
+				} else {
+					continue;
+				}
+			}
+
+			$options[$status] = [
+				'label'    => $config['label'],
+				'icon'     => $config['icon'],
+				'disabled' => ($status === 'publish' && !$this->userCanPublish),
+			];
+		}
+
+		return [
+			'type'       => 'radio',
+			'label'      => 'Status',
+			'options'    => $options,
+			'inputClass' => 'btn',
+			'idPrefix'   => $prefix,
+			'class'      => 'radio-options row',
+			'hint'       => !$this->userCanPublish
+				? 'Your account needs to be verified before you can publish content.'
+				: '',
+		];
+	}
+
 	protected function getApplicableStatuses(string $prefix) {
+		ob_start();
 		foreach ($this->statuses as $status) {
 			if ($status === 'all' || !array_key_exists($status, $this->allowedStatuses)) {
 				continue;
@@ -1734,5 +1784,7 @@
 			</label>
 			<?php
 		}
+		$out = ob_get_clean();
+		echo Form::fieldWrap('post_status', $out, ['type'=>'group']);
 	}
 }

--
Gitblit v1.10.0