fix(php): toc fallback to raw content when filtered has no headings

When plugins like Thrive Visual Editor transform content for
non-logged users, headings may be removed from the filtered content.
This fix uses raw post_content as fallback when filtered content
has no headings but raw content does.

Also removes temporary debug logging added for diagnosis.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
FrankZamora
2025-12-06 23:08:24 -06:00
parent 83d113d669
commit b509b1a2b4
5 changed files with 16 additions and 76 deletions

View File

@@ -43,30 +43,20 @@ final class TableOfContentsRenderer implements RendererInterface
{ {
$data = $component->getData(); $data = $component->getData();
// DEBUG TOC: Log all visibility checks
$isLoggedIn = is_user_logged_in();
$debugPrefix = "TOC DEBUG [" . ($isLoggedIn ? "LOGGED" : "GUEST") . "]";
if (!$this->isEnabled($data)) { if (!$this->isEnabled($data)) {
error_log("{$debugPrefix}: SKIP - isEnabled=false");
return ''; return '';
} }
$shouldShow = PageVisibilityHelper::shouldShow(self::COMPONENT_NAME); if (!PageVisibilityHelper::shouldShow(self::COMPONENT_NAME)) {
if (!$shouldShow) {
error_log("{$debugPrefix}: SKIP - PageVisibilityHelper::shouldShow=false");
return ''; return '';
} }
$tocItems = $this->generateTocItems($data); $tocItems = $this->generateTocItems($data);
if (empty($tocItems)) { if (empty($tocItems)) {
error_log("{$debugPrefix}: SKIP - tocItems empty");
return ''; return '';
} }
error_log("{$debugPrefix}: RENDER - passed all checks, items=" . count($tocItems));
$css = $this->generateCSS($data); $css = $this->generateCSS($data);
$html = $this->buildHTML($data, $tocItems); $html = $this->buildHTML($data, $tocItems);
$script = $this->buildScript($data); $script = $this->buildScript($data);
@@ -117,31 +107,26 @@ final class TableOfContentsRenderer implements RendererInterface
{ {
global $post; global $post;
// DEBUG: Track content processing
$isLoggedIn = is_user_logged_in();
$debugPrefix = "TOC generateTocFromContent [" . ($isLoggedIn ? "LOGGED" : "GUEST") . "]";
if (!$post || empty($post->post_content)) { if (!$post || empty($post->post_content)) {
error_log("{$debugPrefix}: SKIP - no post or empty content");
return []; return [];
} }
error_log("{$debugPrefix}: post_id={$post->ID}, raw_content_length=" . strlen($post->post_content)); // Intentar primero con contenido filtrado (respeta shortcodes, etc.)
// Check if raw content has headings
$rawHeadingCount = preg_match_all('/<h[2-6][^>]*>/i', $post->post_content, $rawMatches);
error_log("{$debugPrefix}: raw_headings_count={$rawHeadingCount}");
$content = apply_filters('the_content', $post->post_content); $content = apply_filters('the_content', $post->post_content);
error_log("{$debugPrefix}: filtered_content_length=" . strlen($content)); // Verificar si el contenido filtrado tiene headings
$hasFilteredHeadings = preg_match('/<h[2-6][^>]*>/i', $content);
// Check if filtered content has headings // FIX: Si el contenido filtrado no tiene headings pero el raw si,
$filteredHeadingCount = preg_match_all('/<h[2-6][^>]*>/i', $content, $filteredMatches); // usar el contenido raw. Esto ocurre cuando plugins como Thrive
error_log("{$debugPrefix}: filtered_headings_count={$filteredHeadingCount}"); // transforman el contenido para usuarios no logueados.
if (!$hasFilteredHeadings) {
// Log first 500 chars of filtered content for diagnosis $hasRawHeadings = preg_match('/<h[2-6][^>]*>/i', $post->post_content);
error_log("{$debugPrefix}: filtered_content_preview=" . substr(strip_tags($content), 0, 300)); if ($hasRawHeadings) {
// Usar wpautop para dar formato basico al contenido raw
$content = wpautop($post->post_content);
}
}
$dom = new DOMDocument(); $dom = new DOMDocument();
libxml_use_internal_errors(true); libxml_use_internal_errors(true);
@@ -157,8 +142,6 @@ final class TableOfContentsRenderer implements RendererInterface
$headings = $xpath->query($xpathQuery); $headings = $xpath->query($xpathQuery);
error_log("{$debugPrefix}: headings_found=" . $headings->length);
if ($headings->length === 0) { if ($headings->length === 0) {
return []; return [];
} }

View File

@@ -36,16 +36,9 @@ final class EvaluateComponentVisibilityUseCase
*/ */
public function execute(string $componentName): bool public function execute(string $componentName): bool
{ {
// DEBUG: Log for TOC only
$debugToc = ($componentName === 'table-of-contents');
// Paso 1: Verificar visibilidad por tipo de pagina // Paso 1: Verificar visibilidad por tipo de pagina
$visibleByPageType = $this->pageVisibilityUseCase->execute($componentName); $visibleByPageType = $this->pageVisibilityUseCase->execute($componentName);
if ($debugToc) {
error_log("EvaluateComponentVisibility [{$componentName}]: visibleByPageType=" . ($visibleByPageType ? "true" : "false"));
}
if (!$visibleByPageType) { if (!$visibleByPageType) {
return false; return false;
} }
@@ -53,10 +46,6 @@ final class EvaluateComponentVisibilityUseCase
// Paso 2: Verificar exclusiones // Paso 2: Verificar exclusiones
$isExcluded = $this->exclusionsUseCase->execute($componentName); $isExcluded = $this->exclusionsUseCase->execute($componentName);
if ($debugToc) {
error_log("EvaluateComponentVisibility [{$componentName}]: isExcluded=" . ($isExcluded ? "true" : "false"));
}
// Mostrar si NO esta excluido // Mostrar si NO esta excluido
return !$isExcluded; return !$isExcluded;
} }

View File

@@ -28,27 +28,15 @@ final class EvaluatePageVisibilityUseCase
{ {
$config = $this->visibilityRepository->getVisibilityConfig($componentName); $config = $this->visibilityRepository->getVisibilityConfig($componentName);
// DEBUG: Log for TOC only
$debugToc = ($componentName === 'table-of-contents');
$usingDefaults = false;
if (empty($config)) { if (empty($config)) {
// Usar defaults especificos por componente si existen // Usar defaults especificos por componente si existen
$config = VisibilityDefaults::getForComponent($componentName); $config = VisibilityDefaults::getForComponent($componentName);
$usingDefaults = true;
} }
$pageType = $this->pageTypeDetector->detect(); $pageType = $this->pageTypeDetector->detect();
$visibilityField = $pageType->toVisibilityField(); $visibilityField = $pageType->toVisibilityField();
$fieldValue = $config[$visibilityField] ?? true;
$result = $this->toBool($fieldValue);
if ($debugToc) { return $this->toBool($config[$visibilityField] ?? true);
error_log("EvaluatePageVisibility [{$componentName}]: pageType=" . $pageType->value() . ", field={$visibilityField}, fieldValue=" . var_export($fieldValue, true) . ", result=" . ($result ? "true" : "false") . ", usingDefaults=" . ($usingDefaults ? "true" : "false"));
error_log("EvaluatePageVisibility [{$componentName}]: config=" . json_encode($config));
}
return $result;
} }
private function toBool(mixed $value): bool private function toBool(mixed $value): bool

View File

@@ -43,15 +43,7 @@ final class PageVisibilityHelper
$container = DIContainer::getInstance(); $container = DIContainer::getInstance();
$useCase = $container->getEvaluateComponentVisibilityUseCase(); $useCase = $container->getEvaluateComponentVisibilityUseCase();
$result = $useCase->execute($componentName); return $useCase->execute($componentName);
// DEBUG: Log visibility evaluation
if ($componentName === 'table-of-contents') {
$isLoggedIn = is_user_logged_in();
error_log("PageVisibilityHelper DEBUG [{$componentName}] [" . ($isLoggedIn ? "LOGGED" : "GUEST") . "]: result=" . ($result ? "true" : "false"));
}
return $result;
} }
/** /**

View File

@@ -15,18 +15,6 @@ final class WordPressPageTypeDetector implements PageTypeDetectorInterface
{ {
public function detect(): PageType public function detect(): PageType
{ {
// DEBUG: Log all checks for diagnosis
$isLoggedIn = is_user_logged_in();
$debugData = [
'is_front_page' => is_front_page(),
'is_home' => is_home(),
'is_single' => is_single(),
'is_page' => is_page(),
'is_search' => is_search(),
'is_archive' => is_archive(),
];
error_log("PageTypeDetector DEBUG [" . ($isLoggedIn ? "LOGGED" : "GUEST") . "]: " . json_encode($debugData));
if ($this->isHome()) { if ($this->isHome()) {
return PageType::home(); return PageType::home();
} }