From c204185ae86a98994f80010abf35a190c9406739 Mon Sep 17 00:00:00 2001
From: Jake Vanderwerf <get@jakevanderwerf.ca>
Date: Sun, 12 Jul 2026 18:08:19 +0000
Subject: [PATCH] =Refactor of Integrations.php. Separated different functionality into traits that classes can use to add that functionality. Hopefully will make maintaining it a little easier. Still have to finish up, as well as refactoring the individual classes to utilize the new system.
---
inc/managers/SEO/BreadcrumbManager.php | 53 +++++++++++++++++------------------------------------
1 files changed, 17 insertions(+), 36 deletions(-)
diff --git a/inc/managers/SEO/BreadcrumbManager.php b/inc/managers/SEO/BreadcrumbManager.php
index 224569f..2993c98 100644
--- a/inc/managers/SEO/BreadcrumbManager.php
+++ b/inc/managers/SEO/BreadcrumbManager.php
@@ -28,6 +28,7 @@
if (JVB_TESTING) {
$this->cache->flush();
}
+ $this->cache->flush();
}
public static function getInstance(): self
@@ -50,27 +51,12 @@
}
- switch (true) {
-
- case is_singular():
- $key = get_queried_object_id();
- break;
- case is_post_type_archive():
- $obj = get_queried_object();
- $key = $obj->name;
- break;
- case is_tax():
- $obj = get_queried_object();
- $key = $obj->taxonomy;
- break;
- case is_home():
- $obj = get_queried_object();
- $key = $obj->post_type;
- break;
- default:
- $key = 'home';
- break;
- }
+ $key = match(true) {
+ is_singular() => get_queried_object_id(),
+ is_tax() => get_queried_object()->taxonomy,
+ is_post_type_archive() => get_queried_object()->name,
+ default => 'home'
+ };
return $this->cache->remember(
$key,
@@ -96,6 +82,9 @@
];
$obj = get_queried_object();
+ if (is_null($obj)) {
+ return $crumbs;
+ }
if (is_tax()) {
$crumbs = $this->addTaxonomyCrumbs($crumbs, $obj);
} elseif (is_singular() || is_home()) {
@@ -145,12 +134,12 @@
// Add directory if exists
- if ($registrar && $registrar->hasFeature('directory')) {
+ if ($registrar && $registrar->hasFeature('show_directory')) {
$directory = JVB()->directories();
- if ($directory && !empty($directory->directories($tax))) {
+ if ($directory && !empty($directory->directories($tax)??[])) {
$crumbs[] = [
- 'name' => $directory['title'],
- 'url' => $directory['url']
+ 'name' => $directory->directories($tax)['title'],
+ 'url' =>$directory->directories($tax)['url']
];
}
}
@@ -172,16 +161,6 @@
$crumbConfig = $registrar->getConfig('breadcrumbs');
}
- if($registrar && $registrar->hasFeature('show_directory')) {
- $directory = JVB()->directories();
- if ($directory && !empty($directory->directories($content)??[])){
- $crumbs[] = [
- 'name' => $directory->directories($content)['title'],
- 'url' =>$directory->directories($content)['url']
- ];
- }
- }
-
// Handle directory posts specially
if (JVB()->directories() && JVB()->directories()->isDirectory()) {
$pos = jvbGetDirectoryInfo();
@@ -224,8 +203,9 @@
$registrar = Registrar::getInstance($name);
- if($registrar && $registrar->hasFeature('show_directory')) {
+ if($registrar && $registrar->hasFeature('show_directory') && JVB()->directories()) {
$directory = JVB()->directories();
+
if ($directory && !empty($directory->directories($name)??[])){
$crumbs[] = [
'name' => $directory->directories($name)['title'],
@@ -233,6 +213,7 @@
];
}
} elseif (Site::has('is_directory') && $name === 'directory') {
+
$crumbs[] = [
'name' => JVB()->directories()->referAs(true),
'url' => get_post_type_archive_link($type)
--
Gitblit v1.10.0