From 266aa37c48222993bf7bdad6834e31bd08736f5e Mon Sep 17 00:00:00 2001
From: Jake Vanderwerf <get@jakevanderwerf.ca>
Date: Sat, 23 May 2026 21:22:07 +0000
Subject: [PATCH] =Post type archive meta title fix
---
inc/managers/SEO/BreadcrumbManager.php | 204 ++++++++++++++++++++++++++++++++-------------------
1 files changed, 128 insertions(+), 76 deletions(-)
diff --git a/inc/managers/SEO/BreadcrumbManager.php b/inc/managers/SEO/BreadcrumbManager.php
index 0a23c5c..299d396 100644
--- a/inc/managers/SEO/BreadcrumbManager.php
+++ b/inc/managers/SEO/BreadcrumbManager.php
@@ -1,8 +1,9 @@
<?php
namespace JVBase\managers\SEO;
-use JVBase\managers\CacheManager;
-use JVBase\utility\Features;
+use JVBase\managers\Cache;
+use JVBase\registrar\Registrar;
+use JVBase\base\Site;
use WP_Post;
use WP_Term;
@@ -18,14 +19,14 @@
*/
class BreadcrumbManager
{
- private CacheManager $cache;
+ private Cache $cache;
private static ?self $instance = null;
private function __construct()
{
- $this->cache = CacheManager::for('breadcrumbs', MONTH_IN_SECONDS)->connectTo('all');
+ $this->cache = Cache::for('breadcrumbs', MONTH_IN_SECONDS)->connect('post')->connect('taxonomy')->connect('user');
if (JVB_TESTING) {
- $this->cache->clear();
+ $this->cache->flush();
}
}
@@ -48,7 +49,9 @@
return [];
}
+
switch (true) {
+
case is_singular():
$key = get_queried_object_id();
break;
@@ -60,21 +63,22 @@
$obj = get_queried_object();
$key = $obj->taxonomy;
break;
+ case is_home():
+ $obj = get_queried_object();
+ $key = $obj->post_type;
+ break;
default:
$key = 'home';
break;
}
- $crumbs = $this->cache->get($key);
- if ($crumbs !== false) {
- return $crumbs;
- }
-
- $crumbs = $this->buildCrumbs();
- $crumbs = apply_filters('jvbBreadcrumbs',$crumbs);
- $this->cache->set($key, $crumbs);
-
- return $crumbs;
+ return $this->cache->remember(
+ $key,
+ function() {
+ $crumbs = $this->buildCrumbs();
+ return apply_filters('jvbBreadcrumbs',$crumbs);
+ }
+ );
}
/**
@@ -86,7 +90,7 @@
// Always start with home
$crumbs[] = [
- 'name' => 'Home',
+ 'name' => get_bloginfo('name'),
'icon' => jvbIcon('house'),
'url' => get_home_url(),
];
@@ -94,7 +98,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;
@@ -111,28 +115,44 @@
private function addTaxonomyCrumbs(array $crumbs, WP_Term $term): array
{
$tax = jvbNoBase($term->taxonomy);
- $config = Features::getConfig($tax, 'term');
-
+ $registrar = Registrar::getInstance($tax);
// Add parent content archive if taxonomy is for single content type
- if (count($config['for_content']) === 1) {
- $contentConfig = JVB_CONTENT[$config['for_content'][0]];
- $crumbs[] = [
- 'name' => $contentConfig['breadcrumb'] ?? $contentConfig['plural'],
- 'url' => get_post_type_archive_link(jvbCheckBase($config['for_content'][0])),
- ];
- $crumbs[] = [
- 'name' => 'By ' . $config['singular'],
- 'url' => false,
- ];
+ if ($registrar) {
+ if (count($registrar->registrar->for) === 1){
+ $content = is_array($registrar->registrar->for) ? $registrar->registrar->for[0] : $registrar->registrar->for;
+ $contentRegistrar = Registrar::getInstance($content);
+
+ 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 (Features::forTaxonomy($tax)->has('directory')) {
- $directory = JVB()->directories()?->directories($tax);
- $crumbs[] = [
- 'name' => $directory['title'],
- 'url' => $directory['url']
- ];
+ if ($registrar && $registrar->hasFeature('directory')) {
+ $directory = JVB()->directories();
+ if ($directory && !empty($directory->directories($tax))) {
+ $crumbs[] = [
+ 'name' => $directory['title'],
+ 'url' => $directory['url']
+ ];
+ }
}
// Add term hierarchy
@@ -146,25 +166,31 @@
{
// Add directory if exists
$content = jvbNoBase($post->post_type);
- if(Features::forContent($content)->has('show_directory')) {
- $directory = JVB()->directories()->getDirectoryList()[$content]??[];
- if (!empty($directory)) {
+ $registrar = Registrar::getInstance($content);
+ $crumbConfig = false;
+ if ($registrar){
+ $crumbConfig = $registrar->getConfig('breadcrumbs');
+ }
+
+ if($registrar && $registrar->hasFeature('show_directory')) {
+ $directory = JVB()->directories();
+ if ($directory && !empty($directory->directories($content)??[])){
$crumbs[] = [
- 'name' => $directory['title'],
- 'url' =>$directory['url']
+ 'name' => $directory->directories($content)['title'],
+ 'url' =>$directory->directories($content)['url']
];
}
}
// Handle directory posts specially
- if (JVB()->directories()->isDirectory()) {
+ if (JVB()->directories() && JVB()->directories()->isDirectory()) {
$pos = jvbGetDirectoryInfo();
if (!empty($pos)) {
// Special case for map
if ($pos['title'] == 'Map') {
$crumbs[] = [
'name' => 'Tattoo Shops',
- 'url' => JVB()->directories()?->directories(BASE.'shop')['url']
+ 'url' => JVB()->directories()->directories(BASE.'shop')['url']
];
}
@@ -174,9 +200,9 @@
];
}
} else {
- $name = jvbNoBase($post->post_type);
- if (Features::forContent($name)->has('addCrumb')) {
- $this->addTaxToCrumbs($crumbs, JVB_CONTENT[$name]['addCrumb']);
+
+ if ($registrar && !empty($crumbConfig['addCrumb'])) {
+ $crumbs = $this->addTaxToCrumbs($crumbs, $crumbConfig['addCrumb']);
}
// Add post hierarchy
$crumbs = array_merge($crumbs, $this->buildPostHierarchy($post));
@@ -190,18 +216,38 @@
*/
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);
- if (Features::forSite()->has('is_directory') && $name === 'directory') {
+ $registrar = Registrar::getInstance($name);
+
+ 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() || !Features::forContent($name)->has('show_directory')) && array_key_exists($name, JVB_CONTENT)) {
+ } elseif ($registrar) {
$crumbs[] = [
- 'name' => JVB_CONTENT[$name]['breadcrumb'] ?? JVB_CONTENT[$name]['plural'],
- '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)
];
}
@@ -215,7 +261,7 @@
{
$url = get_term_link($term->term_id);
array_unshift($crumbs, [
- 'name' => $term->name,
+ 'name' => html_entity_decode($term->name),
'url' => $url,
'id' => $term->term_id,
]);
@@ -320,17 +366,19 @@
$items = [];
$position = 1;
+ global $wp;
+ $current = home_url( add_query_arg( $_GET, $wp->request ) );
foreach ($crumbs as $crumb) {
// Schema requires a URL
if ($crumb['url'] === false) {
- $crumb['url'] = get_permalink();
+ $crumb['url'] = $current;
}
$items[] = [
- '@type' => 'ListItem',
- 'position' => $position,
- 'name' => $crumb['name'],
- 'item' => $crumb['url'],
+ '@type' => 'ListItem',
+ '@id' => $crumb['url'],
+ 'position' => $position,
+ 'name' => $crumb['name'],
];
$position++;
@@ -338,7 +386,7 @@
return [
'@type' => 'BreadcrumbList',
- '@id' => get_permalink() . '/#breadcrumbs',
+ '@id' => $current . '/#breadcrumbs',
'itemListElement' => $items
];
}
@@ -349,35 +397,39 @@
public function invalidateCache(?int $objectId = null): void
{
if ($objectId) {
- $this->cache->delete($objectId);
+ $this->cache->forget($objectId);
} else {
- $this->cache->clear();
+ $this->cache->flush();
}
}
- 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' => $term->name,
- 'url' => get_term_link($term, $taxonomy)
- ];
}
+
return $crumbs;
}
}
--
Gitblit v1.10.0