| @@ -291,12 +291,23 @@ | ||
| 291 | 291 | // provided. `!== null`, not `!empty()`: 0 is a legitimate score and |
| 292 | 292 | // empty() discarded it, so a post the editor scored as 0 kept |
| 293 | 293 | // whatever the calculator had produced instead (#394). Both params |
| 294 | 294 | // already default to null above, so null means "not sent". |
| 295 | - if (null !== $readability_score) { | |
| 295 | + // | |
| 296 | + // Only when the editor actually measured something, though. Inside a | |
| 297 | + // page builder there is no editor content to read, so Refresh sent an | |
| 298 | + // empty `live_content` along with the two labels derived from that | |
| 299 | + // empty string — the literal words "No content" — and they were | |
| 300 | + // applied over the correct server-side analysis this request had just | |
| 301 | + // run, then persisted by save_score(). That is what put "No content" | |
| 302 | + // on the dashboard for a page with a valid score (#778). With nothing | |
| 303 | + // measured, the analysis above is the better answer. | |
| 304 | + $editor_measured = '' !== trim((string) $live_content); | |
| 305 | + | |
| 306 | + if ($editor_measured && null !== $readability_score) { | |
| 296 | 307 | $score_data['readability_score'] = $readability_score; |
| 297 | 308 | } |
| 298 | - if (null !== $content_quality) { | |
| 309 | + if ($editor_measured && null !== $content_quality) { | |
| 299 | 310 | $score_data['content_quality'] = $content_quality; |
| 300 | 311 | } |
| 301 | 312 | |
| 302 | 313 | // Save score if requested |
| @@ -336,8 +347,10 @@ | ||
| 336 | 347 | // Get existing score data from database |
| 337 | 348 | $existing_data = $this->calculator->get_existing_score_data($post_id); |
| 338 | 349 | |
| 339 | 350 | if ($existing_data) { |
| 351 | + $existing_data['keywords'] = $this->keyword_placements((int) $post_id); | |
| 352 | + | |
| 340 | 353 | return new WP_REST_Response([ |
| 341 | 354 | 'success' => true, |
| 342 | 355 | 'data' => $existing_data, |
| 343 | 356 | 'message' => __('Existing SEO score retrieved successfully', 'thinkrank') |
| @@ -355,8 +368,27 @@ | ||
| 355 | 368 | } |
| 356 | 369 | } |
| 357 | 370 | |
| 358 | 371 | /** |
| 372 | + * Where each focus keyword sits (#729), for a stored score. A stored score | |
| 373 | + * predates the editor's current keywords and content, so this is read | |
| 374 | + * from the post as saved now — a text pass, no request, no AI. | |
| 375 | + * | |
| 376 | + * @param int $post_id Post ID. | |
| 377 | + * @return array<int, array<string, mixed>> | |
| 378 | + */ | |
| 379 | + private function keyword_placements(int $post_id): array { | |
| 380 | + return $this->calculator->keyword_placements( | |
| 381 | + $this->calculator->analyze_post_content($post_id), | |
| 382 | + [ | |
| 383 | + 'title' => \ThinkRank\SEO\Pattern_Resolver::effective_title($post_id), | |
| 384 | + 'description' => \ThinkRank\SEO\Pattern_Resolver::effective_description($post_id), | |
| 385 | + ], | |
| 386 | + \ThinkRank\SEO\Focus_Keywords::get($post_id) | |
| 387 | + ); | |
| 388 | + } | |
| 389 | + | |
| 390 | + /** | |
| 359 | 391 | * Get score history for a post |
| 360 | 392 | * |
| 361 | 393 | * @param WP_REST_Request $request Request object |
| 362 | 394 | * @return WP_REST_Response|WP_Error Response object |
| @@ -391,10 +423,14 @@ | ||
| 391 | 423 | public function get_latest_score(WP_REST_Request $request) { |
| 392 | 424 | try { |
| 393 | 425 | $post_id = $request->get_param('post_id'); |
| 394 | 426 | |
| 427 | + // The stored row only — no keyword gauges (#729). The dashboard | |
| 428 | + // asks for five posts at once and reads overall_score; a gauge | |
| 429 | + // pass would analyse and render each whole post for nothing. The | |
| 430 | + // editor reads its gauges from /seo-score/get. | |
| 395 | 431 | $latest_score = $this->calculator->get_latest_score($post_id); |
| 396 | - | |
| 432 | + | |
| 397 | 433 | return new WP_REST_Response([ |
| 398 | 434 | 'success' => true, |
| 399 | 435 | 'data' => $latest_score, |
| 400 | 436 | ], 200); |
