← All changes
|
_inc/lib/core-api/wpcom-endpoints/class-wpcom-rest-api-v3-endpoint-blogging-prompts.php
+96
-53
12.4.2
→
16.3
View file →
| @@ -4,15 +4,21 @@ | ||
| 4 | 4 | * |
| 5 | 5 | * @package automattic/jetpack |
| 6 | 6 | */ |
| 7 | 7 | |
| 8 | -use Automattic\Jetpack\Connection\Client; | |
| 9 | -use Automattic\Jetpack\Connection\Manager; | |
| 8 | +use Automattic\Jetpack\Connection\Traits\WPCOM_REST_API_Proxy_Request; | |
| 10 | 9 | |
| 10 | +if ( ! defined( 'ABSPATH' ) ) { | |
| 11 | + exit( 0 ); | |
| 12 | +} | |
| 13 | + | |
| 11 | 14 | /** |
| 12 | 15 | * REST API endpoint wpcom/v3/sites/%s/blogging-prompts. |
| 13 | 16 | */ |
| 14 | 17 | class WPCOM_REST_API_V3_Endpoint_Blogging_Prompts extends WP_REST_Posts_Controller { |
| 18 | + | |
| 19 | + use WPCOM_REST_API_Proxy_Request; | |
| 20 | + | |
| 15 | 21 | const TEMPLATE_BLOG_ID = 205876834; |
| 16 | 22 | |
| 17 | 23 | /** |
| 18 | 24 | * Whether the endpoint is running on wpcom, or not. |
| @@ -41,9 +47,11 @@ | ||
| 41 | 47 | * Constructor. |
| 42 | 48 | */ |
| 43 | 49 | public function __construct() { |
| 44 | 50 | $this->post_type = 'post'; |
| 45 | - $this->namespace = 'wpcom/v3'; | |
| 51 | + $this->base_api_path = 'wpcom'; | |
| 52 | + $this->version = 'v3'; | |
| 53 | + $this->namespace = $this->base_api_path . '/' . $this->version; | |
| 46 | 54 | $this->rest_base = 'blogging-prompts'; |
| 47 | 55 | $this->wpcom_is_wpcom_only_endpoint = true; |
| 48 | 56 | $this->wpcom_is_site_specific_endpoint = true; |
| 49 | 57 | $this->is_wpcom = defined( 'IS_WPCOM' ) && IS_WPCOM; |
| @@ -98,9 +106,9 @@ | ||
| 98 | 106 | * @return WP_REST_Response|WP_Error Response object on success, or WP_Error object on failure. |
| 99 | 107 | */ |
| 100 | 108 | public function get_items( $request ) { |
| 101 | 109 | if ( ! $this->is_wpcom ) { |
| 102 | - return $this->proxy_request_to_wpcom( $request ); | |
| 110 | + return $this->proxy_request_to_wpcom( $request, '', 'user', true ); | |
| 103 | 111 | } |
| 104 | 112 | |
| 105 | 113 | if ( $request->get_param( 'force_year' ) ) { |
| 106 | 114 | $this->force_year = $request->get_param( 'force_year' ); |
| @@ -107,14 +115,17 @@ | ||
| 107 | 115 | } |
| 108 | 116 | |
| 109 | 117 | switch_to_blog( self::TEMPLATE_BLOG_ID ); |
| 110 | 118 | add_action( 'pre_get_posts', array( $this, 'modify_query' ) ); |
| 111 | - add_filter( 'posts_clauses', array( $this, 'filter_sql' ) ); | |
| 119 | + add_filter( 'posts_clauses', array( $this, 'filter_sql' ), 10, 2 ); | |
| 112 | 120 | $items = parent::get_items( $request ); |
| 113 | 121 | remove_filter( 'posts_clauses', array( $this, 'filter_sql' ) ); |
| 114 | 122 | remove_action( 'pre_get_posts', array( $this, 'modify_query' ) ); |
| 115 | 123 | restore_current_blog(); |
| 116 | 124 | |
| 125 | + // Reset so a later query in the same request can never inherit this state. | |
| 126 | + $this->day_of_year_query = 0; | |
| 127 | + | |
| 117 | 128 | return $items; |
| 118 | 129 | } |
| 119 | 130 | |
| 120 | 131 | /** |
| @@ -124,11 +135,15 @@ | ||
| 124 | 135 | * @return WP_REST_Response|WP_Error Response object on success, or WP_Error object on failure. |
| 125 | 136 | */ |
| 126 | 137 | public function get_item( $request ) { |
| 127 | 138 | if ( ! $this->is_wpcom ) { |
| 128 | - return $this->proxy_request_to_wpcom( $request, $request->get_param( 'id' ) ); | |
| 139 | + return $this->proxy_request_to_wpcom( $request, $request->get_param( 'id' ), 'user', true ); | |
| 129 | 140 | } |
| 130 | 141 | |
| 142 | + if ( $request->get_param( 'force_year' ) ) { | |
| 143 | + $this->force_year = $request->get_param( 'force_year' ); | |
| 144 | + } | |
| 145 | + | |
| 131 | 146 | switch_to_blog( self::TEMPLATE_BLOG_ID ); |
| 132 | 147 | $item = parent::get_item( $request ); |
| 133 | 148 | restore_current_blog(); |
| 134 | 149 | |
| @@ -140,14 +155,22 @@ | ||
| 140 | 155 | * |
| 141 | 156 | * @param WP_Query $wp_query The WP_Query instance (passed by reference). |
| 142 | 157 | */ |
| 143 | 158 | public function modify_query( &$wp_query ) { |
| 144 | - if ( is_array( $wp_query->query_vars['date_query'] ) ) { | |
| 145 | - $wp_query->query_vars['date_query'] = array_map( | |
| 146 | - array( $this, 'map_date_query' ), | |
| 147 | - $wp_query->query_vars['date_query'] | |
| 148 | - ); | |
| 159 | + // parent::get_items() renders each prompt while this hook is still attached, | |
| 160 | + // and rendering can spawn nested WP_Querys (e.g. Gutenberg's wp_global_styles | |
| 161 | + // lookup), so only ever touch the prompts query itself. | |
| 162 | + if ( $this->post_type !== $wp_query->get( 'post_type' ) ) { | |
| 163 | + return; | |
| 149 | 164 | } |
| 165 | + | |
| 166 | + $date_query = $wp_query->get( 'date_query' ); | |
| 167 | + | |
| 168 | + if ( is_array( $date_query ) ) { | |
| 169 | + $wp_query->set( 'date_query', array_map( array( $this, 'map_date_query' ), $date_query ) ); | |
| 170 | + // Mark the query so filter_sql() only modifies this one. | |
| 171 | + $wp_query->set( 'jetpack_blogging_prompts', true ); | |
| 172 | + } | |
| 150 | 173 | } |
| 151 | 174 | |
| 152 | 175 | /** |
| 153 | 176 | * Modify date_query items when querying prompts. |
| @@ -163,9 +186,9 @@ | ||
| 163 | 186 | $date_query['inclusive'] = true; |
| 164 | 187 | |
| 165 | 188 | // If using a "year-less" date, e.g. `--03-16`, override the date_query, and prepare to modify sql manually. |
| 166 | 189 | // `after` should be a date string when making API requests, rather than an array. |
| 167 | - if ( is_string( $date_query['after'] ) && 0 === strpos( $date_query['after'], '-' ) ) { | |
| 190 | + if ( is_string( $date_query['after'] ) && str_starts_with( $date_query['after'], '-' ) ) { | |
| 168 | 191 | $date = date_create_from_format( '--m-d', $date_query['after'] ); |
| 169 | 192 | |
| 170 | 193 | if ( false !== $date ) { |
| 171 | 194 | // PHP day of the year starts with 0; normalize to match SQL DAYOFTHEYEAR which starts with 1. |
| @@ -182,19 +205,24 @@ | ||
| 182 | 205 | |
| 183 | 206 | /** |
| 184 | 207 | * Modify post sql for custom date ordering using the {@see 'posts_clauses'} hook. |
| 185 | 208 | * |
| 186 | - * @param array $clauses SQL clauses for the current query. | |
| 187 | - * @return array Modified SQL clauses. | |
| 209 | + * @param array $clauses SQL clauses for the current query. | |
| 210 | + * @param WP_Query|null $query The WP_Query instance being filtered. | |
| 211 | + * @return array Modified SQL clauses. | |
| 188 | 212 | */ |
| 189 | - public function filter_sql( $clauses ) { | |
| 213 | + public function filter_sql( $clauses, $query = null ) { | |
| 190 | 214 | global $wpdb; |
| 215 | + if ( ! $query instanceof WP_Query || ! $query->get( 'jetpack_blogging_prompts' ) ) { | |
| 216 | + return $clauses; | |
| 217 | + } | |
| 191 | 218 | if ( $this->day_of_year_query > 0 ) { |
| 192 | 219 | $day = $this->day_of_year_query; |
| 193 | 220 | $year = $this->force_year ? $this->force_year : wp_date( 'Y' ); |
| 194 | 221 | |
| 195 | 222 | // Grab the current sort order, `ASC` or `DESC`, so we can reuse it. |
| 196 | - $order = end( explode( ' ', $clauses['orderby'] ) ); | |
| 223 | + $exploded = explode( ' ', $clauses['orderby'] ); | |
| 224 | + $order = end( $exploded ); | |
| 197 | 225 | |
| 198 | 226 | // Calculate the day of year for each prompt, from 1 to 366, but use the current year so that prompts published |
| 199 | 227 | // during leap years have the correct day for non-leap years. |
| 200 | 228 | $fields = $clauses['fields'] . $wpdb->prepare( ", DAYOFYEAR(CONCAT(%d, DATE_FORMAT({$wpdb->posts}.post_date, '-%%m-%%d'))) AS day_of_year", $year ); |
| @@ -270,9 +298,13 @@ | ||
| 270 | 298 | $data['date'] = $this->prepare_date_response( $prompt->post_date_gmt ); |
| 271 | 299 | } |
| 272 | 300 | |
| 273 | 301 | if ( rest_is_field_included( 'label', $fields ) ) { |
| 274 | - $data['label'] = __( 'Daily writing prompt', 'jetpack' ); | |
| 302 | + if ( $this->is_in_bloganuary( $prompt->post_date_gmt ) ) { | |
| 303 | + $data['label'] = __( 'Bloganuary writing prompt', 'jetpack' ); | |
| 304 | + } else { | |
| 305 | + $data['label'] = __( 'Daily writing prompt', 'jetpack' ); | |
| 306 | + } | |
| 275 | 307 | } |
| 276 | 308 | |
| 277 | 309 | if ( rest_is_field_included( 'text', $fields ) ) { |
| 278 | 310 | $text = \BloggingPrompts\prompt_without_blocks( $prompt->post_content ); |
| @@ -298,9 +330,14 @@ | ||
| 298 | 330 | $data['answered_users_sample'] = $this->build_answering_users_sample( $prompt->ID ); |
| 299 | 331 | } |
| 300 | 332 | |
| 301 | 333 | if ( rest_is_field_included( 'answered_link', $fields ) ) { |
| 302 | - $data['answered_link'] = esc_url( "https://wordpress.com/tag/dailyprompt-{$prompt->ID}" ); | |
| 334 | + if ( $this->is_in_bloganuary( $prompt->post_date_gmt ) ) { | |
| 335 | + $bloganuary_id = $this->get_bloganuary_id( $prompt->post_date_gmt ); | |
| 336 | + $data['answered_link'] = esc_url( "https://wordpress.com/tag/{$bloganuary_id}" ); | |
| 337 | + } else { | |
| 338 | + $data['answered_link'] = esc_url( "https://wordpress.com/tag/dailyprompt-{$prompt->ID}" ); | |
| 339 | + } | |
| 303 | 340 | } |
| 304 | 341 | |
| 305 | 342 | if ( rest_is_field_included( 'answered_link_text', $fields ) ) { |
| 306 | 343 | $data['answered_link_text'] = __( 'View all responses', 'jetpack' ); |
| @@ -305,12 +342,47 @@ | ||
| 305 | 342 | if ( rest_is_field_included( 'answered_link_text', $fields ) ) { |
| 306 | 343 | $data['answered_link_text'] = __( 'View all responses', 'jetpack' ); |
| 307 | 344 | } |
| 308 | 345 | |
| 346 | + if ( $this->is_in_bloganuary( $prompt->post_date_gmt ) && rest_is_field_included( 'bloganuary_id', $fields ) ) { | |
| 347 | + $data['bloganuary_id'] = $this->get_bloganuary_id( $prompt->post_date_gmt ); | |
| 348 | + } | |
| 349 | + | |
| 309 | 350 | return $data; |
| 310 | 351 | } |
| 311 | 352 | |
| 312 | 353 | /** |
| 354 | + * Return true if the post is in "Bloganuary" | |
| 355 | + * | |
| 356 | + * @param string $post_date_gmt Unused - Post date in GMT. | |
| 357 | + * @return bool Always returns false as Bloganuary is disabled. | |
| 358 | + */ | |
| 359 | + protected function is_in_bloganuary( $post_date_gmt ) { //phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable | |
| 360 | + | |
| 361 | + /* | |
| 362 | + Disable for January 2025 and beyond (see https://wp.me/p5uIfZ-gxX). | |
| 363 | + Previously, this method would check if the post was published in January: | |
| 364 | + - Extract month from post_date_gmt -- $post_month = gmdate( 'm', strtotime( $post_date_gmt ) ); | |
| 365 | + - Return true if month was '01' -- return $post_month === '01'; | |
| 366 | + */ | |
| 367 | + return false; | |
| 368 | + } | |
| 369 | + | |
| 370 | + /** | |
| 371 | + * Return the bloganuary id of the form `bloganuary-yyyy-dd` | |
| 372 | + * | |
| 373 | + * @param string $post_date_gmt Post date in GMT. | |
| 374 | + * @return string Bloganuary id. | |
| 375 | + */ | |
| 376 | + protected function get_bloganuary_id( $post_date_gmt ) { | |
| 377 | + $post_year_day = gmdate( 'Y-d', strtotime( $post_date_gmt ) ); | |
| 378 | + if ( $this->force_year ) { | |
| 379 | + $post_year_day = $this->force_year . '-' . gmdate( 'd', strtotime( $post_date_gmt ) ); | |
| 380 | + } | |
| 381 | + return 'bloganuary-' . $post_year_day; | |
| 382 | + } | |
| 383 | + | |
| 384 | + /** | |
| 313 | 385 | * Format a date for a blogging prompt, omiting the time. |
| 314 | 386 | * |
| 315 | 387 | * @param string $date_gmt Publish datetime of the prompt in GMT, i.e. 0000-00-00 00:00:00. |
| 316 | 388 | * @param string $date Publish datetime of the prompt, i.e. 0000-00-00 00:00:00. |
| @@ -320,9 +392,9 @@ | ||
| 320 | 392 | $post_date = $date ? $date : $date_gmt; |
| 321 | 393 | $date_obj = date_create( $post_date ); |
| 322 | 394 | |
| 323 | 395 | if ( $this->force_year ) { |
| 324 | - $date_obj->setDate( $this->force_year, $date_obj->format( 'm' ), $date_obj->format( 'd' ) ); | |
| 396 | + $date_obj->setDate( $this->force_year, (int) $date_obj->format( 'n' ), (int) $date_obj->format( 'j' ) ); | |
| 325 | 397 | |
| 326 | 398 | // If ascending by day of year, go to the next year when we pass the last day of the year. |
| 327 | 399 | if ( $date_obj->format( 'm-d' ) === '12-31' ) { |
| 328 | 400 | $this->force_year += 1; |
| @@ -346,9 +418,9 @@ | ||
| 346 | 418 | 'description' => __( 'Show prompts following a given date.', 'jetpack' ), |
| 347 | 419 | 'type' => 'string', |
| 348 | 420 | 'validate_callback' => function ( $param ) { |
| 349 | 421 | // Allow month and day without year, e.g. `--02-28` |
| 350 | - if ( strpos( $param, '-' ) === 0 ) { | |
| 422 | + if ( str_starts_with( $param, '-' ) ) { | |
| 351 | 423 | return false !== date_create_from_format( '--m-d', $param ); |
| 352 | 424 | } |
| 353 | 425 | |
| 354 | 426 | return false !== date_create( $param ); |
| @@ -443,8 +515,12 @@ | ||
| 443 | 515 | 'answered_link_text' => array( |
| 444 | 516 | 'description' => __( 'Text for the link to answers for the prompt.', 'jetpack' ), |
| 445 | 517 | 'type' => 'string', |
| 446 | 518 | ), |
| 519 | + 'bloganuary_id' => array( | |
| 520 | + 'description' => __( 'Id used by the bloganuary promotion', 'jetpack' ), | |
| 521 | + 'type' => 'string', | |
| 522 | + ), | |
| 447 | 523 | ), |
| 448 | 524 | ); |
| 449 | 525 | } |
| 450 | 526 | |
| @@ -474,41 +550,8 @@ | ||
| 474 | 550 | 'rest_cannot_read_prompts', |
| 475 | 551 | __( 'Sorry, you are not allowed to access blogging prompts on this site.', 'jetpack' ), |
| 476 | 552 | array( 'status' => rest_authorization_required_code() ) |
| 477 | 553 | ); |
| 478 | - } | |
| 479 | - | |
| 480 | - /** | |
| 481 | - * Proxy request to wpcom servers for the site and user. | |
| 482 | - * | |
| 483 | - * @param WP_Rest_Request $request Request to proxy. | |
| 484 | - * @param string $path Path to append to the rest base. | |
| 485 | - * @return mixed|WP_Error Response from wpcom servers or an error. | |
| 486 | - */ | |
| 487 | - public function proxy_request_to_wpcom( $request, $path = '' ) { | |
| 488 | - $blog_id = \Jetpack_Options::get_option( 'id' ); | |
| 489 | - $path = '/sites/' . rawurldecode( $blog_id ) . '/' . rawurldecode( $this->rest_base ) . ( $path ? '/' . rawurldecode( $path ) : '' ); | |
| 490 | - $api_url = add_query_arg( $request->get_query_params(), $path ); | |
| 491 | - | |
| 492 | - // Prefer request as user, if possible. Fall back to blog request to show prompt data for unconnected users. | |
| 493 | - $response = ( new Manager() )->is_user_connected() | |
| 494 | - ? Client::wpcom_json_api_request_as_user( $api_url, '3', array(), null, 'wpcom' ) | |
| 495 | - : Client::wpcom_json_api_request_as_blog( $api_url, 'v3', array(), null, 'wpcom' ); | |
| 496 | - | |
| 497 | - if ( is_wp_error( $response ) ) { | |
| 498 | - return $response; | |
| 499 | - } | |
| 500 | - | |
| 501 | - $response_status = wp_remote_retrieve_response_code( $response ); | |
| 502 | - $response_body = json_decode( wp_remote_retrieve_body( $response ) ); | |
| 503 | - | |
| 504 | - if ( $response_status >= 400 ) { | |
| 505 | - $code = isset( $response_body->code ) ? $response_body->code : 'unknown_error'; | |
| 506 | - $message = isset( $response_body->message ) ? $response_body->message : __( 'An unknown error occurred.', 'jetpack' ); | |
| 507 | - return new WP_Error( $code, $message, array( 'status' => $response_status ) ); | |
| 508 | - } | |
| 509 | - | |
| 510 | - return $response_body; | |
| 511 | 554 | } |
| 512 | 555 | |
| 513 | 556 | /** |
| 514 | 557 | * Creates a sample of users who have answered a blogging prompt. |