| @@ -87,16 +87,34 @@ | ||
| 87 | 87 | 'Book', 'Movie', 'Service', 'ImageObject', 'VideoObject', |
| 88 | 88 | ]; |
| 89 | 89 | |
| 90 | 90 | /** |
| 91 | - * Gutenberg FAQ block name. | |
| 91 | + * Types that may carry a `breadcrumb` property. | |
| 92 | + * | |
| 93 | + * Schema.org limits `breadcrumb` to WebPage and its subtypes. Attaching it | |
| 94 | + * to a BlogPosting, Product or Recipe primary fails validation with | |
| 95 | + * "Unexpected property" on every URL; the standalone BreadcrumbList node is | |
| 96 | + * emitted either way (#693). | |
| 97 | + * | |
| 98 | + * @since 2.7.0 | |
| 99 | + * @var array<int,string> | |
| 92 | 100 | */ |
| 93 | - private const FAQ_BLOCK = 'thinkrank/faq'; | |
| 101 | + private const BREADCRUMB_TYPES = [ | |
| 102 | + 'WebPage', 'AboutPage', 'CheckoutPage', 'CollectionPage', 'ContactPage', | |
| 103 | + 'FAQPage', 'ItemPage', 'MedicalWebPage', 'ProfilePage', 'QAPage', | |
| 104 | + 'RealEstateListing', 'SearchResultsPage', 'MediaGallery', 'ImageGallery', | |
| 105 | + 'VideoGallery', | |
| 106 | + ]; | |
| 94 | 107 | |
| 95 | 108 | /** |
| 96 | - * Elementor FAQ widget name. | |
| 109 | + * The FAQ surfaces themselves are read by FAQ_Content, which owns their | |
| 110 | + * names, their stored shapes and their per-surface schema toggles. Only the | |
| 111 | + * *foreign* FAQ producers are listed below, because standing down for them | |
| 112 | + * is this class's decision rather than a fact about where ThinkRank keeps | |
| 113 | + * its own content (#767). | |
| 114 | + * | |
| 115 | + * @since 2.10.1 | |
| 97 | 116 | */ |
| 98 | - private const FAQ_WIDGET = 'thinkrank-faq'; | |
| 99 | 117 | |
| 100 | 118 | /** |
| 101 | 119 | * Third-party Elementor widgets that publish their own FAQPage. |
| 102 | 120 | * |
| @@ -111,8 +129,22 @@ | ||
| 111 | 129 | 'eael-adv-accordion' => 'eael_adv_accordion_faq_schema_show', |
| 112 | 130 | ]; |
| 113 | 131 | |
| 114 | 132 | /** |
| 133 | + * Bricks elements that publish their own FAQPage. | |
| 134 | + * | |
| 135 | + * Bricks is a theme, not a plugin, and its accordions are core elements | |
| 136 | + * rather than a third-party add-on — so unlike FOREIGN_FAQ_WIDGETS this is | |
| 137 | + * a plain list: they share one gate, the `faqSchema` setting, and the | |
| 138 | + * per-element part of the check is whether the element has usable items | |
| 139 | + * (see bricks_element_publishes_faq()). | |
| 140 | + * | |
| 141 | + * @since 2.3.1 | |
| 142 | + * @var string[] | |
| 143 | + */ | |
| 144 | + private const FOREIGN_FAQ_BRICKS_ELEMENTS = ['accordion', 'accordion-nested']; | |
| 145 | + | |
| 146 | + /** | |
| 115 | 147 | * Singleton instance. |
| 116 | 148 | * |
| 117 | 149 | * @var self|null |
| 118 | 150 | */ |
| @@ -118,8 +150,16 @@ | ||
| 118 | 150 | */ |
| 119 | 151 | private static ?self $instance = null; |
| 120 | 152 | |
| 121 | 153 | /** |
| 154 | + * Memoised master switch, or null when it has not been read this request. | |
| 155 | + * | |
| 156 | + * @since 2.7.0 | |
| 157 | + * @var bool|null | |
| 158 | + */ | |
| 159 | + private static ?bool $master_switch_on = null; | |
| 160 | + | |
| 161 | + /** | |
| 122 | 162 | * Competing page-level entities: ['rank' => int, 'schema' => array, 'type' => string]. |
| 123 | 163 | * |
| 124 | 164 | * @var array<int,array> |
| 125 | 165 | */ |
| @@ -201,8 +241,10 @@ | ||
| 201 | 241 | * @return void |
| 202 | 242 | */ |
| 203 | 243 | public static function reset(): void { |
| 204 | 244 | self::$instance = null; |
| 245 | + // Or a test that seeds the switch inherits the previous test's answer. | |
| 246 | + self::$master_switch_on = null; | |
| 205 | 247 | } |
| 206 | 248 | |
| 207 | 249 | /** |
| 208 | 250 | * Register a candidate for the page's single page-level entity. |
| @@ -271,8 +313,61 @@ | ||
| 271 | 313 | return (is_string($actual) && $actual !== '') ? $actual : $declared; |
| 272 | 314 | } |
| 273 | 315 | |
| 274 | 316 | /** |
| 317 | + * Whether a page entity may carry a `breadcrumb` property. | |
| 318 | + * | |
| 319 | + * Reading the node's own `@type` key is not enough, and every case it | |
| 320 | + * misses ends with a real WebPage losing a valid property: | |
| 321 | + * | |
| 322 | + * - A deployed schema whose stored JSON omits `@type` carries the type in | |
| 323 | + * the `schema_type` column instead. `effective_type()` already resolves | |
| 324 | + * that, which is why the node's `@id` reads `#webpage` even though the | |
| 325 | + * node itself has no `@type` — so the resolved value is what has to be | |
| 326 | + * consulted here too. | |
| 327 | + * - JSON-LD permits several types on one node. `["WebPage", "FAQPage"]` is | |
| 328 | + * a WebPage, but a strict in_array() against the array as a whole is | |
| 329 | + * false, so the trail would be dropped from a page that may carry it. | |
| 330 | + * - A list naming nothing usable (`[]`, `[null]`) is the first case wearing | |
| 331 | + * the second's clothes, and resolves the same way. | |
| 332 | + * | |
| 333 | + * @since 2.7.0 | |
| 334 | + * @param array $node The page entity. | |
| 335 | + * @param string $resolved_type Type the graph resolved for it. | |
| 336 | + * @return bool | |
| 337 | + */ | |
| 338 | + private function allows_breadcrumb(array $node, string $resolved_type): bool { | |
| 339 | + $declared = $node['@type'] ?? ''; | |
| 340 | + | |
| 341 | + // One path for both shapes. Splitting them invites the list branch to | |
| 342 | + // grow its own idea of what an absent type means, which is the mistake | |
| 343 | + // being corrected here in the first place. | |
| 344 | + $named = false; | |
| 345 | + | |
| 346 | + foreach (is_array($declared) ? $declared : [$declared] as $type) { | |
| 347 | + if (!is_string($type) || '' === $type) { | |
| 348 | + continue; | |
| 349 | + } | |
| 350 | + | |
| 351 | + $named = true; | |
| 352 | + | |
| 353 | + if (in_array($type, self::BREADCRUMB_TYPES, true)) { | |
| 354 | + return true; | |
| 355 | + } | |
| 356 | + } | |
| 357 | + | |
| 358 | + // The node named a type, and none of them may carry a breadcrumb. | |
| 359 | + if ($named) { | |
| 360 | + return false; | |
| 361 | + } | |
| 362 | + | |
| 363 | + // It named none, so it is whatever the graph resolved for it — the same | |
| 364 | + // value its @id was minted from. An empty list is no more informative | |
| 365 | + // than a missing key and must not read as "definitely not a WebPage". | |
| 366 | + return in_array($resolved_type, self::BREADCRUMB_TYPES, true); | |
| 367 | + } | |
| 368 | + | |
| 369 | + /** | |
| 275 | 370 | * Register a node that does not compete for the page-level slot. |
| 276 | 371 | * |
| 277 | 372 | * @since 1.32.0 |
| 278 | 373 | * @param array $schema Schema array. |
| @@ -448,124 +543,156 @@ | ||
| 448 | 543 | if (function_exists('post_password_required') && post_password_required($post)) { |
| 449 | 544 | return; |
| 450 | 545 | } |
| 451 | 546 | |
| 452 | - $this->collect_block_faq($post); | |
| 453 | - $this->collect_elementor_faq($post); | |
| 547 | + // The same gate, for the same reason, with a different cause: a Bricks | |
| 548 | + // page throws `post_content` away, so a FAQ block left there when the | |
| 549 | + // page was switched over never renders. Publishing its questions would | |
| 550 | + // put schema on the page for content no visitor can see — which Google | |
| 551 | + // treats as a violation, not merely a duplicate (#650). | |
| 552 | + foreach (self::faq_groups($post) as $group) { | |
| 553 | + // The stored toggle decides whether a producer contributes schema. | |
| 554 | + // FAQ_Content reports it rather than applying it, because a block | |
| 555 | + // with schema switched off is still visible FAQ content that the | |
| 556 | + // abilities have to describe (#767). | |
| 557 | + if (empty($group['schema'])) { | |
| 558 | + continue; | |
| 559 | + } | |
| 560 | + | |
| 561 | + $this->absorb_content_faq($this->questions_from_pairs($group['pairs'])); | |
| 562 | + } | |
| 454 | 563 | } |
| 455 | 564 | |
| 456 | 565 | /** |
| 457 | - * Record that a body FAQ producer's content is represented in the graph. | |
| 566 | + * Whether ThinkRank would publish a FAQPage for this post. | |
| 458 | 567 | * |
| 459 | - * Deliberately not keyed on the entity count growing: when a block asks the | |
| 460 | - * same question as the per-post deployment, dedup means nothing is added, | |
| 461 | - * but the block's content *is* covered and it must still stay quiet. | |
| 568 | + * Answers the question `get-faq` has to put to an agent — "is this actually | |
| 569 | + * being emitted?" — without rendering the page. Kept here rather than in the | |
| 570 | + * ability because every term of it is this class's own rule: the password | |
| 571 | + * form, the master switch, the per-content-type Schema switch, the other | |
| 572 | + * plugins whose FAQ output makes ThinkRank stand down, and the filter that | |
| 573 | + * overrides all of it. | |
| 462 | 574 | * |
| 463 | - * @since 1.32.0 | |
| 464 | - * @param array $entities Questions found on that producer. | |
| 465 | - * @return void | |
| 466 | - */ | |
| 467 | - private function absorb_content_faq(array $entities): void { | |
| 468 | - if (empty($entities)) { | |
| 469 | - return; | |
| 470 | - } | |
| 471 | - | |
| 472 | - $this->add_faq_entities($entities); | |
| 473 | - $this->absorbed_content_faq = true; | |
| 474 | - } | |
| 475 | - | |
| 476 | - /** | |
| 477 | - * Collect FAQ questions from thinkrank/faq blocks, including nested ones. | |
| 575 | + * It answers for the post as a visitor gets it. A draft or a scheduled post | |
| 576 | + * is reported on what it will emit once it is served, because that is the | |
| 577 | + * question an agent preparing one is asking; a password-protected post is | |
| 578 | + * not, because publishing it changes nothing — the form stays. | |
| 478 | 579 | * |
| 479 | - * @since 1.32.0 | |
| 480 | - * @param \WP_Post $post Post being viewed. | |
| 481 | - * @return void | |
| 580 | + * @since 2.10.1 | |
| 581 | + * @param \WP_Post $post Post to test. | |
| 582 | + * @return bool | |
| 482 | 583 | */ |
| 483 | - private function collect_block_faq(\WP_Post $post): void { | |
| 484 | - if (!function_exists('parse_blocks') || !has_blocks($post->post_content)) { | |
| 485 | - return; | |
| 584 | + public static function will_emit_faqpage(\WP_Post $post): bool { | |
| 585 | + // The same gate collect_post_faq() applies, and for the same reason: | |
| 586 | + // behind a password form the block never renders, so no FAQPage is | |
| 587 | + // published. Without this the ability answered "yes, it is emitted" | |
| 588 | + // about a page that emits nothing — which is the answer that stops an | |
| 589 | + // agent looking any further. | |
| 590 | + if (function_exists('post_password_required') && post_password_required($post)) { | |
| 591 | + return false; | |
| 486 | 592 | } |
| 487 | 593 | |
| 488 | - $this->walk_blocks(parse_blocks($post->post_content)); | |
| 489 | - } | |
| 594 | + $has_questions = false; | |
| 490 | 595 | |
| 491 | - /** | |
| 492 | - * Recurse a parsed block tree collecting FAQ entries. | |
| 493 | - * | |
| 494 | - * @since 1.32.0 | |
| 495 | - * @param array $blocks Parsed blocks. | |
| 496 | - * @return void | |
| 497 | - */ | |
| 498 | - private function walk_blocks(array $blocks): void { | |
| 499 | - foreach ($blocks as $block) { | |
| 500 | - if (!is_array($block)) { | |
| 596 | + foreach (self::faq_groups($post) as $group) { | |
| 597 | + if (empty($group['schema'])) { | |
| 501 | 598 | continue; |
| 502 | 599 | } |
| 503 | 600 | |
| 504 | - if (($block['blockName'] ?? '') === self::FAQ_BLOCK) { | |
| 505 | - $attrs = $block['attrs'] ?? []; | |
| 506 | - | |
| 507 | - // Mirrors Blocks_Manager: schema is on unless explicitly disabled. | |
| 508 | - $disabled = array_key_exists('outputSchema', $attrs) && false === $attrs['outputSchema']; | |
| 509 | - | |
| 510 | - if (!$disabled) { | |
| 511 | - $this->absorb_content_faq($this->questions_from_pairs($attrs['faqs'] ?? [])); | |
| 601 | + foreach ($group['pairs'] as $pair) { | |
| 602 | + if ('' !== trim((string) ($pair['question'] ?? '')) && '' !== trim((string) ($pair['answer'] ?? ''))) { | |
| 603 | + $has_questions = true; | |
| 604 | + break 2; | |
| 512 | 605 | } |
| 513 | 606 | } |
| 607 | + } | |
| 514 | 608 | |
| 515 | - if (!empty($block['innerBlocks']) && is_array($block['innerBlocks'])) { | |
| 516 | - $this->walk_blocks($block['innerBlocks']); | |
| 517 | - } | |
| 609 | + if (!$has_questions) { | |
| 610 | + return false; | |
| 518 | 611 | } |
| 612 | + | |
| 613 | + if (!self::schema_allowed_for_post($post)) { | |
| 614 | + return false; | |
| 615 | + } | |
| 616 | + | |
| 617 | + $emit = !(new self())->has_foreign_faq_source($post); | |
| 618 | + | |
| 619 | + /** This filter is documented in includes/frontend/class-schema-graph.php */ | |
| 620 | + return (bool) apply_filters('thinkrank_emit_faqpage', $emit, $post); | |
| 519 | 621 | } |
| 520 | 622 | |
| 521 | 623 | /** |
| 522 | - * Collect FAQ questions from Elementor FAQ widgets. | |
| 624 | + * output_allowed(), asked about a named post instead of the current query. | |
| 523 | 625 | * |
| 524 | - * @since 1.32.0 | |
| 525 | - * @param \WP_Post $post Post being viewed. | |
| 526 | - * @return void | |
| 626 | + * output_allowed() resolves the content type from the main query, which in | |
| 627 | + * an admin or MCP request is not the post being asked about. | |
| 628 | + * | |
| 629 | + * @since 2.10.1 | |
| 630 | + * @param \WP_Post $post Post to test. | |
| 631 | + * @return bool | |
| 527 | 632 | */ |
| 528 | - private function collect_elementor_faq(\WP_Post $post): void { | |
| 529 | - $raw = get_post_meta($post->ID, '_elementor_data', true); | |
| 530 | - if (empty($raw) || !is_string($raw)) { | |
| 531 | - return; | |
| 633 | + private static function schema_allowed_for_post(\WP_Post $post): bool { | |
| 634 | + if (null === self::$master_switch_on) { | |
| 635 | + // Resolve the site-wide half through the existing reader. | |
| 636 | + self::output_allowed(); | |
| 532 | 637 | } |
| 533 | 638 | |
| 534 | - $elements = json_decode($raw, true); | |
| 535 | - if (!is_array($elements)) { | |
| 536 | - return; | |
| 639 | + if (!self::$master_switch_on) { | |
| 640 | + return false; | |
| 537 | 641 | } |
| 538 | 642 | |
| 539 | - $this->walk_elementor($elements); | |
| 643 | + if (class_exists('ThinkRank\\SEO\\Content_Type_Settings')) { | |
| 644 | + return \ThinkRank\SEO\Content_Type_Settings::is_enabled( | |
| 645 | + \ThinkRank\SEO\Content_Type_Settings::FEATURE_SCHEMA, | |
| 646 | + (string) $post->post_type, | |
| 647 | + true | |
| 648 | + ); | |
| 649 | + } | |
| 650 | + | |
| 651 | + return true; | |
| 540 | 652 | } |
| 541 | 653 | |
| 542 | 654 | /** |
| 543 | - * Recurse an Elementor element tree collecting FAQ entries. | |
| 655 | + * Every FAQ producer on a post, loaded defensively. | |
| 544 | 656 | * |
| 545 | - * @since 1.32.0 | |
| 546 | - * @param array $elements Elementor elements. | |
| 547 | - * @return void | |
| 657 | + * The reader lives in the SEO namespace, and this class runs in contexts | |
| 658 | + * where that autoloader is not guaranteed — which is why the Bricks gate it | |
| 659 | + * replaced carried the same require. | |
| 660 | + * | |
| 661 | + * @since 2.10.1 | |
| 662 | + * @param \WP_Post $post Post being read. | |
| 663 | + * @return array<int, array{source: string, schema: bool, pairs: array}> | |
| 548 | 664 | */ |
| 549 | - private function walk_elementor(array $elements): void { | |
| 550 | - foreach ($elements as $element) { | |
| 551 | - if (!is_array($element)) { | |
| 552 | - continue; | |
| 665 | + private static function faq_groups(\WP_Post $post): array { | |
| 666 | + if (!class_exists('ThinkRank\\SEO\\FAQ_Content')) { | |
| 667 | + $file = THINKRANK_PLUGIN_DIR . 'includes/seo/class-faq-content.php'; | |
| 668 | + if (!file_exists($file)) { | |
| 669 | + return []; | |
| 553 | 670 | } |
| 671 | + require_once $file; | |
| 672 | + } | |
| 554 | 673 | |
| 555 | - if (($element['widgetType'] ?? '') === self::FAQ_WIDGET) { | |
| 556 | - $settings = $element['settings'] ?? []; | |
| 674 | + return \ThinkRank\SEO\FAQ_Content::groups($post); | |
| 675 | + } | |
| 557 | 676 | |
| 558 | - // Mirrors FAQ_Widget: schema unless the toggle is off. | |
| 559 | - if ('yes' === ($settings['output_schema'] ?? 'yes')) { | |
| 560 | - $this->absorb_content_faq($this->questions_from_pairs($settings['faqs'] ?? [])); | |
| 561 | - } | |
| 562 | - } | |
| 677 | + /** | |
| 678 | + * Record that a body FAQ producer's content is represented in the graph. | |
| 679 | + * | |
| 680 | + * Deliberately not keyed on the entity count growing: when a block asks the | |
| 681 | + * same question as the per-post deployment, dedup means nothing is added, | |
| 682 | + * but the block's content *is* covered and it must still stay quiet. | |
| 683 | + * | |
| 684 | + * @since 1.32.0 | |
| 685 | + * @param array $entities Questions found on that producer. | |
| 686 | + * @return void | |
| 687 | + */ | |
| 688 | + private function absorb_content_faq(array $entities): void { | |
| 689 | + if (empty($entities)) { | |
| 690 | + return; | |
| 691 | + } | |
| 563 | 692 | |
| 564 | - if (!empty($element['elements']) && is_array($element['elements'])) { | |
| 565 | - $this->walk_elementor($element['elements']); | |
| 566 | - } | |
| 567 | - } | |
| 693 | + $this->add_faq_entities($entities); | |
| 694 | + $this->absorbed_content_faq = true; | |
| 568 | 695 | } |
| 569 | 696 | |
| 570 | 697 | /** |
| 571 | 698 | * Turn stored question/answer pairs into Question entities. |
| @@ -624,8 +751,59 @@ | ||
| 624 | 751 | return !empty($this->primary_candidates) || !empty($this->supporting) || !empty($this->faq_entities); |
| 625 | 752 | } |
| 626 | 753 | |
| 627 | 754 | /** |
| 755 | + * Whether ThinkRank may emit structured data for this request at all. | |
| 756 | + * | |
| 757 | + * The master switch and the matrix's per-content-type Schema switch, asked | |
| 758 | + * once. #461 put the master switch inside output_site_schema_markup(), | |
| 759 | + * which is one of four producers; the other three never learned about it, | |
| 760 | + * so turning Schema off removed the deployed rows and left the live | |
| 761 | + * generator running — the page emitted *more* types with the switch off | |
| 762 | + * than with it on (#688). | |
| 763 | + * | |
| 764 | + * Static because the producers that need it do not share a base class: the | |
| 765 | + * three graph producers converge on render(), but Blocks_Manager emits its | |
| 766 | + * own script tag from a content filter and never touches the graph, so it | |
| 767 | + * has to ask the same question independently. | |
| 768 | + * | |
| 769 | + * @since 2.7.0 | |
| 770 | + * @return bool True when structured data may be emitted. | |
| 771 | + */ | |
| 772 | + public static function output_allowed(): bool { | |
| 773 | + // Memoised because inject_block_schema() asks once per matching block, | |
| 774 | + // and Schema_Management_System's constructor builds a schema builder and | |
| 775 | + // a cache manager and registers listeners — it is not something to spin | |
| 776 | + // up per block. The switch is site-wide, so it cannot change within a | |
| 777 | + // request; the per-content-type check below is query-dependent and stays | |
| 778 | + // live. | |
| 779 | + if (null === self::$master_switch_on) { | |
| 780 | + self::$master_switch_on = true; | |
| 781 | + | |
| 782 | + if (class_exists('ThinkRank\\SEO\\Schema_Management_System')) { | |
| 783 | + $settings = (new \ThinkRank\SEO\Schema_Management_System())->get_settings('site', null); | |
| 784 | + | |
| 785 | + // Absent means "not configured", which every other reader treats | |
| 786 | + // as enabled; only a value that is present and off disables. | |
| 787 | + self::$master_switch_on = !(array_key_exists('enabled', $settings) && empty($settings['enabled'])); | |
| 788 | + } | |
| 789 | + } | |
| 790 | + | |
| 791 | + if (!self::$master_switch_on) { | |
| 792 | + return false; | |
| 793 | + } | |
| 794 | + | |
| 795 | + if (class_exists('ThinkRank\\SEO\\Content_Type_Settings')) { | |
| 796 | + return \ThinkRank\SEO\Content_Type_Settings::is_enabled_for_current( | |
| 797 | + \ThinkRank\SEO\Content_Type_Settings::FEATURE_SCHEMA, | |
| 798 | + true | |
| 799 | + ); | |
| 800 | + } | |
| 801 | + | |
| 802 | + return true; | |
| 803 | + } | |
| 804 | + | |
| 805 | + /** | |
| 628 | 806 | * Assemble and emit the graph. Safe to call more than once. |
| 629 | 807 | * |
| 630 | 808 | * @since 1.32.0 |
| 631 | 809 | * @return void |
| @@ -634,8 +812,14 @@ | ||
| 634 | 812 | if ($this->rendered || !$this->has_nodes()) { |
| 635 | 813 | return; |
| 636 | 814 | } |
| 637 | 815 | |
| 816 | + // Every graph producer converges here, so this is the one place the | |
| 817 | + // master switch has to hold for all of them (#688). | |
| 818 | + if (!self::output_allowed()) { | |
| 819 | + return; | |
| 820 | + } | |
| 821 | + | |
| 638 | 822 | // A 404 response represents no content, so there is nothing for |
| 639 | 823 | // structured data to describe. The page-level producers already skip |
| 640 | 824 | // this context, but the site-identity entity does not, so without this |
| 641 | 825 | // guard every miss — including crawlers probing URLs that never existed |
| @@ -670,8 +854,18 @@ | ||
| 670 | 854 | if (empty($graph)) { |
| 671 | 855 | return; |
| 672 | 856 | } |
| 673 | 857 | |
| 858 | + // One pass over the assembled graph, rather than at each producer. | |
| 859 | + // @id and url values arrive from a dozen of them — some derived from | |
| 860 | + // WordPress, some read straight out of stored settings — and on a | |
| 861 | + // misconfigured site that produced a single graph carrying both | |
| 862 | + // schemes at once, with @ids that no longer matched the canonical they | |
| 863 | + // are supposed to identify (#638). Normalizing where the graph is | |
| 864 | + // serialized is the only place that catches all of them, including | |
| 865 | + // nodes an add-on added through the filter above. | |
| 866 | + $graph = \ThinkRank\SEO\Url_Scheme::apply_deep($graph); | |
| 867 | + | |
| 674 | 868 | $json = wp_json_encode( |
| 675 | 869 | ['@context' => self::SCHEMA_CONTEXT, '@graph' => array_values($graph)], |
| 676 | 870 | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE | JSON_PRETTY_PRINT |
| 677 | 871 | | JSON_HEX_TAG | JSON_HEX_AMP | JSON_HEX_APOS | JSON_HEX_QUOT |
| @@ -681,9 +875,9 @@ | ||
| 681 | 875 | return; |
| 682 | 876 | } |
| 683 | 877 | |
| 684 | 878 | echo "<!-- ThinkRank Schema Graph -->\n"; |
| 685 | - echo '<script type="application/ld+json">' . "\n"; | |
| 879 | + echo '<script type="application/ld+json" data-thinkrank="schema-graph">' . "\n"; | |
| 686 | 880 | echo $json . "\n"; // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped -- wp_json_encode with JSON_HEX_* cannot break out of the script block. |
| 687 | 881 | echo '</script>' . "\n"; |
| 688 | 882 | echo "<!-- /ThinkRank Schema Graph -->\n"; |
| 689 | 883 | } |
| @@ -794,11 +988,12 @@ | ||
| 794 | 988 | $primary = ['schema' => $faq, 'type' => 'FAQPage']; |
| 795 | 989 | $faq = null; |
| 796 | 990 | } |
| 797 | 991 | |
| 798 | - $nodes = []; | |
| 799 | - $primary_id = ''; | |
| 800 | - $used_ids = []; | |
| 992 | + $nodes = []; | |
| 993 | + $primary_id = ''; | |
| 994 | + $primary_type = ''; | |
| 995 | + $used_ids = []; | |
| 801 | 996 | |
| 802 | 997 | if (null !== $primary) { |
| 803 | 998 | $node = $primary['schema']; |
| 804 | 999 | |
| @@ -806,9 +1001,10 @@ | ||
| 806 | 1001 | // so an "Article" setting that renders BlogPosting reads #blogposting. |
| 807 | 1002 | $resolved_type = $this->effective_type($node, $primary['type']); |
| 808 | 1003 | |
| 809 | 1004 | $node = $this->assign_id($node, $base . '#' . strtolower($resolved_type), $used_ids); |
| 810 | - $primary_id = $node['@id']; | |
| 1005 | + $primary_id = $node['@id']; | |
| 1006 | + $primary_type = $resolved_type; | |
| 811 | 1007 | $nodes['primary'] = $node; |
| 812 | 1008 | } |
| 813 | 1009 | |
| 814 | 1010 | // Entities deployed alongside the winner (Pro's Multi-Schema lets a post |
| @@ -849,8 +1045,9 @@ | ||
| 849 | 1045 | $person_nodes = []; |
| 850 | 1046 | |
| 851 | 1047 | foreach ($this->supporting as $index => $node) { |
| 852 | 1048 | $type = $node['@type'] ?? ''; |
| 1049 | + $slot = $this->site_level_slot($type); | |
| 853 | 1050 | |
| 854 | 1051 | if ('BreadcrumbList' === $type) { |
| 855 | 1052 | $node = $this->assign_id($node, $base . '#breadcrumb', $used_ids); |
| 856 | 1053 | $breadcrumb_id = $node['@id']; |
| @@ -859,9 +1056,9 @@ | ||
| 859 | 1056 | $website_id = $node['@id']; |
| 860 | 1057 | } elseif ('Organization' === $type) { |
| 861 | 1058 | $node = $this->assign_id($node, home_url('/#organization'), $used_ids); |
| 862 | 1059 | $organization_nodes[] = $node; |
| 863 | - } elseif (in_array($type, self::SITE_LEVEL_TYPES, true)) { | |
| 1060 | + } elseif ('' !== $slot) { | |
| 864 | 1061 | // Site-level entities describe the site, not the page, so their |
| 865 | 1062 | // @id must be stable across URLs. Falling through to the |
| 866 | 1063 | // page-scoped branch minted a fresh identity on every URL, so |
| 867 | 1064 | // one business became N entities in a crawler's graph and |
| @@ -871,9 +1068,9 @@ | ||
| 871 | 1068 | // arrive here claiming the same @id. assign_id() would resolve |
| 872 | 1069 | // that collision by minting "#person-2", turning a duplicate |
| 873 | 1070 | // into two competing entities that split the identity a |
| 874 | 1071 | // knowledge graph is meant to consolidate (#479). |
| 875 | - $duplicate_key = $this->find_same_entity($nodes, $type, $node); | |
| 1072 | + $duplicate_key = $this->find_same_entity($nodes, $slot, $node); | |
| 876 | 1073 | |
| 877 | 1074 | if (null !== $duplicate_key) { |
| 878 | 1075 | $nodes[$duplicate_key] = $this->merge_entity($nodes[$duplicate_key], $node); |
| 879 | 1076 | continue; |
| @@ -878,11 +1075,11 @@ | ||
| 878 | 1075 | $nodes[$duplicate_key] = $this->merge_entity($nodes[$duplicate_key], $node); |
| 879 | 1076 | continue; |
| 880 | 1077 | } |
| 881 | 1078 | |
| 882 | - $node = $this->assign_id($node, home_url('/#' . strtolower($type)), $used_ids); | |
| 1079 | + $node = $this->assign_id($node, home_url('/#' . strtolower($slot)), $used_ids); | |
| 883 | 1080 | |
| 884 | - if ('Person' === $type) { | |
| 1081 | + if ('Person' === $slot) { | |
| 885 | 1082 | $person_nodes[] = $node; |
| 886 | 1083 | } |
| 887 | 1084 | } elseif (is_string($type) && $type !== '') { |
| 888 | 1085 | $node = $this->assign_id($node, $base . '#' . strtolower($type), $used_ids); |
| @@ -895,9 +1092,13 @@ | ||
| 895 | 1092 | if (isset($nodes['primary'])) { |
| 896 | 1093 | if ($website_id !== '' && !isset($nodes['primary']['isPartOf'])) { |
| 897 | 1094 | $nodes['primary']['isPartOf'] = ['@id' => $website_id]; |
| 898 | 1095 | } |
| 899 | - if ($breadcrumb_id !== '' && !isset($nodes['primary']['breadcrumb'])) { | |
| 1096 | + if ( | |
| 1097 | + $breadcrumb_id !== '' | |
| 1098 | + && !isset($nodes['primary']['breadcrumb']) | |
| 1099 | + && $this->allows_breadcrumb($nodes['primary'], $primary_type) | |
| 1100 | + ) { | |
| 900 | 1101 | $nodes['primary']['breadcrumb'] = ['@id' => $breadcrumb_id]; |
| 901 | 1102 | } |
| 902 | 1103 | |
| 903 | 1104 | // Point publisher/author at the full nodes already in the graph. |
| @@ -971,8 +1172,32 @@ | ||
| 971 | 1172 | return ['winner' => array_shift($kept), 'siblings' => array_values($kept)]; |
| 972 | 1173 | } |
| 973 | 1174 | |
| 974 | 1175 | /** |
| 1176 | + * The site-level entity a node's @type makes it, or '' for none. | |
| 1177 | + * | |
| 1178 | + * A LocalBusiness is published under the subtype the site chose in Local | |
| 1179 | + * SEO ("Dentist", "Restaurant"), but it is still the one business entity, | |
| 1180 | + * so every subtype shares the LocalBusiness slot: the same home-scoped | |
| 1181 | + * `/#localbusiness` @id it always had, and the same entity for | |
| 1182 | + * find_same_entity() to fold a per-post copy into. Matching the literal | |
| 1183 | + * type instead would have given a Dentist a fresh page-scoped @id on every | |
| 1184 | + * URL, the exact split #471 closed. | |
| 1185 | + * | |
| 1186 | + * @since 2.10.0 | |
| 1187 | + * | |
| 1188 | + * @param mixed $type Node @type. | |
| 1189 | + * @return string 'LocalBusiness', 'Person' or ''. | |
| 1190 | + */ | |
| 1191 | + private function site_level_slot($type): string { | |
| 1192 | + if (\ThinkRank\Config\Local_Business_Types_Config::is_local_business($type)) { | |
| 1193 | + return 'LocalBusiness'; | |
| 1194 | + } | |
| 1195 | + | |
| 1196 | + return (is_string($type) && in_array($type, self::SITE_LEVEL_TYPES, true)) ? $type : ''; | |
| 1197 | + } | |
| 1198 | + | |
| 1199 | + /** | |
| 975 | 1200 | * Find an already-placed node describing the same entity as $node. |
| 976 | 1201 | * |
| 977 | 1202 | * Identity is `email` when both carry one — two people can share a name, |
| 978 | 1203 | * but not a mailbox — and a case-insensitive `name` match otherwise. A node |
| @@ -981,9 +1206,9 @@ | ||
| 981 | 1206 | * |
| 982 | 1207 | * @since 2.0.2 |
| 983 | 1208 | * |
| 984 | 1209 | * @param array $nodes Nodes placed so far, keyed. |
| 985 | - * @param string $type Schema type to match within. | |
| 1210 | + * @param string $type Site-level slot to match within (see site_level_slot()). | |
| 986 | 1211 | * @param array $node Candidate node. |
| 987 | 1212 | * @return string|null Key of the matching node, or null. |
| 988 | 1213 | */ |
| 989 | 1214 | private function find_same_entity(array $nodes, string $type, array $node): ?string { |
| @@ -994,9 +1219,9 @@ | ||
| 994 | 1219 | return null; |
| 995 | 1220 | } |
| 996 | 1221 | |
| 997 | 1222 | foreach ($nodes as $key => $placed) { |
| 998 | - if (($placed['@type'] ?? '') !== $type) { | |
| 1223 | + if ($this->site_level_slot($placed['@type'] ?? '') !== $type) { | |
| 999 | 1224 | continue; |
| 1000 | 1225 | } |
| 1001 | 1226 | |
| 1002 | 1227 | $placed_email = isset($placed['email']) ? strtolower(trim((string) $placed['email'])) : ''; |
| @@ -1140,8 +1365,19 @@ | ||
| 1140 | 1365 | if (!$post instanceof \WP_Post) { |
| 1141 | 1366 | return false; |
| 1142 | 1367 | } |
| 1143 | 1368 | |
| 1369 | + return $this->has_foreign_elementor_faq($post) || $this->has_foreign_bricks_faq($post); | |
| 1370 | + } | |
| 1371 | + | |
| 1372 | + /** | |
| 1373 | + * Whether an Elementor widget on this post publishes a FAQPage. | |
| 1374 | + * | |
| 1375 | + * @since 2.1.0 | |
| 1376 | + * @param \WP_Post $post Post being viewed. | |
| 1377 | + * @return bool | |
| 1378 | + */ | |
| 1379 | + private function has_foreign_elementor_faq(\WP_Post $post): bool { | |
| 1144 | 1380 | $raw = get_post_meta($post->ID, '_elementor_data', true); |
| 1145 | 1381 | if (empty($raw) || !is_string($raw)) { |
| 1146 | 1382 | return false; |
| 1147 | 1383 | } |
| @@ -1148,8 +1384,99 @@ | ||
| 1148 | 1384 | |
| 1149 | 1385 | $elements = json_decode($raw, true); |
| 1150 | 1386 | |
| 1151 | 1387 | return is_array($elements) && $this->elements_have_foreign_faq($elements); |
| 1388 | + } | |
| 1389 | + | |
| 1390 | + /** | |
| 1391 | + * Whether a Bricks element on this post publishes a FAQPage. | |
| 1392 | + * | |
| 1393 | + * Bricks' accordions emit their FAQPage from the body render, so — exactly | |
| 1394 | + * as with EA's accordion — the stored tree is the only signal available at | |
| 1395 | + * `wp_head`, where this decision has to be made. | |
| 1396 | + * | |
| 1397 | + * The tree comes from Builder_Content rather than a direct meta read: a | |
| 1398 | + * Bricks page's content can live on a content template, be assembled from | |
| 1399 | + * components, or be stored but not rendered because the post was switched | |
| 1400 | + * back to the block editor. Reading the meta key here would get all three | |
| 1401 | + * wrong (#649). | |
| 1402 | + * | |
| 1403 | + * @since 2.3.1 | |
| 1404 | + * @param \WP_Post $post Post being viewed. | |
| 1405 | + * @return bool | |
| 1406 | + */ | |
| 1407 | + private function has_foreign_bricks_faq(\WP_Post $post): bool { | |
| 1408 | + foreach ($this->bricks_tree((int) $post->ID) as $element) { | |
| 1409 | + if (is_array($element) && $this->bricks_element_publishes_faq($element)) { | |
| 1410 | + return true; | |
| 1411 | + } | |
| 1412 | + } | |
| 1413 | + | |
| 1414 | + return false; | |
| 1415 | + } | |
| 1416 | + | |
| 1417 | + /** | |
| 1418 | + * Whether one Bricks element will put a FAQPage on the page. | |
| 1419 | + * | |
| 1420 | + * Mirrors Bricks' own emission condition rather than trusting the toggle: | |
| 1421 | + * `accordion` records a question only for an item that has BOTH a title and | |
| 1422 | + * content, so an armed but empty accordion publishes nothing and must not | |
| 1423 | + * cost the page ThinkRank's FAQ node. `accordion-nested` builds its items | |
| 1424 | + * from child elements instead of a repeater, so having children is the | |
| 1425 | + * equivalent test there. | |
| 1426 | + * | |
| 1427 | + * @since 2.3.1 | |
| 1428 | + * @param array $element One Bricks element. | |
| 1429 | + * @return bool | |
| 1430 | + */ | |
| 1431 | + private function bricks_element_publishes_faq(array $element): bool { | |
| 1432 | + $name = is_string($element['name'] ?? null) ? $element['name'] : ''; | |
| 1433 | + if (!in_array($name, self::FOREIGN_FAQ_BRICKS_ELEMENTS, true)) { | |
| 1434 | + return false; | |
| 1435 | + } | |
| 1436 | + | |
| 1437 | + $settings = is_array($element['settings'] ?? null) ? $element['settings'] : []; | |
| 1438 | + | |
| 1439 | + // Bricks writes a checkbox as `true`, and clears it by removing the key. | |
| 1440 | + if (empty($settings['faqSchema'])) { | |
| 1441 | + return false; | |
| 1442 | + } | |
| 1443 | + | |
| 1444 | + if ('accordion-nested' === $name) { | |
| 1445 | + return !empty($element['children']) && is_array($element['children']); | |
| 1446 | + } | |
| 1447 | + | |
| 1448 | + $items = is_array($settings['accordions'] ?? null) ? $settings['accordions'] : []; | |
| 1449 | + | |
| 1450 | + foreach ($items as $item) { | |
| 1451 | + if (is_array($item) | |
| 1452 | + && '' !== trim((string) ($item['title'] ?? '')) | |
| 1453 | + && '' !== trim((string) ($item['content'] ?? '')) | |
| 1454 | + ) { | |
| 1455 | + return true; | |
| 1456 | + } | |
| 1457 | + } | |
| 1458 | + | |
| 1459 | + return false; | |
| 1460 | + } | |
| 1461 | + | |
| 1462 | + /** | |
| 1463 | + * The Bricks element tree that renders for a post. | |
| 1464 | + * | |
| 1465 | + * @since 2.3.1 | |
| 1466 | + * @param int $post_id Post being viewed. | |
| 1467 | + * @return array<int,mixed> | |
| 1468 | + */ | |
| 1469 | + private function bricks_tree(int $post_id): array { | |
| 1470 | + if (!class_exists('ThinkRank\\SEO\\Builder_Content')) { | |
| 1471 | + $file = THINKRANK_PLUGIN_DIR . 'includes/seo/class-builder-content.php'; | |
| 1472 | + if (!file_exists($file)) { | |
| 1473 | + return []; | |
| 1474 | + } | |
| 1475 | + require_once $file; | |
| 1476 | + } | |
| 1477 | + | |
| 1478 | + return \ThinkRank\SEO\Builder_Content::bricks_tree($post_id); | |
| 1152 | 1479 | } |
| 1153 | 1480 | |
| 1154 | 1481 | /** |
| 1155 | 1482 | * Recurse an Elementor element tree looking for a third-party FAQ producer. |
