| @@ -16,8 +16,12 @@ | ||
| 16 | 16 | use ThinkRank\AI\Metadata_Generator; |
| 17 | 17 | use ThinkRank\AI\SEOScoreCalculator; |
| 18 | 18 | use ThinkRank\Core\Settings; |
| 19 | 19 | use ThinkRank\Core\Database; |
| 20 | +use ThinkRank\Core\Plan_Config; | |
| 21 | +use ThinkRank\SEO\Focus_Keywords; | |
| 22 | +use ThinkRank\SEO\Object_Redirect; | |
| 23 | +use ThinkRank\SEO\Pattern_Resolver; | |
| 20 | 24 | |
| 21 | 25 | // Prevent direct access |
| 22 | 26 | if (!defined('ABSPATH')) { |
| 23 | 27 | exit; |
| @@ -60,9 +64,9 @@ | ||
| 60 | 64 | * @param Metadata_Generator|null $metadata_generator Metadata generator instance |
| 61 | 65 | * @param SEOScoreCalculator|null $seo_calculator SEO Score Calculator instance |
| 62 | 66 | */ |
| 63 | 67 | public function __construct(?Settings $settings = null, ?Metadata_Generator $metadata_generator = null, ?SEOScoreCalculator $seo_calculator = null) { |
| 64 | - $this->settings = $settings ?? new Settings(); | |
| 68 | + $this->settings = $settings ?? Settings::instance(); | |
| 65 | 69 | $this->metadata_generator = $metadata_generator ?? new Metadata_Generator(); |
| 66 | 70 | $this->seo_calculator = $seo_calculator ?? new SEOScoreCalculator(new Database()); |
| 67 | 71 | } |
| 68 | 72 | |
| @@ -74,14 +78,18 @@ | ||
| 74 | 78 | public function init(): void { |
| 75 | 79 | add_action('add_meta_boxes', [$this, 'add_meta_boxes']); |
| 76 | 80 | add_action('save_post', [$this, 'save_meta_boxes'], 10, 2); |
| 77 | 81 | add_action('admin_enqueue_scripts', [$this, 'enqueue_metabox_scripts']); |
| 82 | + add_action('admin_notices', [$this, 'render_redirect_notice']); | |
| 78 | 83 | add_action('init', [$this, 'register_meta_fields']); |
| 79 | 84 | |
| 80 | 85 | // AJAX handlers for meta box functionality |
| 81 | 86 | add_action('wp_ajax_thinkrank_generate_post_metadata', [$this, 'ajax_generate_post_metadata']); |
| 82 | - add_action('wp_ajax_thinkrank_save_post_metadata', [$this, 'ajax_save_post_metadata']); | |
| 83 | 87 | |
| 88 | + // Full metabox save used by editors that don't submit the #post form | |
| 89 | + // (e.g. the Elementor editor). Persists every metabox field at once. | |
| 90 | + add_action('wp_ajax_thinkrank_save_metabox', [$this, 'ajax_save_metabox']); | |
| 91 | + | |
| 84 | 92 | // Removed debug hooks |
| 85 | 93 | } |
| 86 | 94 | |
| 87 | 95 | /** |
| @@ -118,8 +126,116 @@ | ||
| 118 | 126 | 'auth_callback' => function() { |
| 119 | 127 | return current_user_can('edit_posts') || current_user_can('edit_pages'); |
| 120 | 128 | } |
| 121 | 129 | ]); |
| 130 | + | |
| 131 | + // SEO meta fields for import support | |
| 132 | + $string_meta_fields = [ | |
| 133 | + '_thinkrank_canonical_url', | |
| 134 | + '_thinkrank_og_title', | |
| 135 | + '_thinkrank_og_description', | |
| 136 | + '_thinkrank_og_image', | |
| 137 | + '_thinkrank_twitter_title', | |
| 138 | + '_thinkrank_twitter_description', | |
| 139 | + '_thinkrank_twitter_image', | |
| 140 | + '_thinkrank_imported_from', | |
| 141 | + ]; | |
| 142 | + | |
| 143 | + foreach ($string_meta_fields as $meta_key) { | |
| 144 | + register_post_meta('', $meta_key, [ | |
| 145 | + 'show_in_rest' => true, | |
| 146 | + 'single' => true, | |
| 147 | + 'type' => 'string', | |
| 148 | + 'auth_callback' => function () { | |
| 149 | + return current_user_can('edit_posts') || current_user_can('edit_pages'); | |
| 150 | + } | |
| 151 | + ]); | |
| 152 | + } | |
| 153 | + | |
| 154 | + // Multiple focus keywords (array). The legacy single-value | |
| 155 | + // `_thinkrank_focus_keyword` is kept in sync by Focus_Keywords for | |
| 156 | + // backward compatibility and registered for REST as a string elsewhere. | |
| 157 | + register_post_meta('', Focus_Keywords::META_KEY, [ | |
| 158 | + 'show_in_rest' => [ | |
| 159 | + 'schema' => [ | |
| 160 | + 'type' => 'array', | |
| 161 | + 'items' => ['type' => 'string'], | |
| 162 | + ], | |
| 163 | + ], | |
| 164 | + 'single' => true, | |
| 165 | + 'type' => 'array', | |
| 166 | + 'sanitize_callback' => function ($value) { | |
| 167 | + return Focus_Keywords::normalize($value); | |
| 168 | + }, | |
| 169 | + 'auth_callback' => function () { | |
| 170 | + return current_user_can('edit_posts') || current_user_can('edit_pages'); | |
| 171 | + } | |
| 172 | + ]); | |
| 173 | + | |
| 174 | + register_post_meta('', Focus_Keywords::LEGACY_META_KEY, [ | |
| 175 | + 'show_in_rest' => true, | |
| 176 | + 'single' => true, | |
| 177 | + 'type' => 'string', | |
| 178 | + 'auth_callback' => function () { | |
| 179 | + return current_user_can('edit_posts') || current_user_can('edit_pages'); | |
| 180 | + } | |
| 181 | + ]); | |
| 182 | + | |
| 183 | + register_post_meta('', '_thinkrank_robots_meta_enabled', [ | |
| 184 | + 'show_in_rest' => true, | |
| 185 | + 'single' => true, | |
| 186 | + 'type' => 'integer', | |
| 187 | + 'auth_callback' => function () { | |
| 188 | + return current_user_can('edit_posts') || current_user_can('edit_pages'); | |
| 189 | + } | |
| 190 | + ]); | |
| 191 | + | |
| 192 | + register_post_meta('', '_thinkrank_robots_meta', [ | |
| 193 | + 'show_in_rest' => true, | |
| 194 | + 'single' => true, | |
| 195 | + 'type' => 'string', | |
| 196 | + 'sanitize_callback' => [$this, 'sanitize_json_meta_field'], | |
| 197 | + 'auth_callback' => function () { | |
| 198 | + return current_user_can('edit_posts') || current_user_can('edit_pages'); | |
| 199 | + } | |
| 200 | + ]); | |
| 201 | + | |
| 202 | + register_post_meta('', '_thinkrank_advanced_robots_meta', [ | |
| 203 | + 'show_in_rest' => true, | |
| 204 | + 'single' => true, | |
| 205 | + 'type' => 'string', | |
| 206 | + 'sanitize_callback' => [$this, 'sanitize_json_meta_field'], | |
| 207 | + 'auth_callback' => function () { | |
| 208 | + return current_user_can('edit_posts') || current_user_can('edit_pages'); | |
| 209 | + } | |
| 210 | + ]); | |
| 211 | + | |
| 212 | + register_post_meta('', \ThinkRank\SEO\Content_Visibility::SEARCH_META, [ | |
| 213 | + 'show_in_rest' => true, | |
| 214 | + 'single' => true, | |
| 215 | + 'type' => 'integer', | |
| 216 | + 'auth_callback' => function () { | |
| 217 | + return current_user_can('edit_posts') || current_user_can('edit_pages'); | |
| 218 | + } | |
| 219 | + ]); | |
| 220 | + | |
| 221 | + register_post_meta('', \ThinkRank\SEO\Content_Visibility::ARCHIVE_META, [ | |
| 222 | + 'show_in_rest' => true, | |
| 223 | + 'single' => true, | |
| 224 | + 'type' => 'integer', | |
| 225 | + 'auth_callback' => function () { | |
| 226 | + return current_user_can('edit_posts') || current_user_can('edit_pages'); | |
| 227 | + } | |
| 228 | + ]); | |
| 229 | + | |
| 230 | + register_post_meta('', '_thinkrank_primary_category', [ | |
| 231 | + 'show_in_rest' => true, | |
| 232 | + 'single' => true, | |
| 233 | + 'type' => 'integer', | |
| 234 | + 'auth_callback' => function () { | |
| 235 | + return current_user_can('edit_posts') || current_user_can('edit_pages'); | |
| 236 | + } | |
| 237 | + ]); | |
| 122 | 238 | } |
| 123 | 239 | |
| 124 | 240 | /** |
| 125 | 241 | * Sanitize JSON meta field data |
| @@ -162,9 +278,9 @@ | ||
| 162 | 278 | * @param mixed $data Data to sanitize |
| 163 | 279 | * @param int $depth Current recursion depth |
| 164 | 280 | * @return mixed Sanitized data |
| 165 | 281 | */ |
| 166 | - private function sanitize_json_recursively($data, int $depth = 0): mixed { | |
| 282 | + private function sanitize_json_recursively($data, int $depth = 0) { | |
| 167 | 283 | // Prevent deep recursion attacks |
| 168 | 284 | if ($depth > 10) { |
| 169 | 285 | return null; |
| 170 | 286 | } |
| @@ -241,9 +357,9 @@ | ||
| 241 | 357 | * @param int $depth Current recursion depth |
| 242 | 358 | * |
| 243 | 359 | * @return mixed Sanitized data |
| 244 | 360 | */ |
| 245 | - private function sanitize_json_ld_recursively( $data, int $depth = 0 ): mixed { | |
| 361 | + private function sanitize_json_ld_recursively( $data, int $depth = 0 ) { | |
| 246 | 362 | // Prevent deep recursion attacks |
| 247 | 363 | if ( $depth > 10 ) { |
| 248 | 364 | return null; |
| 249 | 365 | } |
| @@ -286,8 +402,10 @@ | ||
| 286 | 402 | */ |
| 287 | 403 | public function add_meta_boxes(): void { |
| 288 | 404 | $post_types = $this->get_supported_post_types(); |
| 289 | 405 | |
| 406 | + $is_block_editor = $this->is_block_editor_screen(); | |
| 407 | + | |
| 290 | 408 | foreach ($post_types as $post_type) { |
| 291 | 409 | add_meta_box( |
| 292 | 410 | 'thinkrank-seo-metabox', |
| 293 | 411 | __('ThinkRank SEO', 'thinkrank'), |
| @@ -295,12 +413,68 @@ | ||
| 295 | 413 | $post_type, |
| 296 | 414 | 'normal', |
| 297 | 415 | 'high' |
| 298 | 416 | ); |
| 417 | + | |
| 418 | + // Classic Editor sidebar quick-access widget (mirrors SureRank's | |
| 419 | + // "Manage your SEO" sidebar box). Skipped in the Block Editor, which | |
| 420 | + // surfaces the full panel below the content instead. | |
| 421 | + if (!$is_block_editor) { | |
| 422 | + add_meta_box( | |
| 423 | + 'thinkrank-seo-sidebar', | |
| 424 | + __('ThinkRank', 'thinkrank'), | |
| 425 | + [$this, 'render_sidebar_meta_box'], | |
| 426 | + $post_type, | |
| 427 | + 'side', | |
| 428 | + 'high' | |
| 429 | + ); | |
| 430 | + } | |
| 299 | 431 | } |
| 300 | 432 | } |
| 301 | 433 | |
| 302 | 434 | /** |
| 435 | + * Whether the current edit screen is using the Block Editor. | |
| 436 | + * | |
| 437 | + * @return bool True when the Block Editor is active, false for Classic Editor. | |
| 438 | + */ | |
| 439 | + private function is_block_editor_screen(): bool { | |
| 440 | + if (!function_exists('get_current_screen')) { | |
| 441 | + return false; | |
| 442 | + } | |
| 443 | + | |
| 444 | + $screen = get_current_screen(); | |
| 445 | + | |
| 446 | + return $screen instanceof \WP_Screen | |
| 447 | + && method_exists($screen, 'is_block_editor') | |
| 448 | + && $screen->is_block_editor(); | |
| 449 | + } | |
| 450 | + | |
| 451 | + /** | |
| 452 | + * Render the Classic Editor sidebar quick-access widget. | |
| 453 | + * | |
| 454 | + * Shows a short label plus a button that scrolls to (and expands) the full | |
| 455 | + * "ThinkRank SEO" panel in the main column. | |
| 456 | + * | |
| 457 | + * @param \WP_Post $post Post object. | |
| 458 | + * @return void | |
| 459 | + */ | |
| 460 | + public function render_sidebar_meta_box(\WP_Post $post): void { | |
| 461 | + $box_title = apply_filters('thinkrank_seo_sidebar_box_title', __('Optimize this content for search & AI with ThinkRank.', 'thinkrank'), $post); | |
| 462 | + $cta_label = apply_filters('thinkrank_seo_sidebar_cta_label', __('Open ThinkRank SEO', 'thinkrank'), $post); | |
| 463 | + ?> | |
| 464 | + <div class="thinkrank-classic-sidebar-box"> | |
| 465 | + <p class="thinkrank-classic-sidebar-box-title"><?php echo esc_html($box_title); ?></p> | |
| 466 | + <button | |
| 467 | + type="button" | |
| 468 | + class="button button-primary thinkrank-classic-sidebar-box-cta" | |
| 469 | + > | |
| 470 | + <?php echo esc_html($cta_label); ?> | |
| 471 | + </button> | |
| 472 | + </div> | |
| 473 | + <?php | |
| 474 | + } | |
| 475 | + | |
| 476 | + /** | |
| 303 | 477 | * Render SEO meta box |
| 304 | 478 | * |
| 305 | 479 | * @param \WP_Post $post Post object |
| 306 | 480 | * @return void |
| @@ -317,20 +491,43 @@ | ||
| 317 | 491 | |
| 318 | 492 | // Render React metabox container with hidden form fields for data |
| 319 | 493 | ?> |
| 320 | 494 | <div id="thinkrank-metabox-container" class="thinkrank-metabox"> |
| 321 | - <div class="thinkrank-loading"> | |
| 322 | - <span class="spinner is-active"></span> | |
| 323 | - <p><?php esc_html_e('Loading ThinkRank metabox...', 'thinkrank'); ?></p> | |
| 324 | - </div> | |
| 325 | 495 | |
| 326 | 496 | <!-- Hidden form fields for React to read initial data --> |
| 327 | 497 | <input type="hidden" id="thinkrank_seo_title" name="thinkrank_seo_title" value="<?php echo esc_attr($existing_metadata['title'] ?? ''); ?>" /> |
| 328 | 498 | <input type="hidden" id="thinkrank_meta_description" name="thinkrank_meta_description" value="<?php echo esc_attr($existing_metadata['description'] ?? ''); ?>" /> |
| 499 | + <?php // Render-time mirrors of the two fields a background writer (Auto AI, bulk, import) can fill after load. The React app never touches these, so on save they still hold the value shown when the form loaded — letting persist_metadata() tell a stale blank apart from a deliberate clear. ?> | |
| 500 | + <input type="hidden" id="thinkrank_seo_title__orig" name="thinkrank_seo_title__orig" value="<?php echo esc_attr($existing_metadata['title'] ?? ''); ?>" /> | |
| 501 | + <input type="hidden" id="thinkrank_meta_description__orig" name="thinkrank_meta_description__orig" value="<?php echo esc_attr($existing_metadata['description'] ?? ''); ?>" /> | |
| 329 | 502 | <input type="hidden" id="thinkrank_focus_keyword" name="thinkrank_focus_keyword" value="<?php echo esc_attr($existing_metadata['focus_keyword'] ?? ''); ?>" /> |
| 503 | + <input type="hidden" id="thinkrank_focus_keywords" name="thinkrank_focus_keywords" value="<?php echo esc_attr(wp_json_encode($existing_metadata['focus_keywords'] ?? [])); ?>" /> | |
| 330 | 504 | <input type="hidden" id="thinkrank_seo_score" name="thinkrank_seo_score" value="<?php echo esc_attr($existing_metadata['seo_score'] ?? '0'); ?>" /> |
| 331 | 505 | <input type="hidden" id="thinkrank_generated_at" name="thinkrank_generated_at" value="<?php echo esc_attr($existing_metadata['generated_at'] ?? ''); ?>" /> |
| 332 | 506 | <input type="hidden" id="thinkrank_pillar_content" name="thinkrank_pillar_content" value="<?php echo esc_attr($existing_metadata['pillar_content'] ?? ''); ?>" /> |
| 507 | + <input type="hidden" id="thinkrank_exclude_from_search" name="thinkrank_exclude_from_search" value="<?php echo esc_attr((string) ($existing_metadata['exclude_from_search'] ?? '')); ?>" /> | |
| 508 | + <input type="hidden" id="thinkrank_exclude_from_archives" name="thinkrank_exclude_from_archives" value="<?php echo esc_attr((string) ($existing_metadata['exclude_from_archives'] ?? '')); ?>" /> | |
| 509 | + <input type="hidden" id="thinkrank_canonical_url" name="thinkrank_canonical_url" value="<?php echo esc_url($existing_metadata['canonical_url'] ?? ''); ?>" /> | |
| 510 | + <?php | |
| 511 | + // The redirect lives in Pro's rules table, not post meta, so nothing | |
| 512 | + // else hands it to the React app. Without these the field loads | |
| 513 | + // empty, and its own hidden input then posts that empty value on the | |
| 514 | + // next save, which Object_Redirect reads as "remove the redirect". | |
| 515 | + // Rendered only when a provider can store it, matching MetaboxApp. | |
| 516 | + if (Object_Redirect::is_supported()) : | |
| 517 | + ?> | |
| 518 | + <input type="hidden" id="thinkrank_redirect_url" name="thinkrank_redirect_url" value="<?php echo esc_attr((string) ($existing_metadata['redirect_url'] ?? '')); ?>" /> | |
| 519 | + <input type="hidden" id="thinkrank_redirect_type" name="thinkrank_redirect_type" value="<?php echo esc_attr((string) ($existing_metadata['redirect_type'] ?? Object_Redirect::DEFAULT_TYPE)); ?>" /> | |
| 520 | + <?php endif; ?> | |
| 521 | + <input type="hidden" id="thinkrank_robots_meta_enabled" name="thinkrank_robots_meta_enabled" value="<?php echo esc_attr((string) ($existing_metadata['robots_meta_enabled'] ?? '0')); ?>" /> | |
| 522 | + <input type="hidden" id="thinkrank_robots_meta" name="thinkrank_robots_meta" value="<?php echo esc_attr((string) ($existing_metadata['robots_meta'] ?? '')); ?>" /> | |
| 523 | + <input type="hidden" id="thinkrank_advanced_robots_meta" name="thinkrank_advanced_robots_meta" value="<?php echo esc_attr((string) ($existing_metadata['advanced_robots_meta'] ?? '')); ?>" /> | |
| 524 | + <input type="hidden" id="thinkrank_og_title" name="thinkrank_og_title" value="<?php echo esc_attr((string) ($existing_metadata['og_title'] ?? '')); ?>" /> | |
| 525 | + <input type="hidden" id="thinkrank_og_description" name="thinkrank_og_description" value="<?php echo esc_attr((string) ($existing_metadata['og_description'] ?? '')); ?>" /> | |
| 526 | + <input type="hidden" id="thinkrank_og_image" name="thinkrank_og_image" value="<?php echo esc_url((string) ($existing_metadata['og_image'] ?? '')); ?>" /> | |
| 527 | + <input type="hidden" id="thinkrank_twitter_title" name="thinkrank_twitter_title" value="<?php echo esc_attr((string) ($existing_metadata['twitter_title'] ?? '')); ?>" /> | |
| 528 | + <input type="hidden" id="thinkrank_twitter_description" name="thinkrank_twitter_description" value="<?php echo esc_attr((string) ($existing_metadata['twitter_description'] ?? '')); ?>" /> | |
| 529 | + <input type="hidden" id="thinkrank_twitter_image" name="thinkrank_twitter_image" value="<?php echo esc_url((string) ($existing_metadata['twitter_image'] ?? '')); ?>" /> | |
| 333 | 530 | <textarea id="thinkrank_content_preview" style="display: none;"><?php echo esc_textarea($content_preview); ?></textarea> |
| 334 | 531 | </div> |
| 335 | 532 | <?php |
| 336 | 533 | |
| @@ -363,33 +560,336 @@ | ||
| 363 | 560 | if (defined('DOING_AUTOSAVE') && DOING_AUTOSAVE) { |
| 364 | 561 | return; |
| 365 | 562 | } |
| 366 | 563 | |
| 367 | - // Save metadata | |
| 564 | + // The classic/block editor submits the metabox fields as part of the | |
| 565 | + // #post form, so they arrive (slashed) in $_POST. Hand them straight to | |
| 566 | + // the shared persistence routine. | |
| 567 | + // phpcs:ignore WordPress.Security.NonceVerification.Missing -- nonce verified above | |
| 568 | + $this->persist_metadata($post_id, wp_unslash($_POST)); | |
| 569 | + | |
| 570 | + // The redirect is the one field here that can be refused outright. The | |
| 571 | + // response to this request is a redirect back to the editor, so the | |
| 572 | + // reason has to survive one page load to be seen at all. | |
| 573 | + $redirect_error = $this->get_last_redirect_error(); | |
| 574 | + if (null !== $redirect_error) { | |
| 575 | + $this->store_redirect_error($redirect_error); | |
| 576 | + } | |
| 577 | + } | |
| 578 | + | |
| 579 | + /** | |
| 580 | + * Persist metabox fields for a post from a form-field-name => value map, | |
| 581 | + * reusing the shared metabox persistence (sanitization, JSON encoding, | |
| 582 | + * focus-keyword normalization, empty-value deletion). | |
| 583 | + * | |
| 584 | + * Intended for non-form callers such as the MCP abilities layer. The | |
| 585 | + * caller is responsible for authorization; keys use the same | |
| 586 | + * `thinkrank_*` field names accepted by the metabox form (e.g. | |
| 587 | + * `thinkrank_seo_title`, `thinkrank_meta_description`, `thinkrank_robots_meta`). | |
| 588 | + * | |
| 589 | + * @param int $post_id Post to update. | |
| 590 | + * @param array $fields Field name => value map. | |
| 591 | + * @return void | |
| 592 | + */ | |
| 593 | + public function save_seo_fields(int $post_id, array $fields): void { | |
| 594 | + $this->persist_metadata($post_id, $fields); | |
| 595 | + } | |
| 596 | + | |
| 597 | + /** | |
| 598 | + * Persist all metabox fields for a post from a $_POST-shaped (already | |
| 599 | + * unslashed) source array. | |
| 600 | + * | |
| 601 | + * Shared by `save_meta_boxes()` (classic/block editor form submit) and | |
| 602 | + * `ajax_save_metabox()` (editors like Elementor that don't submit the #post | |
| 603 | + * form). Nonce/permission checks are the caller's responsibility. Each field | |
| 604 | + * is independently sanitized; missing keys are left untouched. | |
| 605 | + * | |
| 606 | + * @param int $post_id Post to update. | |
| 607 | + * @param array $src Field name => raw value map (unslashed). | |
| 608 | + * @return void | |
| 609 | + */ | |
| 610 | + private function persist_metadata(int $post_id, array $src): void { | |
| 611 | + // Title & meta description are handled separately below: they can be | |
| 612 | + // written out-of-band (Auto AI on publish, imports) | |
| 613 | + // after an editor was opened, so a plain save from that now-stale editor | |
| 614 | + // would clobber the generated value with a blank. Focus keywords are | |
| 615 | + // likewise handled separately (array meta) via Focus_Keywords below. | |
| 616 | + // | |
| 617 | + // Both fields may hold variable tags, so they are sanitized as templates: | |
| 618 | + // sanitize_text_field()/sanitize_textarea_field() strip %date% and | |
| 619 | + // %category% as percent-encoding and store "te%" / "tegory%" (#521). | |
| 620 | + $this->persist_seo_text_field($post_id, $src, 'thinkrank_seo_title', '_thinkrank_seo_title', [Pattern_Resolver::class, 'sanitize_template']); | |
| 621 | + $this->persist_seo_text_field($post_id, $src, 'thinkrank_meta_description', '_thinkrank_meta_description', [Pattern_Resolver::class, 'sanitize_template_textarea']); | |
| 622 | + | |
| 368 | 623 | $fields = [ |
| 369 | - 'thinkrank_seo_title' => 'sanitize_text_field', | |
| 370 | - 'thinkrank_meta_description' => 'sanitize_textarea_field', | |
| 371 | - 'thinkrank_focus_keyword' => 'sanitize_text_field', | |
| 372 | 624 | 'thinkrank_seo_score' => 'absint', |
| 373 | 625 | 'thinkrank_generated_at' => 'sanitize_text_field', |
| 374 | 626 | 'thinkrank_pillar_content' => 'sanitize_text_field', |
| 627 | + 'thinkrank_exclude_from_search' => 'sanitize_text_field', | |
| 628 | + 'thinkrank_exclude_from_archives' => 'sanitize_text_field', | |
| 375 | 629 | ]; |
| 376 | 630 | |
| 631 | + // Focus keywords: prefer the JSON array field; fall back to the legacy | |
| 632 | + // single string. Focus_Keywords::save() normalizes (dedupe, drop empty, | |
| 633 | + // cap at MAX) and keeps the legacy single-value meta in sync. | |
| 634 | + if (isset($src['thinkrank_focus_keywords'])) { | |
| 635 | + $raw_keywords = $src['thinkrank_focus_keywords']; | |
| 636 | + $decoded = is_string($raw_keywords) ? json_decode($raw_keywords, true) : $raw_keywords; | |
| 637 | + Focus_Keywords::save($post_id, is_array($decoded) ? $decoded : []); | |
| 638 | + } elseif (isset($src['thinkrank_focus_keyword'])) { | |
| 639 | + Focus_Keywords::save($post_id, $src['thinkrank_focus_keyword']); | |
| 640 | + } | |
| 641 | + | |
| 642 | + // Update the post slug (post_name) when the metabox permalink field | |
| 643 | + // was edited. This touches the WP post itself, not post meta. | |
| 644 | + // | |
| 645 | + // The baseline is what the field was RENDERED with. Without it the | |
| 646 | + // guard here was a bare isset(), and the hidden input is always | |
| 647 | + // posted — so a user who edited WordPress's own permalink field in | |
| 648 | + // the Classic Editor had their new slug written by core and then | |
| 649 | + // overwritten by this page-load snapshot (#441). | |
| 650 | + if (isset($src['thinkrank_post_slug'])) { | |
| 651 | + $this->maybe_update_slug( | |
| 652 | + $post_id, | |
| 653 | + (string) $src['thinkrank_post_slug'], | |
| 654 | + isset($src['thinkrank_post_slug_baseline']) | |
| 655 | + ? (string) $src['thinkrank_post_slug_baseline'] | |
| 656 | + : null | |
| 657 | + ); | |
| 658 | + } | |
| 659 | + | |
| 660 | + // Save canonical URL separately with URL sanitization | |
| 661 | + if (isset($src['thinkrank_canonical_url'])) { | |
| 662 | + $canonical_url = esc_url_raw((string) $src['thinkrank_canonical_url']); | |
| 663 | + if (empty($canonical_url)) { | |
| 664 | + delete_post_meta($post_id, '_thinkrank_canonical_url'); | |
| 665 | + } else { | |
| 666 | + update_post_meta($post_id, '_thinkrank_canonical_url', $canonical_url); | |
| 667 | + } | |
| 668 | + } | |
| 669 | + | |
| 670 | + $this->last_redirect_error = $this->save_object_redirect('post', $post_id, $src); | |
| 671 | + | |
| 377 | 672 | foreach ($fields as $field => $sanitize_callback) { |
| 378 | - if (isset($_POST[$field])) { | |
| 379 | - // phpcs:ignore WordPress.Security.ValidatedSanitizedInput.InputNotSanitized -- Value is sanitized via callback below | |
| 380 | - $raw_value = wp_unslash($_POST[$field]); | |
| 381 | - $value = call_user_func($sanitize_callback, $raw_value); | |
| 673 | + if (isset($src[$field])) { | |
| 674 | + $value = call_user_func($sanitize_callback, $src[$field]); | |
| 382 | 675 | update_post_meta($post_id, "_{$field}", $value); |
| 383 | 676 | } |
| 384 | 677 | } |
| 385 | 678 | |
| 679 | + $this->save_robots_meta($post_id, $src); | |
| 680 | + $this->save_visibility_meta($post_id, $src); | |
| 681 | + $this->save_social_meta($post_id, $src); | |
| 682 | + | |
| 386 | 683 | // Update last modified timestamp |
| 387 | 684 | update_post_meta($post_id, '_thinkrank_last_updated', current_time('mysql')); |
| 388 | 685 | } |
| 389 | 686 | |
| 687 | + /** | |
| 688 | + * Persist one SEO text field with an out-of-band-write guard. | |
| 689 | + * | |
| 690 | + * Auto AI (on publish) and imports write the SEO title / meta description | |
| 691 | + * directly to post meta. When that happens after an editor | |
| 692 | + * was opened, the editor's hidden input is a stale blank; a normal save | |
| 693 | + * would overwrite the freshly generated value with that blank. This guard | |
| 694 | + * skips the write only when the submitted value is empty AND it was also | |
| 695 | + * empty when the form was rendered (the `<field>__orig` mirror), yet the | |
| 696 | + * stored value is now non-empty — i.e. a background writer won the race. | |
| 697 | + * | |
| 698 | + * A deliberate clear still applies: if the field held a value at render and | |
| 699 | + * is submitted empty, `$orig` is non-empty so the guard does not trigger. | |
| 700 | + * Callers that don't send the `__orig` mirror (e.g. the MCP/Elementor | |
| 701 | + * paths) keep the plain write behavior. | |
| 702 | + * | |
| 703 | + * @param int $post_id Post being saved. | |
| 704 | + * @param array $src Unslashed field map. | |
| 705 | + * @param string $field POST field name (e.g. thinkrank_seo_title). | |
| 706 | + * @param string $meta_key Target post meta key. | |
| 707 | + * @param callable $sanitize Sanitizer applied to the submitted value. | |
| 708 | + * @return void | |
| 709 | + */ | |
| 710 | + private function persist_seo_text_field(int $post_id, array $src, string $field, string $meta_key, callable $sanitize): void { | |
| 711 | + if (!isset($src[$field])) { | |
| 712 | + return; // Field absent from this submit → leave the stored value untouched. | |
| 713 | + } | |
| 390 | 714 | |
| 715 | + $submitted = (string) call_user_func($sanitize, (string) $src[$field]); | |
| 716 | + | |
| 717 | + if ('' === $submitted && isset($src[$field . '__orig'])) { | |
| 718 | + $orig = (string) $src[$field . '__orig']; | |
| 719 | + $stored = (string) get_post_meta($post_id, $meta_key, true); | |
| 720 | + // Blank now, blank at render, but populated in storage → a background | |
| 721 | + // write landed after this editor loaded; don't clobber it. | |
| 722 | + if ('' === $orig && '' !== $stored) { | |
| 723 | + return; | |
| 724 | + } | |
| 725 | + } | |
| 726 | + | |
| 727 | + update_post_meta($post_id, $meta_key, $submitted); | |
| 728 | + } | |
| 729 | + | |
| 391 | 730 | /** |
| 731 | + * Update the post slug (post_name) from the metabox permalink field. | |
| 732 | + * | |
| 733 | + * Runs inside the save_post cycle, so wp_update_post() would recurse — a | |
| 734 | + * static guard prevents re-entry. WordPress applies wp_unique_post_slug(), | |
| 735 | + * so a colliding slug is de-duplicated automatically. Empty input is left | |
| 736 | + * alone (WP keeps/auto-generates the slug); auto-drafts and revisions are | |
| 737 | + * skipped so we don't fight the editor's own slug generation. | |
| 738 | + * | |
| 739 | + * @param int $post_id Post to update. | |
| 740 | + * @param string $raw_slug Desired slug from the metabox. | |
| 741 | + * @return void | |
| 742 | + */ | |
| 743 | + private function maybe_update_slug(int $post_id, string $raw_slug, ?string $baseline = null): void { | |
| 744 | + static $updating = false; | |
| 745 | + if ($updating) { | |
| 746 | + return; | |
| 747 | + } | |
| 748 | + | |
| 749 | + $post = get_post($post_id); | |
| 750 | + if (!$post || wp_is_post_revision($post_id)) { | |
| 751 | + return; | |
| 752 | + } | |
| 753 | + | |
| 754 | + if (in_array($post->post_status, ['auto-draft', 'trash'], true)) { | |
| 755 | + return; | |
| 756 | + } | |
| 757 | + | |
| 758 | + $desired = sanitize_title($raw_slug); | |
| 759 | + if ($desired === '') { | |
| 760 | + return; | |
| 761 | + } | |
| 762 | + | |
| 763 | + // Unchanged from what the form was rendered with, so the user did not | |
| 764 | + // choose this value — they left it alone. Writing it back would undo | |
| 765 | + // whatever core already saved from WordPress's own permalink field a | |
| 766 | + // moment ago, on the same save_post priority (#441). | |
| 767 | + // | |
| 768 | + // Compared against the BASELINE rather than the current post_name on | |
| 769 | + // purpose: by the time this runs core has already updated post_name, | |
| 770 | + // so that comparison cannot tell a deliberate edit from a stale one. | |
| 771 | + if ($baseline !== null && $desired === sanitize_title($baseline)) { | |
| 772 | + return; | |
| 773 | + } | |
| 774 | + | |
| 775 | + if ($desired === $post->post_name) { | |
| 776 | + return; | |
| 777 | + } | |
| 778 | + | |
| 779 | + $updating = true; | |
| 780 | + wp_update_post([ | |
| 781 | + 'ID' => $post_id, | |
| 782 | + 'post_name' => $desired, | |
| 783 | + ]); | |
| 784 | + $updating = false; | |
| 785 | + } | |
| 786 | + | |
| 787 | + /** | |
| 788 | + * Persist per-post Open Graph and Twitter Card overrides. | |
| 789 | + * | |
| 790 | + * Empty values delete the meta entry so the frontend falls back to the | |
| 791 | + * SEO title / meta description / featured image chain. | |
| 792 | + */ | |
| 793 | + private function save_social_meta(int $post_id, array $src): void { | |
| 794 | + $text_fields = [ | |
| 795 | + 'thinkrank_og_title' => '_thinkrank_og_title', | |
| 796 | + 'thinkrank_og_description' => '_thinkrank_og_description', | |
| 797 | + 'thinkrank_twitter_title' => '_thinkrank_twitter_title', | |
| 798 | + 'thinkrank_twitter_description' => '_thinkrank_twitter_description', | |
| 799 | + ]; | |
| 800 | + foreach ($text_fields as $field => $meta_key) { | |
| 801 | + if (!isset($src[$field])) { | |
| 802 | + continue; | |
| 803 | + } | |
| 804 | + // Template fields: the frontend resolves their variable tags, so the | |
| 805 | + // %tokens% have to survive the save (#521). | |
| 806 | + $value = Pattern_Resolver::sanitize_template_textarea((string) $src[$field]); | |
| 807 | + if ($value === '') { | |
| 808 | + delete_post_meta($post_id, $meta_key); | |
| 809 | + } else { | |
| 810 | + update_post_meta($post_id, $meta_key, $value); | |
| 811 | + } | |
| 812 | + } | |
| 813 | + | |
| 814 | + $url_fields = [ | |
| 815 | + 'thinkrank_og_image' => '_thinkrank_og_image', | |
| 816 | + 'thinkrank_twitter_image' => '_thinkrank_twitter_image', | |
| 817 | + ]; | |
| 818 | + foreach ($url_fields as $field => $meta_key) { | |
| 819 | + if (!isset($src[$field])) { | |
| 820 | + continue; | |
| 821 | + } | |
| 822 | + $value = esc_url_raw((string) $src[$field]); | |
| 823 | + if ($value === '') { | |
| 824 | + delete_post_meta($post_id, $meta_key); | |
| 825 | + } else { | |
| 826 | + update_post_meta($post_id, $meta_key, $value); | |
| 827 | + } | |
| 828 | + } | |
| 829 | + } | |
| 830 | + | |
| 831 | + /** | |
| 832 | + * Save per-post robots meta and advanced robots meta from the metabox. | |
| 833 | + * | |
| 834 | + * Stores the robots payloads as JSON-encoded strings, sanitized via | |
| 835 | + * sanitize_json_meta_field. The toggle plus the two JSON blobs are the | |
| 836 | + * single source of truth for per-post robots overrides. | |
| 837 | + */ | |
| 838 | + private function save_robots_meta(int $post_id, array $src): void { | |
| 839 | + if (!isset($src['thinkrank_robots_meta_enabled'])) { | |
| 840 | + return; | |
| 841 | + } | |
| 842 | + | |
| 843 | + $enabled = (int) (bool) $src['thinkrank_robots_meta_enabled']; | |
| 844 | + update_post_meta($post_id, '_thinkrank_robots_meta_enabled', $enabled); | |
| 845 | + | |
| 846 | + if (isset($src['thinkrank_robots_meta'])) { | |
| 847 | + update_post_meta($post_id, '_thinkrank_robots_meta', $this->sanitize_json_meta_field((string) $src['thinkrank_robots_meta'])); | |
| 848 | + } | |
| 849 | + | |
| 850 | + if (isset($src['thinkrank_advanced_robots_meta'])) { | |
| 851 | + update_post_meta($post_id, '_thinkrank_advanced_robots_meta', $this->sanitize_json_meta_field((string) $src['thinkrank_advanced_robots_meta'])); | |
| 852 | + } | |
| 853 | + } | |
| 854 | + | |
| 855 | + | |
| 856 | + /** | |
| 857 | + * Save the per-post listing-visibility switches. | |
| 858 | + * | |
| 859 | + * Stored as 1 or deleted rather than 1/0: the excluded set is read with a | |
| 860 | + * `meta_value = '1'` query, so a row holding 0 would be dead weight on every | |
| 861 | + * post anyone ever unticked. Deleting keeps the postmeta table proportional | |
| 862 | + * to the number of posts actually hidden. | |
| 863 | + * | |
| 864 | + * @since 2.7.0 | |
| 865 | + * | |
| 866 | + * @param int $post_id Post being saved. | |
| 867 | + * @param array $src Submitted fields. | |
| 868 | + * @return void | |
| 869 | + */ | |
| 870 | + private function save_visibility_meta(int $post_id, array $src): void { | |
| 871 | + $fields = [ | |
| 872 | + 'thinkrank_exclude_from_search' => \ThinkRank\SEO\Content_Visibility::SEARCH_META, | |
| 873 | + 'thinkrank_exclude_from_archives' => \ThinkRank\SEO\Content_Visibility::ARCHIVE_META, | |
| 874 | + ]; | |
| 875 | + | |
| 876 | + foreach ($fields as $field => $meta_key) { | |
| 877 | + if (!isset($src[$field])) { | |
| 878 | + continue; | |
| 879 | + } | |
| 880 | + | |
| 881 | + if ((bool) $src[$field]) { | |
| 882 | + update_post_meta($post_id, $meta_key, 1); | |
| 883 | + } else { | |
| 884 | + delete_post_meta($post_id, $meta_key); | |
| 885 | + } | |
| 886 | + } | |
| 887 | + | |
| 888 | + \ThinkRank\SEO\Content_Visibility::flush(); | |
| 889 | + } | |
| 890 | + | |
| 891 | + /** | |
| 392 | 892 | * Enqueue meta box scripts |
| 393 | 893 | * |
| 394 | 894 | * @param string $hook Current admin page hook |
| 395 | 895 | * @return void |
| @@ -395,15 +895,15 @@ | ||
| 395 | 895 | * @return void |
| 396 | 896 | */ |
| 397 | 897 | public function enqueue_metabox_scripts(string $hook): void { |
| 398 | 898 | // Only load on post edit screens (including block editor) |
| 399 | - if (!in_array($hook, ['post.php', 'post-new.php'])) { | |
| 899 | + if (!in_array($hook, ['post.php', 'post-new.php'], true)) { | |
| 400 | 900 | return; |
| 401 | 901 | } |
| 402 | 902 | |
| 403 | 903 | // Get current post type - handle both classic and block editor contexts |
| 404 | 904 | $current_post_type = $this->get_current_post_type(); |
| 405 | - if (!$current_post_type || !in_array($current_post_type, $this->get_supported_post_types())) { | |
| 905 | + if (!$current_post_type || !in_array($current_post_type, $this->get_supported_post_types(), true)) { | |
| 406 | 906 | return; |
| 407 | 907 | } |
| 408 | 908 | |
| 409 | 909 | // Get post object for additional data |
| @@ -408,8 +908,11 @@ | ||
| 408 | 908 | |
| 409 | 909 | // Get post object for additional data |
| 410 | 910 | global $post; |
| 411 | 911 | |
| 912 | + // Ensure wp.media is available for the social-image media picker. | |
| 913 | + wp_enqueue_media(); | |
| 914 | + | |
| 412 | 915 | // No chunk dependencies needed - all bundled into main metabox.js |
| 413 | 916 | // Enqueue React metabox script with direct dependencies |
| 414 | 917 | $asset_file = THINKRANK_PLUGIN_DIR . 'assets/metabox.asset.php'; |
| 415 | 918 | $asset = file_exists($asset_file) ? include $asset_file : [ |
| @@ -416,9 +919,13 @@ | ||
| 416 | 919 | 'dependencies' => ['react', 'wp-element', 'wp-i18n', 'wp-api-fetch', 'wp-components'], |
| 417 | 920 | 'version' => THINKRANK_VERSION |
| 418 | 921 | ]; |
| 419 | 922 | |
| 420 | - // Use dependencies directly from asset file | |
| 923 | + // Use dependencies directly from the asset file. The pinned "Configure | |
| 924 | + // SEO" launcher needs wp-plugins (already listed by the build) and | |
| 925 | + // resolves PinnedItems from wp.editor / wp.interface at runtime, so we do | |
| 926 | + // NOT add wp-interface here: it is not a registered script handle on | |
| 927 | + // WP 7.x, and an unmet dependency would drop the whole metabox script. | |
| 421 | 928 | $dependencies = $asset['dependencies']; |
| 422 | 929 | |
| 423 | 930 | wp_enqueue_script( |
| 424 | 931 | 'thinkrank-metabox', |
| @@ -427,29 +934,81 @@ | ||
| 427 | 934 | $asset['version'], |
| 428 | 935 | true |
| 429 | 936 | ); |
| 430 | 937 | |
| 431 | - // Localize script data | |
| 938 | + // Localize script data (shared builder; reused by the Elementor editor | |
| 939 | + // integration, which has no #post form / hidden inputs of its own). | |
| 940 | + wp_localize_script('thinkrank-metabox', 'thinkrankMetabox', $this->get_localized_data($post->ID)); | |
| 941 | + | |
| 942 | + // Add defer attribute for non-blocking script loading | |
| 943 | + wp_script_add_data('thinkrank-metabox', 'defer', true); | |
| 944 | + | |
| 945 | + // Enqueue metabox styles | |
| 946 | + wp_enqueue_style( | |
| 947 | + 'thinkrank-metabox', | |
| 948 | + THINKRANK_PLUGIN_URL . 'assets/metabox.css', | |
| 949 | + ['wp-components'], | |
| 950 | + THINKRANK_VERSION | |
| 951 | + ); | |
| 952 | + | |
| 953 | + // Classic Editor sidebar widget: lightweight styles + a vanilla-JS | |
| 954 | + // handler so the "Optimize Here" button works without depending on the | |
| 955 | + // React bundle. The box itself is only registered in the Classic Editor. | |
| 956 | + $sidebar_css = <<<'CSS' | |
| 957 | +.thinkrank-classic-sidebar-box-title{margin:0 0 12px;font-size:13px;line-height:1.5;color:#1e1e1e;} | |
| 958 | +.thinkrank-classic-sidebar-box-cta{width:100%;text-align:center;justify-content:center;} | |
| 959 | +CSS; | |
| 960 | + wp_add_inline_style('thinkrank-metabox', $sidebar_css); | |
| 961 | + | |
| 962 | + $sidebar_js = <<<'JS' | |
| 963 | +(function(){ | |
| 964 | + document.addEventListener('click', function(e){ | |
| 965 | + var btn = e.target.closest && e.target.closest('.thinkrank-classic-sidebar-box-cta'); | |
| 966 | + if(!btn){return;} | |
| 967 | + e.preventDefault(); | |
| 968 | + // Open the ThinkRank SEO drawer mounted by the React metabox app. | |
| 969 | + window.dispatchEvent(new CustomEvent('thinkrank:toggle-seo-drawer')); | |
| 970 | + }); | |
| 971 | +})(); | |
| 972 | +JS; | |
| 973 | + wp_add_inline_script('thinkrank-metabox', $sidebar_js); | |
| 974 | + } | |
| 975 | + | |
| 976 | + /** | |
| 977 | + * Build the data object localized into the metabox script (`thinkrankMetabox`). | |
| 978 | + * | |
| 979 | + * Extracted so the Elementor editor integration can reuse the exact same | |
| 980 | + * configuration. Callers that run outside the #post form (Elementor) also | |
| 981 | + * read `existingMetadata`/`contentPreview`, which they add on top of this. | |
| 982 | + * | |
| 983 | + * @param int $post_id Post being edited. | |
| 984 | + * @return array Localized config consumed by the React metabox. | |
| 985 | + */ | |
| 986 | + public function get_localized_data(int $post_id): array { | |
| 987 | + $post = get_post($post_id); | |
| 988 | + $post_type = $post ? $post->post_type : 'post'; | |
| 989 | + | |
| 432 | 990 | // Resolve the correct REST API base for the current post type. |
| 433 | - // Falls back to 'posts' for any type that doesn't register a rest_base. | |
| 434 | - $post_type_obj = get_post_type_object( $post->post_type ); | |
| 435 | - $post_rest_base = ( $post_type_obj && ! empty( $post_type_obj->rest_base ) ) | |
| 991 | + // Falls back to the post type slug for any type without a rest_base. | |
| 992 | + $post_type_obj = get_post_type_object($post_type); | |
| 993 | + $post_rest_base = ($post_type_obj && !empty($post_type_obj->rest_base)) | |
| 436 | 994 | ? $post_type_obj->rest_base |
| 437 | - : $post->post_type; | |
| 995 | + : $post_type; | |
| 438 | 996 | |
| 439 | - wp_localize_script('thinkrank-metabox', 'thinkrankMetabox', [ | |
| 997 | + return [ | |
| 440 | 998 | 'ajaxUrl' => admin_url('admin-ajax.php'), |
| 441 | 999 | 'nonce' => wp_create_nonce('thinkrank_metabox_ajax'), |
| 442 | - 'postId' => $post->ID, | |
| 443 | - 'postType' => $post->post_type, | |
| 1000 | + 'postId' => $post_id, | |
| 1001 | + 'postType' => $post_type, | |
| 444 | 1002 | 'postRestBase' => $post_rest_base, |
| 445 | - 'postPermalink' => get_permalink($post->ID), | |
| 1003 | + 'postPermalink' => get_permalink($post_id), | |
| 1004 | + 'postSlug' => $post ? $post->post_name : '', | |
| 446 | 1005 | 'restUrl' => rest_url('thinkrank/v1/'), |
| 447 | 1006 | 'restNonce' => wp_create_nonce('wp_rest'), |
| 448 | 1007 | 'homeUrl' => home_url(), |
| 449 | 1008 | 'siteName' => get_bloginfo('name'), |
| 450 | 1009 | 'faviconUrl' => $this->get_site_favicon_url(), |
| 451 | - 'featuredImageUrl' => $this->get_post_featured_image_url($post->ID), | |
| 1010 | + 'featuredImageUrl' => $this->get_post_featured_image_url($post_id), | |
| 452 | 1011 | 'strings' => [ |
| 453 | 1012 | 'generating' => __('Generating...', 'thinkrank'), |
| 454 | 1013 | 'analyzing' => __('Analyzing...', 'thinkrank'), |
| 455 | 1014 | 'error' => __('Error occurred', 'thinkrank'), |
| @@ -457,21 +1016,78 @@ | ||
| 457 | 1016 | 'generated' => __('Metadata generated successfully', 'thinkrank'), |
| 458 | 1017 | 'contentTooShort' => __('Please add some content before generating SEO metadata.', 'thinkrank'), |
| 459 | 1018 | 'apiError' => __('Failed to connect to AI service. Please check your API settings.', 'thinkrank'), |
| 460 | 1019 | ], |
| 461 | - 'postModified' => get_the_modified_date('c', $post), | |
| 462 | - 'linkSuggestionsEnabled' => $this->is_link_suggestions_enabled($post->post_type), | |
| 463 | - 'postStatus' => get_post_status($post->ID), | |
| 464 | - 'isPro' => (bool) apply_filters('thinkrank_is_pro_active', false), | |
| 465 | - ]); | |
| 1020 | + 'seoScore' => $this->get_persisted_seo_score($post_id), | |
| 1021 | + 'postModified' => $post ? get_the_modified_date('c', $post) : '', | |
| 1022 | + 'linkSuggestionsEnabled' => $this->is_link_suggestions_enabled($post_type), | |
| 1023 | + 'postStatus' => get_post_status($post_id), | |
| 1024 | + 'isPro' => Plan_Config::is_pro(), | |
| 1025 | + // Whether a provider (Pro's Redirections feature) can actually store | |
| 1026 | + // a redirect. False renders the field as an upsell rather than an | |
| 1027 | + // input that accepts text nothing will ever act on. | |
| 1028 | + 'redirectSupported' => Object_Redirect::is_supported(), | |
| 1029 | + 'redirectTypes' => Object_Redirect::TYPES, | |
| 1030 | + /** | |
| 1031 | + * Filter the editor SEO panel's post-load refresh behaviour. | |
| 1032 | + * | |
| 1033 | + * The panel re-checks `/metadata/{id}` after load so values written | |
| 1034 | + * by a background writer (Auto AI on publish, imports) appear | |
| 1035 | + * without a reload. It only polls while the server | |
| 1036 | + * reports a write in flight, but the poll lives in JavaScript, so | |
| 1037 | + * the switch has to be localized into the bundle rather than being | |
| 1038 | + * a PHP-side filter alone (#329). | |
| 1039 | + * | |
| 1040 | + * Set `enabled` to false to switch the refresh off entirely. | |
| 1041 | + * | |
| 1042 | + * @since 1.30.0 | |
| 1043 | + * | |
| 1044 | + * @param array $config enabled (bool), intervalMs (int), maxTicks (int). | |
| 1045 | + * @param int $post_id Post being edited. | |
| 1046 | + */ | |
| 1047 | + 'seoRefresh' => apply_filters( | |
| 1048 | + 'thinkrank_metabox_seo_refresh', | |
| 1049 | + [ | |
| 1050 | + 'enabled' => true, | |
| 1051 | + 'intervalMs' => 4000, | |
| 1052 | + 'maxTicks' => 10, | |
| 1053 | + ], | |
| 1054 | + $post_id | |
| 1055 | + ), | |
| 1056 | + // Whether any AI provider API key is configured — gates the | |
| 1057 | + // "Generate with AI" button in the metabox | |
| 1058 | + 'aiConfigured' => !empty($this->settings->get('openai_api_key', '')) | |
| 1059 | + || !empty($this->settings->get('claude_api_key', '')) | |
| 1060 | + || !empty($this->settings->get('gemini_api_key', '')) | |
| 1061 | + || !empty($this->settings->get('openrouter_api_key', '')), | |
| 1062 | + // Resolved Global/Bulk SEO variable-tag patterns for this post, shown | |
| 1063 | + // as placeholder previews when a field is empty (the frontend applies | |
| 1064 | + // these same patterns on output). Typing a value overrides them. | |
| 1065 | + 'patternPreviews' => Pattern_Resolver::previews($post_id), | |
| 1066 | + // Token => value map (keys without %), for live client-side preview of | |
| 1067 | + // a custom pattern typed into a metabox field. | |
| 1068 | + 'patternVariables' => Pattern_Resolver::variables($post_id), | |
| 1069 | + ]; | |
| 1070 | + } | |
| 466 | 1071 | |
| 467 | - // Enqueue metabox styles | |
| 468 | - wp_enqueue_style( | |
| 469 | - 'thinkrank-metabox', | |
| 470 | - THINKRANK_PLUGIN_URL . 'assets/metabox.css', | |
| 471 | - [], | |
| 472 | - THINKRANK_VERSION | |
| 473 | - ); | |
| 1072 | + /** | |
| 1073 | + * Latest persisted SEO score for a post. | |
| 1074 | + * | |
| 1075 | + * Read from the scores table — the same source the posts list column and the | |
| 1076 | + * Analysis panel use — so the editor badge agrees with them on load rather | |
| 1077 | + * than showing 0 until the Analysis panel mounts and fetches the score. | |
| 1078 | + * | |
| 1079 | + * The `thinkrank_seo_score` form field is unusable for this: it is only | |
| 1080 | + * written by the AI "Analyze Content" flow and is reset to 0 on every save. | |
| 1081 | + * | |
| 1082 | + * @param int $post_id Post ID | |
| 1083 | + * @return int|null Score, or null when the post has never been analyzed. | |
| 1084 | + */ | |
| 1085 | + private function get_persisted_seo_score(int $post_id): ?int { | |
| 1086 | + $existing = $this->seo_calculator->get_existing_score_data($post_id); | |
| 1087 | + $score = $existing['overall_score'] ?? null; | |
| 1088 | + | |
| 1089 | + return $score !== null ? (int) $score : null; | |
| 474 | 1090 | } |
| 475 | 1091 | |
| 476 | 1092 | /** |
| 477 | 1093 | * Check if link suggestions are enabled for a post type |
| @@ -536,9 +1152,9 @@ | ||
| 536 | 1152 | * Get supported post types |
| 537 | 1153 | * |
| 538 | 1154 | * @return array Supported post types |
| 539 | 1155 | */ |
| 540 | - private function get_supported_post_types(): array { | |
| 1156 | + public function get_supported_post_types(): array { | |
| 541 | 1157 | $default_types = ['post', 'page']; |
| 542 | 1158 | |
| 543 | 1159 | // Add WooCommerce product if available |
| 544 | 1160 | if (class_exists('WooCommerce')) { |
| @@ -547,9 +1163,9 @@ | ||
| 547 | 1163 | |
| 548 | 1164 | // Add other common e-commerce post types |
| 549 | 1165 | $ecommerce_types = ['product', 'shop_order', 'shop_coupon']; |
| 550 | 1166 | foreach ($ecommerce_types as $type) { |
| 551 | - if (post_type_exists($type) && !in_array($type, $default_types)) { | |
| 1167 | + if (post_type_exists($type) && !in_array($type, $default_types, true)) { | |
| 552 | 1168 | $default_types[] = $type; |
| 553 | 1169 | } |
| 554 | 1170 | } |
| 555 | 1171 | |
| @@ -559,28 +1175,45 @@ | ||
| 559 | 1175 | 'show_ui' => true, |
| 560 | 1176 | '_builtin' => false, |
| 561 | 1177 | ]); |
| 562 | 1178 | |
| 1179 | + // WordPress internals that should never carry an SEO metabox. Fixed, | |
| 1180 | + // so it is built once rather than per post type. | |
| 1181 | + $wp_internal_types = [ | |
| 1182 | + 'attachment', | |
| 1183 | + 'revision', | |
| 1184 | + 'nav_menu_item', | |
| 1185 | + 'custom_css', | |
| 1186 | + 'customize_changeset', | |
| 1187 | + 'oembed_cache', | |
| 1188 | + 'user_request', | |
| 1189 | + 'wp_block', | |
| 1190 | + 'wp_template', | |
| 1191 | + 'wp_template_part', | |
| 1192 | + 'wp_global_styles', | |
| 1193 | + 'wp_navigation', | |
| 1194 | + 'acf-field', | |
| 1195 | + 'acf-field-group', | |
| 1196 | + ]; | |
| 1197 | + | |
| 1198 | + // Builder template CPTs (Bricks, Elementor, Divi, Beaver Builder) are | |
| 1199 | + // layout fragments, not pages with their own SEO. Global SEO already | |
| 1200 | + // refuses them; this list is shared with that policy so the two cannot | |
| 1201 | + // drift apart again (#621). | |
| 1202 | + if (!class_exists('\ThinkRank\SEO\Global_SEO_Post_Types')) { | |
| 1203 | + require_once THINKRANK_PLUGIN_DIR . 'includes/seo/class-global-seo-post-types.php'; | |
| 1204 | + } | |
| 1205 | + | |
| 563 | 1206 | foreach ($custom_post_types as $post_type) { |
| 564 | - // Skip certain post types that shouldn't have SEO metabox | |
| 565 | - $excluded_types = [ | |
| 566 | - 'attachment', | |
| 567 | - 'revision', | |
| 568 | - 'nav_menu_item', | |
| 569 | - 'custom_css', | |
| 570 | - 'customize_changeset', | |
| 571 | - 'oembed_cache', | |
| 572 | - 'user_request', | |
| 573 | - 'wp_block', | |
| 574 | - 'wp_template', | |
| 575 | - 'wp_template_part', | |
| 576 | - 'wp_global_styles', | |
| 577 | - 'wp_navigation', | |
| 578 | - 'acf-field', | |
| 579 | - 'acf-field-group', | |
| 580 | - ]; | |
| 1207 | + // Resolved per post type, not hoisted: the shared list runs through | |
| 1208 | + // a public filter that receives the post-type object, so an | |
| 1209 | + // integrator can answer differently for different post types. | |
| 1210 | + $excluded_types = array_merge( | |
| 1211 | + $wp_internal_types, | |
| 1212 | + \ThinkRank\SEO\Global_SEO_Post_Types::excluded_post_types(get_post_type_object($post_type)) | |
| 1213 | + ); | |
| 581 | 1214 | |
| 582 | - if (!in_array($post_type, $excluded_types) && !in_array($post_type, $default_types)) { | |
| 1215 | + if (!in_array($post_type, $excluded_types, true) && !in_array($post_type, $default_types, true)) { | |
| 583 | 1216 | $default_types[] = $post_type; |
| 584 | 1217 | } |
| 585 | 1218 | } |
| 586 | 1219 | |
| @@ -587,21 +1220,163 @@ | ||
| 587 | 1220 | return apply_filters('thinkrank_supported_post_types', $default_types); |
| 588 | 1221 | } |
| 589 | 1222 | |
| 590 | 1223 | /** |
| 1224 | + * Transient holding the last redirect error for the current user. | |
| 1225 | + */ | |
| 1226 | + private const REDIRECT_ERROR_TRANSIENT = 'thinkrank_redirect_error_'; | |
| 1227 | + | |
| 1228 | + /** | |
| 1229 | + * Why the redirect field was refused on the most recent persist, if it was. | |
| 1230 | + * | |
| 1231 | + * @var \WP_Error|null | |
| 1232 | + */ | |
| 1233 | + private ?\WP_Error $last_redirect_error = null; | |
| 1234 | + | |
| 1235 | + /** | |
| 1236 | + * Persist the edit-screen redirect field. | |
| 1237 | + * | |
| 1238 | + * Absent keys are left alone, so a caller that never rendered the field | |
| 1239 | + * (the AJAX save from an editor that submits a subset, an import) cannot | |
| 1240 | + * clear a redirect by omission. | |
| 1241 | + * | |
| 1242 | + * The destination is not post meta — Pro's rules table holds it — so unlike | |
| 1243 | + * every other field here this save can fail for reasons the editor needs to | |
| 1244 | + * hear about: no Pro, plain permalinks, a destination that is the page's own | |
| 1245 | + * URL. Failing silently would be the worst of both, since the field would | |
| 1246 | + * redisplay empty on the next load with no explanation, so the reason is | |
| 1247 | + * stashed for the notice rendered on the next screen. | |
| 1248 | + * | |
| 1249 | + * @param string $object_type 'post' or 'term'. | |
| 1250 | + * @param int $object_id Object ID. | |
| 1251 | + * @param array $src Field name => raw value map. | |
| 1252 | + * @return void | |
| 1253 | + */ | |
| 1254 | + private function save_object_redirect(string $object_type, int $object_id, array $src): ?\WP_Error { | |
| 1255 | + if (!array_key_exists('thinkrank_redirect_url', $src)) { | |
| 1256 | + return null; | |
| 1257 | + } | |
| 1258 | + | |
| 1259 | + $url = (string) $src['thinkrank_redirect_url']; | |
| 1260 | + | |
| 1261 | + // With no provider there is nothing to store and nothing to clear. | |
| 1262 | + // Staying quiet when the field was submitted empty keeps every ordinary | |
| 1263 | + // save on a free site from raising an error about a field the editor | |
| 1264 | + // never touched. | |
| 1265 | + if (!Object_Redirect::is_supported()) { | |
| 1266 | + if ('' !== trim($url)) { | |
| 1267 | + return new \WP_Error( | |
| 1268 | + 'thinkrank_redirect_unsupported', | |
| 1269 | + __('Redirects require ThinkRank Pro with the Redirections feature active.', 'thinkrank') | |
| 1270 | + ); | |
| 1271 | + } | |
| 1272 | + return null; | |
| 1273 | + } | |
| 1274 | + | |
| 1275 | + $type = array_key_exists('thinkrank_redirect_type', $src) | |
| 1276 | + ? $src['thinkrank_redirect_type'] | |
| 1277 | + : Object_Redirect::DEFAULT_TYPE; | |
| 1278 | + | |
| 1279 | + $result = Object_Redirect::save($object_type, $object_id, $url, $type); | |
| 1280 | + | |
| 1281 | + return is_wp_error($result) ? $result : null; | |
| 1282 | + } | |
| 1283 | + | |
| 1284 | + /** | |
| 1285 | + * Why the last persist_metadata() call could not store the redirect. | |
| 1286 | + * | |
| 1287 | + * Every other metabox field either saves or is sanitized into something | |
| 1288 | + * that does; this one can be refused, and each caller reports that | |
| 1289 | + * differently — a notice for the form post, a JSON field for the AJAX save, | |
| 1290 | + * an error message for the MCP ability. | |
| 1291 | + * | |
| 1292 | + * @return \WP_Error|null | |
| 1293 | + */ | |
| 1294 | + public function get_last_redirect_error(): ?\WP_Error { | |
| 1295 | + return $this->last_redirect_error; | |
| 1296 | + } | |
| 1297 | + | |
| 1298 | + /** | |
| 1299 | + * Remember why a redirect could not be saved, for the next admin screen. | |
| 1300 | + * | |
| 1301 | + * @param \WP_Error $error Failure. | |
| 1302 | + * @return void | |
| 1303 | + */ | |
| 1304 | + private function store_redirect_error(\WP_Error $error): void { | |
| 1305 | + $user_id = get_current_user_id(); | |
| 1306 | + if ($user_id <= 0) { | |
| 1307 | + return; | |
| 1308 | + } | |
| 1309 | + | |
| 1310 | + set_transient(self::REDIRECT_ERROR_TRANSIENT . $user_id, $error->get_error_message(), MINUTE_IN_SECONDS); | |
| 1311 | + } | |
| 1312 | + | |
| 1313 | + /** | |
| 1314 | + * Show, once, why the last redirect save failed. | |
| 1315 | + * | |
| 1316 | + * @return void | |
| 1317 | + */ | |
| 1318 | + public function render_redirect_notice(): void { | |
| 1319 | + $user_id = get_current_user_id(); | |
| 1320 | + if ($user_id <= 0) { | |
| 1321 | + return; | |
| 1322 | + } | |
| 1323 | + | |
| 1324 | + $key = self::REDIRECT_ERROR_TRANSIENT . $user_id; | |
| 1325 | + $message = get_transient($key); | |
| 1326 | + | |
| 1327 | + if (!is_string($message) || '' === $message) { | |
| 1328 | + return; | |
| 1329 | + } | |
| 1330 | + | |
| 1331 | + delete_transient($key); | |
| 1332 | + | |
| 1333 | + printf( | |
| 1334 | + '<div class="notice notice-error is-dismissible"><p>%s</p></div>', | |
| 1335 | + esc_html( | |
| 1336 | + sprintf( | |
| 1337 | + /* translators: %s: reason the redirect was not saved. */ | |
| 1338 | + __('ThinkRank could not save the redirect: %s', 'thinkrank'), | |
| 1339 | + $message | |
| 1340 | + ) | |
| 1341 | + ) | |
| 1342 | + ); | |
| 1343 | + } | |
| 1344 | + | |
| 1345 | + /** | |
| 591 | 1346 | * Get existing post metadata |
| 592 | 1347 | * |
| 593 | 1348 | * @param int $post_id Post ID |
| 594 | 1349 | * @return array Existing metadata |
| 595 | 1350 | */ |
| 596 | - private function get_post_metadata(int $post_id): array { | |
| 1351 | + public function get_post_metadata(int $post_id): array { | |
| 1352 | + // One lookup: get() goes through a filter Pro answers from the database. | |
| 1353 | + $redirect = Object_Redirect::get('post', $post_id); | |
| 1354 | + | |
| 597 | 1355 | return [ |
| 598 | 1356 | 'title' => get_post_meta($post_id, '_thinkrank_seo_title', true), |
| 599 | 1357 | 'description' => get_post_meta($post_id, '_thinkrank_meta_description', true), |
| 600 | - 'focus_keyword' => get_post_meta($post_id, '_thinkrank_focus_keyword', true), | |
| 1358 | + 'focus_keyword' => Focus_Keywords::get_primary($post_id), | |
| 1359 | + 'focus_keywords' => Focus_Keywords::get($post_id), | |
| 601 | 1360 | 'seo_score' => get_post_meta($post_id, '_thinkrank_seo_score', true), |
| 602 | 1361 | 'generated_at' => get_post_meta($post_id, '_thinkrank_generated_at', true), |
| 603 | 1362 | 'pillar_content' => get_post_meta($post_id, '_thinkrank_pillar_content', true), |
| 1363 | + 'exclude_from_search' => get_post_meta($post_id, \ThinkRank\SEO\Content_Visibility::SEARCH_META, true), | |
| 1364 | + 'exclude_from_archives' => get_post_meta($post_id, \ThinkRank\SEO\Content_Visibility::ARCHIVE_META, true), | |
| 1365 | + 'canonical_url' => get_post_meta($post_id, '_thinkrank_canonical_url', true), | |
| 1366 | + // Not post meta: the rule in Pro's redirections table is the value. | |
| 1367 | + // See ThinkRank\SEO\Object_Redirect. | |
| 1368 | + 'redirect_url' => $redirect['url'], | |
| 1369 | + 'redirect_type' => $redirect['type'], | |
| 1370 | + 'robots_meta_enabled' => get_post_meta($post_id, '_thinkrank_robots_meta_enabled', true), | |
| 1371 | + 'robots_meta' => get_post_meta($post_id, '_thinkrank_robots_meta', true), | |
| 1372 | + 'advanced_robots_meta' => get_post_meta($post_id, '_thinkrank_advanced_robots_meta', true), | |
| 1373 | + 'og_title' => get_post_meta($post_id, '_thinkrank_og_title', true), | |
| 1374 | + 'og_description' => get_post_meta($post_id, '_thinkrank_og_description', true), | |
| 1375 | + 'og_image' => get_post_meta($post_id, '_thinkrank_og_image', true), | |
| 1376 | + 'twitter_title' => get_post_meta($post_id, '_thinkrank_twitter_title', true), | |
| 1377 | + 'twitter_description' => get_post_meta($post_id, '_thinkrank_twitter_description', true), | |
| 1378 | + 'twitter_image' => get_post_meta($post_id, '_thinkrank_twitter_image', true), | |
| 604 | 1379 | ]; |
| 605 | 1380 | } |
| 606 | 1381 | |
| 607 | 1382 | /** |
| @@ -606,21 +1381,35 @@ | ||
| 606 | 1381 | |
| 607 | 1382 | /** |
| 608 | 1383 | * Get site favicon URL |
| 609 | 1384 | * |
| 610 | - * @return string Favicon URL | |
| 1385 | + * Prefers WordPress' native Site Icon (zero HTTP). Only when no Site Icon is | |
| 1386 | + * configured does it probe common favicon locations — and that probe is | |
| 1387 | + * cached (including a "no favicon" sentinel) so the editor never fires | |
| 1388 | + * blocking loopback HEAD requests on every load. | |
| 1389 | + * | |
| 1390 | + * @since 1.16.2 Prefer get_site_icon_url() and cache the fallback probe. | |
| 1391 | + * @return string Favicon URL, or '' when none can be resolved. | |
| 611 | 1392 | */ |
| 612 | 1393 | private function get_site_favicon_url(): string { |
| 613 | - // Try to get site icon first (WordPress 4.3+) | |
| 614 | - $site_icon_id = get_option('site_icon'); | |
| 615 | - if ($site_icon_id) { | |
| 616 | - $site_icon_url = wp_get_attachment_image_url($site_icon_id, 'full'); | |
| 617 | - if ($site_icon_url) { | |
| 618 | - return $site_icon_url; | |
| 619 | - } | |
| 1394 | + // Try the native Site Icon first (WordPress 4.3+) — no HTTP required. | |
| 1395 | + $site_icon_url = get_site_icon_url(); | |
| 1396 | + if ($site_icon_url) { | |
| 1397 | + return $site_icon_url; | |
| 620 | 1398 | } |
| 621 | 1399 | |
| 622 | - // Fallback to common favicon locations | |
| 1400 | + // No Site Icon configured: fall back to probing common favicon paths. | |
| 1401 | + // Cache the resolved value (empty string included) so the blocking | |
| 1402 | + // loopback probe runs at most once per ~12h per site host instead of on | |
| 1403 | + // every editor load. | |
| 1404 | + $cache_key = 'thinkrank_favicon_url_' . md5((string) wp_parse_url(home_url(), PHP_URL_HOST)); | |
| 1405 | + $cached = get_transient($cache_key); | |
| 1406 | + if (false !== $cached) { | |
| 1407 | + return (string) $cached; | |
| 1408 | + } | |
| 1409 | + | |
| 1410 | + $favicon_url = ''; | |
| 1411 | + | |
| 623 | 1412 | $favicon_paths = [ |
| 624 | 1413 | '/favicon.ico', |
| 625 | 1414 | '/favicon.png', |
| 626 | 1415 | '/apple-touch-icon.png', |
| @@ -626,17 +1415,19 @@ | ||
| 626 | 1415 | '/apple-touch-icon.png', |
| 627 | 1416 | ]; |
| 628 | 1417 | |
| 629 | 1418 | foreach ($favicon_paths as $path) { |
| 630 | - $favicon_url = home_url($path); | |
| 631 | - $response = wp_remote_head($favicon_url); | |
| 1419 | + $candidate = home_url($path); | |
| 1420 | + $response = wp_remote_head($candidate, ['timeout' => 3]); | |
| 632 | 1421 | if (!is_wp_error($response) && wp_remote_retrieve_response_code($response) === 200) { |
| 633 | - return $favicon_url; | |
| 1422 | + $favicon_url = $candidate; | |
| 1423 | + break; | |
| 634 | 1424 | } |
| 635 | 1425 | } |
| 636 | 1426 | |
| 637 | - // Final fallback - return a default favicon URL | |
| 638 | - return home_url('/favicon.ico'); | |
| 1427 | + set_transient($cache_key, $favicon_url, 12 * HOUR_IN_SECONDS); | |
| 1428 | + | |
| 1429 | + return $favicon_url; | |
| 639 | 1430 | } |
| 640 | 1431 | |
| 641 | 1432 | /** |
| 642 | 1433 | * Get post featured image URL |
| @@ -646,9 +1437,11 @@ | ||
| 646 | 1437 | */ |
| 647 | 1438 | private function get_post_featured_image_url(int $post_id): ?string { |
| 648 | 1439 | $thumbnail_id = get_post_thumbnail_id($post_id); |
| 649 | 1440 | if ($thumbnail_id) { |
| 650 | - $image_url = wp_get_attachment_image_url($thumbnail_id, 'medium'); | |
| 1441 | + // Use the full-size image — social share images need a large source | |
| 1442 | + // (the preview scales it down), not a small thumbnail. | |
| 1443 | + $image_url = wp_get_attachment_image_url($thumbnail_id, 'full'); | |
| 651 | 1444 | return $image_url ?: null; |
| 652 | 1445 | } |
| 653 | 1446 | return null; |
| 654 | 1447 | } |
| @@ -658,9 +1451,9 @@ | ||
| 658 | 1451 | * |
| 659 | 1452 | * @param \WP_Post $post Post object |
| 660 | 1453 | * @return string Content preview |
| 661 | 1454 | */ |
| 662 | - private function get_content_preview(\WP_Post $post): string { | |
| 1455 | + public function get_content_preview(\WP_Post $post): string { | |
| 663 | 1456 | $content = $post->post_title . "\n\n"; |
| 664 | 1457 | |
| 665 | 1458 | if (!empty($post->post_excerpt)) { |
| 666 | 1459 | $content .= $post->post_excerpt . "\n\n"; |
| @@ -665,15 +1458,24 @@ | ||
| 665 | 1458 | if (!empty($post->post_excerpt)) { |
| 666 | 1459 | $content .= $post->post_excerpt . "\n\n"; |
| 667 | 1460 | } |
| 668 | 1461 | |
| 669 | - $content .= $post->post_content; | |
| 1462 | + // Resolve through Builder_Content: a page builder keeps its words | |
| 1463 | + // outside post_content (Oxygen empties it entirely), and the editor | |
| 1464 | + // cannot reach postmeta, so without this the preview handed to the | |
| 1465 | + // browser is just the title and the live panel reports "No content". | |
| 1466 | + if (!class_exists('\\ThinkRank\\SEO\\Builder_Content')) { | |
| 1467 | + require_once THINKRANK_PLUGIN_DIR . 'includes/seo/class-builder-content.php'; | |
| 1468 | + } | |
| 1469 | + $content .= \ThinkRank\SEO\Builder_Content::resolve($post); | |
| 670 | 1470 | |
| 671 | 1471 | // Clean and limit content |
| 672 | 1472 | $content = wp_strip_all_tags($content); |
| 673 | 1473 | $content = preg_replace('/\s+/', ' ', $content); |
| 674 | 1474 | |
| 675 | - return trim(substr($content, 0, 4000)); | |
| 1475 | + // substr() counts BYTES: on Thai or CJK this handed the model a third | |
| 1476 | + // of the intended content and cut the last character in half (#687). | |
| 1477 | + return trim(\ThinkRank\Core\Seo_Text::trim_to_length($content, 4000)); | |
| 676 | 1478 | } |
| 677 | 1479 | |
| 678 | 1480 | /** |
| 679 | 1481 | * AJAX handler for generating post metadata |
| @@ -713,40 +1515,45 @@ | ||
| 713 | 1515 | } |
| 714 | 1516 | } |
| 715 | 1517 | |
| 716 | 1518 | /** |
| 717 | - * AJAX handler for saving post metadata | |
| 718 | - * | |
| 1519 | + * AJAX: persist the full set of metabox fields for a post. | |
| 1520 | + * | |
| 1521 | + * Used by editors that don't submit the #post form (Elementor). Expects the | |
| 1522 | + * same field names the classic/block metabox submits, sent as POST params. | |
| 1523 | + * $_POST is slashed by WordPress, matching what `persist_metadata()` expects. | |
| 1524 | + * | |
| 719 | 1525 | * @return void |
| 720 | 1526 | */ |
| 721 | - public function ajax_save_post_metadata(): void { | |
| 722 | - // Verify nonce | |
| 1527 | + public function ajax_save_metabox(): void { | |
| 723 | 1528 | $nonce = sanitize_text_field(wp_unslash($_POST['nonce'] ?? '')); |
| 724 | 1529 | if (!wp_verify_nonce($nonce, 'thinkrank_metabox_ajax')) { |
| 725 | - wp_die('Security check failed'); | |
| 1530 | + wp_send_json_error(['message' => __('Security check failed', 'thinkrank')], 403); | |
| 726 | 1531 | } |
| 727 | 1532 | |
| 728 | - // Check permissions | |
| 729 | 1533 | $post_id = absint($_POST['post_id'] ?? 0); |
| 730 | - if (!current_user_can('edit_post', $post_id)) { | |
| 731 | - wp_die('Insufficient permissions'); | |
| 1534 | + if (!$post_id || !current_user_can('edit_post', $post_id)) { | |
| 1535 | + wp_send_json_error(['message' => __('Insufficient permissions', 'thinkrank')], 403); | |
| 732 | 1536 | } |
| 733 | 1537 | |
| 734 | - // Save metadata | |
| 735 | - $metadata = [ | |
| 736 | - 'title' => sanitize_text_field(wp_unslash($_POST['title'] ?? '')), | |
| 737 | - 'description' => sanitize_textarea_field(wp_unslash($_POST['description'] ?? '')), | |
| 738 | - 'focus_keyword' => sanitize_text_field(wp_unslash($_POST['focus_keyword'] ?? '')), | |
| 739 | - 'seo_score' => absint($_POST['seo_score'] ?? 0), | |
| 740 | - ]; | |
| 1538 | + // phpcs:ignore WordPress.Security.NonceVerification.Missing, WordPress.Security.ValidatedSanitizedInput.InputNotSanitized -- nonce verified above; each field sanitized inside persist_metadata() | |
| 1539 | + $this->persist_metadata($post_id, wp_unslash($_POST)); | |
| 741 | 1540 | |
| 742 | - foreach ($metadata as $key => $value) { | |
| 743 | - update_post_meta($post_id, "_thinkrank_{$key}", $value); | |
| 1541 | + // Everything else saved; only the redirect can have been refused. Report | |
| 1542 | + // it in this response rather than as a notice on some later screen — | |
| 1543 | + // this caller never reloads the page. | |
| 1544 | + $redirect_error = $this->get_last_redirect_error(); | |
| 1545 | + if (null !== $redirect_error) { | |
| 1546 | + wp_send_json_error([ | |
| 1547 | + 'message' => sprintf( | |
| 1548 | + /* translators: %s: reason the redirect was not saved. */ | |
| 1549 | + __('Saved, except the redirect: %s', 'thinkrank'), | |
| 1550 | + $redirect_error->get_error_message() | |
| 1551 | + ), | |
| 1552 | + ], 400); | |
| 744 | 1553 | } |
| 745 | 1554 | |
| 746 | - update_post_meta($post_id, '_thinkrank_last_updated', current_time('mysql')); | |
| 747 | - | |
| 748 | 1555 | wp_send_json_success([ |
| 749 | - 'message' => __('Metadata saved successfully!', 'thinkrank'), | |
| 1556 | + 'message' => __('SEO settings saved successfully!', 'thinkrank'), | |
| 750 | 1557 | ]); |
| 751 | 1558 | } |
| 752 | 1559 | } |
