From 3baf3d2545ba6ece6b74a64c0def59bd0774cf54 Mon Sep 17 00:00:00 2001
From: Jake Vanderwerf <get@jakevanderwerf.ca>
Date: Wed, 10 Jun 2026 16:34:12 +0000
Subject: [PATCH] =Laid the groundwork for an improved DashboardManager.php setup. Have to put it aside so I can get the dang Northeh done though.
---
inc/managers/SEO/BreadcrumbManager.php | 137 ++++++++++++++++++++++++++++-----------------
1 files changed, 85 insertions(+), 52 deletions(-)
diff --git a/inc/managers/SEO/BreadcrumbManager.php b/inc/managers/SEO/BreadcrumbManager.php
index 8d3288e..41f82ac 100644
--- a/inc/managers/SEO/BreadcrumbManager.php
+++ b/inc/managers/SEO/BreadcrumbManager.php
@@ -3,7 +3,7 @@
use JVBase\managers\Cache;
use JVBase\registrar\Registrar;
-use JVBase\utility\Features;
+use JVBase\base\Site;
use WP_Post;
use WP_Term;
@@ -25,10 +25,10 @@
private function __construct()
{
$this->cache = Cache::for('breadcrumbs', MONTH_IN_SECONDS)->connect('post')->connect('taxonomy')->connect('user');
- $this->cache->flush();
if (JVB_TESTING) {
$this->cache->flush();
}
+ $this->cache->flush();
}
public static function getInstance(): self
@@ -50,17 +50,23 @@
return [];
}
+
switch (true) {
+
case is_singular():
$key = get_queried_object_id();
break;
+ case is_tax():
+ $obj = get_queried_object();
+ $key = $obj->taxonomy;
+ break;
case is_post_type_archive():
$obj = get_queried_object();
$key = $obj->name;
break;
- case is_tax():
+ case is_home():
$obj = get_queried_object();
- $key = $obj->taxonomy;
+ $key = $obj->post_type;
break;
default:
$key = 'home';
@@ -93,7 +99,7 @@
$obj = get_queried_object();
if (is_tax()) {
$crumbs = $this->addTaxonomyCrumbs($crumbs, $obj);
- } elseif (is_singular()) {
+ } elseif (is_singular() || is_home()) {
$crumbs = $this->addArchiveCrumbs($crumbs, $obj);
$hierarchy = $this->addSingularCrumbs($crumbs, $obj);
$crumbs = $crumbs + $hierarchy;
@@ -117,25 +123,37 @@
$content = is_array($registrar->registrar->for) ? $registrar->registrar->for[0] : $registrar->registrar->for;
$contentRegistrar = Registrar::getInstance($content);
- $crumbs[] = [
- 'name' => $contentRegistrar->getConfig('breadcrumbs')['title']??$contentRegistrar->getPlural(),
- 'url' => get_post_type_archive_link(jvbCheckBase($content)),
- ];
- $crumbs[] = [
- 'name' => 'By ' . $registrar->getSingular(),
- 'url' => false,
- ];
+ if($contentRegistrar && $contentRegistrar->hasFeature('show_directory')) {
+ $directory = JVB()->directories();
+ if ($directory && !empty($directory->directories($content)??[])){
+ $crumbs[] = [
+ 'name' => $directory->directories($content)['title'],
+ 'url' =>$directory->directories($content)['url']
+ ];
+ }
+ } else {
+ $crumbs[] = [
+ 'name' => $contentRegistrar->getConfig('breadcrumbs')['title']??$contentRegistrar->getPlural(),
+ 'url' => get_post_type_archive_link(jvbCheckBase($content)),
+ ];
+ }
+// $crumbs[] = [
+// 'name' => 'By ' . $registrar->getSingular(),
+// 'url' => false,
+// ];
}
}
// Add directory if exists
- if ($registrar && $registrar->hasFeature('directory')) {
- $directory = JVB()->directories()?->directories($tax);
- $crumbs[] = [
- 'name' => $directory['title'],
- 'url' => $directory['url']
- ];
+ if ($registrar && $registrar->hasFeature('show_directory')) {
+ $directory = JVB()->directories();
+ if ($directory && !empty($directory->directories($tax)??[])) {
+ $crumbs[] = [
+ 'name' => $directory->directories($tax)['title'],
+ 'url' =>$directory->directories($tax)['url']
+ ];
+ }
}
// Add term hierarchy
@@ -150,18 +168,10 @@
// Add directory if exists
$content = jvbNoBase($post->post_type);
$registrar = Registrar::getInstance($content);
+ $crumbConfig = false;
if ($registrar){
$crumbConfig = $registrar->getConfig('breadcrumbs');
}
- if($registrar && $registrar->hasFeature('show_directory')) {
- $directory = JVB()->directories()?->directories($content)??[];
- if (!empty($directory)) {
- $crumbs[] = [
- 'name' => $directory['title'],
- 'url' =>$directory['url']
- ];
- }
- }
// Handle directory posts specially
if (JVB()->directories() && JVB()->directories()->isDirectory()) {
@@ -171,7 +181,7 @@
if ($pos['title'] == 'Map') {
$crumbs[] = [
'name' => 'Tattoo Shops',
- 'url' => JVB()->directories()?->directories(BASE.'shop')['url']
+ 'url' => JVB()->directories()->directories(BASE.'shop')['url']
];
}
@@ -197,20 +207,39 @@
*/
private function addArchiveCrumbs(array $crumbs, object $obj): array
{
+ if (is_singular('page') || is_home()) {
+ return $crumbs;
+ }
$type = is_singular() ? $obj->post_type : $obj->name;
$name = jvbNoBase($type);
$registrar = Registrar::getInstance($name);
- if (Features::forSite()->has('is_directory') && $name === 'directory') {
+
+ if($registrar && $registrar->hasFeature('show_directory') && JVB()->directories()) {
+ $directory = JVB()->directories();
+
+ if ($directory && !empty($directory->directories($name)??[])){
+ $crumbs[] = [
+ 'name' => $directory->directories($name)['title'],
+ 'url' =>$directory->directories($name)['url']
+ ];
+ }
+ } elseif (Site::has('is_directory') && $name === 'directory') {
+
$crumbs[] = [
'name' => JVB()->directories()->referAs(true),
'url' => get_post_type_archive_link($type)
];
- } elseif (is_post_type_archive() && $registrar && $registrar->hasFeature('show_directory')) {
-
+ } elseif ($registrar) {
$crumbs[] = [
- 'name' => $registrar->getConfig('breadcrumb')['title'] ?? $registrar->getPlural(),
- 'url' => get_post_type_archive_link($type)
+ 'name' => $registrar->getConfig('breadcrumbs')['title'] ?? $registrar->getPlural(),
+ 'url' => get_post_type_archive_link($type)
+ ];
+ } else {
+ $postTypeObject = get_post_type_object($type);
+ $crumbs[] = [
+ 'name' => $postTypeObject->label,
+ 'url' => get_post_type_archive_link($type)
];
}
@@ -366,29 +395,33 @@
}
}
- public function addTaxToCrumbs(array $crumbs, string $taxonomy):array
+ public function addTaxToCrumbs(array $crumbs, string|array $taxonomy):array
{
$ID = get_the_ID();
- $taxonomy = jvbCheckBase($taxonomy);
- $terms = get_the_terms($ID, $taxonomy);
- if ($terms && !is_wp_error($terms)) {
- $term = $terms[0];
- $ancestors = get_ancestors($term->term_id, $taxonomy, 'taxonomy');
- $ancestors = array_reverse($ancestors);
- foreach ($ancestors as $ancestor) {
- $aTerm = get_term($ancestor, $taxonomy);
- if ($aTerm && !is_wp_error($aTerm)) {
- $crumbs[] = [
- 'name' => $aTerm->name,
- 'url' => get_term_link($ancestor, $taxonomy)
- ];
+ $taxonomies = is_string($taxonomy) ? [$taxonomy] : $taxonomy;
+ foreach ($taxonomies as $tax) {
+ $taxonomy = jvbCheckBase($tax);
+ $terms = get_the_terms($ID, $taxonomy);
+ if ($terms && !is_wp_error($terms)) {
+ $term = $terms[0];
+ $ancestors = get_ancestors($term->term_id, $taxonomy, 'taxonomy');
+ $ancestors = array_reverse($ancestors);
+ foreach ($ancestors as $ancestor) {
+ $aTerm = get_term($ancestor, $taxonomy);
+ if ($aTerm && !is_wp_error($aTerm)) {
+ $crumbs[] = [
+ 'name' => $aTerm->name,
+ 'url' => get_term_link($ancestor, $taxonomy)
+ ];
+ }
}
+ $crumbs[] = [
+ 'name' => html_entity_decode($term->name),
+ 'url' => get_term_link($term, $taxonomy)
+ ];
}
- $crumbs[] = [
- 'name' => html_entity_decode($term->name),
- 'url' => get_term_link($term, $taxonomy)
- ];
}
+
return $crumbs;
}
}
--
Gitblit v1.10.0