| @@ -8,11 +8,35 @@ | ||
| 8 | 8 | * |
| 9 | 9 | * @since 1.4 |
| 10 | 10 | */ |
| 11 | 11 | class PLL_Filters { |
| 12 | - public $links_model, $model, $options, $curlang; | |
| 12 | + /** | |
| 13 | + * Stores the plugin options. | |
| 14 | + * | |
| 15 | + * @var array | |
| 16 | + */ | |
| 17 | + public $options; | |
| 13 | 18 | |
| 14 | 19 | /** |
| 20 | + * @var PLL_Model | |
| 21 | + */ | |
| 22 | + public $model; | |
| 23 | + | |
| 24 | + /** | |
| 25 | + * Instance of a child class of PLL_Links_Model. | |
| 26 | + * | |
| 27 | + * @var PLL_Links_Model | |
| 28 | + */ | |
| 29 | + public $links_model; | |
| 30 | + | |
| 31 | + /** | |
| 32 | + * Current language. | |
| 33 | + * | |
| 34 | + * @var PLL_Language|null | |
| 35 | + */ | |
| 36 | + public $curlang; | |
| 37 | + | |
| 38 | + /** | |
| 15 | 39 | * Constructor: setups filters |
| 16 | 40 | * |
| 17 | 41 | * @since 1.4 |
| 18 | 42 | * |
| @@ -23,8 +47,13 @@ | ||
| 23 | 47 | $this->model = &$polylang->model; |
| 24 | 48 | $this->options = &$polylang->options; |
| 25 | 49 | $this->curlang = &$polylang->curlang; |
| 26 | 50 | |
| 51 | + // Deletes our cache for sticky posts when the list is updated. | |
| 52 | + add_action( 'update_option_sticky_posts', array( $this, 'delete_sticky_posts_cache' ) ); | |
| 53 | + add_action( 'add_option_sticky_posts', array( $this, 'delete_sticky_posts_cache' ) ); | |
| 54 | + add_action( 'delete_option_sticky_posts', array( $this, 'delete_sticky_posts_cache' ) ); | |
| 55 | + | |
| 27 | 56 | // Filters the comments according to the current language |
| 28 | 57 | add_action( 'parse_comment_query', array( $this, 'parse_comment_query' ) ); |
| 29 | 58 | add_filter( 'comments_clauses', array( $this, 'comments_clauses' ), 10, 2 ); |
| 30 | 59 | |
| @@ -39,11 +68,8 @@ | ||
| 39 | 68 | |
| 40 | 69 | // Converts the locale to a valid W3C locale |
| 41 | 70 | add_filter( 'language_attributes', array( $this, 'language_attributes' ) ); |
| 42 | 71 | |
| 43 | - // Prevents deleting all the translations of the default category | |
| 44 | - add_filter( 'map_meta_cap', array( $this, 'fix_delete_default_category' ), 10, 4 ); | |
| 45 | - | |
| 46 | 72 | // Translate the site title in emails sent to users |
| 47 | 73 | add_filter( 'password_change_email', array( $this, 'translate_user_email' ) ); |
| 48 | 74 | add_filter( 'email_change_email', array( $this, 'translate_user_email' ) ); |
| 49 | 75 | |
| @@ -52,67 +78,100 @@ | ||
| 52 | 78 | add_filter( 'map_meta_cap', array( $this, 'fix_privacy_policy_page_editing' ), 10, 4 ); |
| 53 | 79 | |
| 54 | 80 | // Personal data exporter |
| 55 | 81 | add_filter( 'wp_privacy_personal_data_exporters', array( $this, 'register_personal_data_exporter' ), 0 ); // Since WP 4.9.6 |
| 82 | + | |
| 83 | + // Fix for `term_exists()`. | |
| 84 | + add_filter( 'term_exists_default_query_args', array( $this, 'term_exists_default_query_args' ), 0, 3 ); // Since WP 6.0.0. | |
| 56 | 85 | } |
| 57 | 86 | |
| 58 | 87 | /** |
| 59 | - * Get the language to filter a comments query | |
| 88 | + * Deletes the cache for multilingual sticky posts. | |
| 60 | 89 | * |
| 90 | + * @since 2.8.4 | |
| 91 | + * | |
| 92 | + * @return void | |
| 93 | + */ | |
| 94 | + public function delete_sticky_posts_cache() { | |
| 95 | + wp_cache_delete( 'sticky_posts', 'options' ); | |
| 96 | + } | |
| 97 | + | |
| 98 | + /** | |
| 99 | + * Get the language to filter a comments query. | |
| 100 | + * | |
| 61 | 101 | * @since 2.0 |
| 102 | + * @since 3.1 Always returns an array. Renamed from get_comments_queried_language(). | |
| 62 | 103 | * |
| 63 | - * @param object $query | |
| 64 | - * @return object|bool the language(s) to use in the filter, false otherwise | |
| 104 | + * @param WP_Comment_Query $query WP_Comment_Query object. | |
| 105 | + * @return PLL_Language[] The languages to use in the filter. | |
| 65 | 106 | */ |
| 66 | - protected function get_comments_queried_language( $query ) { | |
| 67 | - // Don't filter comments if comment ids or post ids are specified | |
| 107 | + protected function get_comments_queried_languages( $query ) { | |
| 108 | + // Don't filter comments if comment ids or post ids are specified. | |
| 68 | 109 | $plucked = wp_array_slice_assoc( $query->query_vars, array( 'comment__in', 'parent', 'post_id', 'post__in', 'post_parent' ) ); |
| 69 | 110 | $fields = array_filter( $plucked ); |
| 70 | 111 | if ( ! empty( $fields ) ) { |
| 71 | - return false; | |
| 112 | + return array(); | |
| 72 | 113 | } |
| 73 | 114 | |
| 74 | - // Don't filter comments if a non translated post type is specified | |
| 115 | + // Don't filter comments if a non translated post type is specified. | |
| 75 | 116 | if ( ! empty( $query->query_vars['post_type'] ) && ! $this->model->is_translated_post_type( $query->query_vars['post_type'] ) ) { |
| 76 | - return false; | |
| 117 | + return array(); | |
| 77 | 118 | } |
| 78 | 119 | |
| 79 | - return empty( $query->query_vars['lang'] ) ? $this->curlang : $this->model->get_language( $query->query_vars['lang'] ); | |
| 120 | + // If comments are queried with a 'lang' parameter, keeps only language codes. | |
| 121 | + if ( isset( $query->query_vars['lang'] ) ) { | |
| 122 | + $languages = is_string( $query->query_vars['lang'] ) ? explode( ',', $query->query_vars['lang'] ) : $query->query_vars['lang']; | |
| 123 | + if ( is_array( $languages ) ) { | |
| 124 | + $languages = array_map( array( $this->model, 'get_language' ), $languages ); | |
| 125 | + return array_filter( $languages ); | |
| 126 | + } | |
| 127 | + } | |
| 128 | + | |
| 129 | + if ( ! empty( $this->curlang ) ) { | |
| 130 | + return array( $this->curlang ); | |
| 131 | + } | |
| 132 | + | |
| 133 | + return array(); | |
| 80 | 134 | } |
| 81 | 135 | |
| 82 | 136 | /** |
| 83 | - * Adds language dependent cache domain when querying comments | |
| 84 | - * Useful as the 'lang' parameter is not included in cache key by WordPress | |
| 85 | - * Needed since WP 4.6 as comments have been added to persistent cache. See #36906, #37419 | |
| 137 | + * Adds a language dependent cache domain when querying comments. | |
| 138 | + * Useful as the 'lang' parameter is not included in cache key by WordPress. | |
| 139 | + * Needed since WP 4.6 as comments have been added to persistent cache. See #36906, #37419. | |
| 86 | 140 | * |
| 87 | 141 | * @since 2.0 |
| 88 | 142 | * |
| 89 | - * @param object $query | |
| 143 | + * @param WP_Comment_Query $query WP_Comment_Query object. | |
| 144 | + * @return void | |
| 90 | 145 | */ |
| 91 | 146 | public function parse_comment_query( $query ) { |
| 92 | - if ( $lang = $this->get_comments_queried_language( $query ) ) { | |
| 93 | - $key = '_' . ( is_array( $lang ) ? implode( ',', $lang ) : $this->model->get_language( $lang )->slug ); | |
| 147 | + $lang = $this->get_comments_queried_languages( $query ); | |
| 148 | + if ( ! empty( $lang ) ) { | |
| 149 | + $lang = wp_list_pluck( $lang, 'slug' ); | |
| 150 | + $key = '_' . implode( ',', $lang ); | |
| 94 | 151 | $query->query_vars['cache_domain'] = empty( $query->query_vars['cache_domain'] ) ? 'pll' . $key : $query->query_vars['cache_domain'] . $key; |
| 95 | 152 | } |
| 96 | 153 | } |
| 97 | 154 | |
| 98 | 155 | /** |
| 99 | - * Filters the comments according to the current language | |
| 100 | - * Used by the recent comments widget and admin language filter | |
| 156 | + * Filters the comments according to the current language. | |
| 157 | + * Used by the recent comments widget and admin language filter. | |
| 101 | 158 | * |
| 102 | 159 | * @since 0.2 |
| 103 | 160 | * |
| 104 | - * @param array $clauses sql clauses | |
| 105 | - * @param object $query WP_Comment_Query object | |
| 106 | - * @return array modified $clauses | |
| 161 | + * @param string[] $clauses SQL clauses. | |
| 162 | + * @param WP_Comment_Query $query WP_Comment_Query object. | |
| 163 | + * @return string[] Modified $clauses. | |
| 107 | 164 | */ |
| 108 | 165 | public function comments_clauses( $clauses, $query ) { |
| 109 | 166 | global $wpdb; |
| 110 | 167 | |
| 111 | - $lang = $this->get_comments_queried_language( $query ); | |
| 168 | + $lang = $this->get_comments_queried_languages( $query ); | |
| 112 | 169 | |
| 113 | 170 | if ( ! empty( $lang ) ) { |
| 114 | - // If this clause is not already added by WP | |
| 171 | + $lang = wp_list_pluck( $lang, 'slug' ); | |
| 172 | + | |
| 173 | + // If this clause is not already added by WP. | |
| 115 | 174 | if ( ! strpos( $clauses['join'], '.ID' ) ) { |
| 116 | 175 | $clauses['join'] .= " JOIN $wpdb->posts ON $wpdb->posts.ID = $wpdb->comments.comment_post_ID"; |
| 117 | 176 | } |
| 118 | 177 | |
| @@ -122,15 +181,15 @@ | ||
| 122 | 181 | return $clauses; |
| 123 | 182 | } |
| 124 | 183 | |
| 125 | 184 | /** |
| 126 | - * Filters get_pages per language | |
| 185 | + * Filters get_pages() per language. | |
| 127 | 186 | * |
| 128 | 187 | * @since 1.4 |
| 129 | 188 | * |
| 130 | - * @param array $pages an array of pages already queried | |
| 131 | - * @param array $args get_pages arguments | |
| 132 | - * @return array modified list of pages | |
| 189 | + * @param WP_Post[] $pages An array of pages already queried. | |
| 190 | + * @param array $args Array of get_pages() arguments. | |
| 191 | + * @return WP_Post[] Modified list of pages. | |
| 133 | 192 | */ |
| 134 | 193 | public function get_pages( $pages, $args ) { |
| 135 | 194 | if ( isset( $args['lang'] ) && empty( $args['lang'] ) ) { |
| 136 | 195 | return $pages; |
| @@ -143,38 +202,22 @@ | ||
| 143 | 202 | } |
| 144 | 203 | |
| 145 | 204 | static $once = false; |
| 146 | 205 | |
| 147 | - // Obliged to redo the get_pages query if we want to get the right number | |
| 148 | 206 | if ( ! empty( $args['number'] ) && ! $once ) { |
| 149 | - $once = true; // avoid infinite loop | |
| 207 | + // We are obliged to redo the get_pages() query if we want to get the right number. | |
| 208 | + $once = true; // Avoid infinite loop. | |
| 150 | 209 | |
| 151 | - $r = array( | |
| 152 | - 'lang' => 0, // So this query is not filtered | |
| 153 | - 'numberposts' => -1, | |
| 154 | - 'nopaging' => true, | |
| 155 | - 'post_type' => $args['post_type'], | |
| 156 | - 'fields' => 'ids', | |
| 157 | - 'tax_query' => array( | |
| 158 | - array( | |
| 159 | - 'taxonomy' => 'language', | |
| 160 | - 'field' => 'term_taxonomy_id', // Since WP 3.5 | |
| 161 | - 'terms' => $language->term_taxonomy_id, | |
| 162 | - 'operator' => 'NOT IN', | |
| 163 | - ), | |
| 164 | - ), | |
| 165 | - ); | |
| 166 | - | |
| 167 | - // Take care that 'exclude' argument accepts integer or strings too | |
| 168 | - $args['exclude'] = array_merge( wp_parse_id_list( $args['exclude'] ), get_posts( $r ) ); | |
| 210 | + // Take care that 'exclude' argument accepts integer or strings too. | |
| 211 | + $args['exclude'] = array_merge( wp_parse_id_list( $args['exclude'] ), $this->get_related_page_ids( $language, 'NOT IN', $args ) ); // phpcs:ignore WordPressVIPMinimum.Performance.WPQueryParams.PostNotIn_exclude | |
| 169 | 212 | $pages = get_pages( $args ); |
| 170 | 213 | } |
| 171 | 214 | |
| 172 | 215 | $ids = wp_list_pluck( $pages, 'ID' ); |
| 173 | 216 | |
| 174 | - // Filters the queried list of pages by language | |
| 175 | 217 | if ( ! $once ) { |
| 176 | - $ids = array_intersect( $ids, $this->model->post->get_objects_in_language( $language ) ); | |
| 218 | + // Filters the queried list of pages by language. | |
| 219 | + $ids = array_intersect( $ids, $this->get_related_page_ids( $language, 'IN', $args ) ); | |
| 177 | 220 | |
| 178 | 221 | foreach ( $pages as $key => $page ) { |
| 179 | 222 | if ( ! in_array( $page->ID, $ids ) ) { |
| 180 | 223 | unset( $pages[ $key ] ); |
| @@ -180,47 +223,78 @@ | ||
| 180 | 223 | unset( $pages[ $key ] ); |
| 181 | 224 | } |
| 182 | 225 | } |
| 183 | 226 | |
| 184 | - $pages = array_values( $pages ); // In case 3rd parties suppose the existence of $pages[0] | |
| 227 | + $pages = array_values( $pages ); // In case 3rd parties suppose the existence of $pages[0]. | |
| 185 | 228 | } |
| 186 | 229 | |
| 187 | - // Not done by WP but extremely useful for performance when manipulating taxonomies | |
| 230 | + // Not done by WP but extremely useful for performance when manipulating taxonomies. | |
| 188 | 231 | update_object_term_cache( $ids, $args['post_type'] ); |
| 189 | 232 | |
| 190 | - $once = false; // In case get_pages is called another time | |
| 233 | + $once = false; // In case get_pages() is called another time. | |
| 191 | 234 | return $pages; |
| 192 | 235 | } |
| 193 | 236 | |
| 194 | 237 | /** |
| 195 | - * Modifies the sql request for get_adjacent_post to filter by the current language | |
| 238 | + * Get page ids related to a get_pages() in or not in a given language. | |
| 196 | 239 | * |
| 240 | + * @since 3.2 | |
| 241 | + * | |
| 242 | + * @param PLL_Language $language The language to use in the relationship | |
| 243 | + * @param string $relation 'IN' or 'NOT IN'. | |
| 244 | + * @param array $args Array of get_pages() arguments. | |
| 245 | + * @return int[] | |
| 246 | + */ | |
| 247 | + protected function get_related_page_ids( $language, $relation, $args ) { | |
| 248 | + $r = array( | |
| 249 | + 'lang' => '', // Ensure this query is not filtered. | |
| 250 | + 'numberposts' => -1, | |
| 251 | + 'nopaging' => true, | |
| 252 | + 'post_type' => $args['post_type'], | |
| 253 | + 'post_status' => $args['post_status'], | |
| 254 | + 'fields' => 'ids', | |
| 255 | + 'tax_query' => array( | |
| 256 | + array( | |
| 257 | + 'taxonomy' => 'language', | |
| 258 | + 'field' => 'term_taxonomy_id', // Since WP 3.5. | |
| 259 | + 'terms' => $language->term_taxonomy_id, | |
| 260 | + 'operator' => $relation, | |
| 261 | + ), | |
| 262 | + ), | |
| 263 | + ); | |
| 264 | + | |
| 265 | + return get_posts( $r ); | |
| 266 | + } | |
| 267 | + | |
| 268 | + /** | |
| 269 | + * Modifies the sql request for get_adjacent_post to filter by the current language. | |
| 270 | + * | |
| 197 | 271 | * @since 0.1 |
| 198 | 272 | * |
| 199 | 273 | * @param string $sql The JOIN clause in the SQL. |
| 200 | 274 | * @param bool $in_same_term Whether post should be in a same taxonomy term. |
| 201 | - * @param array $excluded_terms Array of excluded term IDs. | |
| 275 | + * @param int[] $excluded_terms Array of excluded term IDs. | |
| 202 | 276 | * @param string $taxonomy Taxonomy. Used to identify the term used when `$in_same_term` is true. |
| 203 | 277 | * @param WP_Post $post WP_Post object. |
| 204 | - * @return string modified JOIN clause | |
| 278 | + * @return string Modified JOIN clause. | |
| 205 | 279 | */ |
| 206 | - public function posts_join( $sql, $in_same_term, $excluded_terms, $taxonomy = '', $post = null ) { | |
| 280 | + public function posts_join( $sql, $in_same_term, $excluded_terms, $taxonomy, $post ) { | |
| 207 | 281 | return $this->model->is_translated_post_type( $post->post_type ) && ! empty( $this->curlang ) ? $sql . $this->model->post->join_clause( 'p' ) : $sql; |
| 208 | 282 | } |
| 209 | 283 | |
| 210 | 284 | /** |
| 211 | - * Modifies the sql request for wp_get_archives and get_adjacent_post to filter by the current language | |
| 285 | + * Modifies the sql request for wp_get_archives and get_adjacent_post to filter by the current language. | |
| 212 | 286 | * |
| 213 | 287 | * @since 0.1 |
| 214 | 288 | * |
| 215 | 289 | * @param string $sql The WHERE clause in the SQL. |
| 216 | 290 | * @param bool $in_same_term Whether post should be in a same taxonomy term. |
| 217 | - * @param array $excluded_terms Array of excluded term IDs. | |
| 291 | + * @param int[] $excluded_terms Array of excluded term IDs. | |
| 218 | 292 | * @param string $taxonomy Taxonomy. Used to identify the term used when `$in_same_term` is true. |
| 219 | 293 | * @param WP_Post $post WP_Post object. |
| 220 | - * @return string modified WHERE clause | |
| 294 | + * @return string Modified WHERE clause. | |
| 221 | 295 | */ |
| 222 | - public function posts_where( $sql, $in_same_term, $excluded_terms, $taxonomy = '', $post = null ) { | |
| 296 | + public function posts_where( $sql, $in_same_term, $excluded_terms, $taxonomy, $post ) { | |
| 223 | 297 | return $this->model->is_translated_post_type( $post->post_type ) && ! empty( $this->curlang ) ? $sql . $this->model->post->where_clause( $this->curlang ) : $sql; |
| 224 | 298 | } |
| 225 | 299 | |
| 226 | 300 | /** |
| @@ -238,40 +312,15 @@ | ||
| 238 | 312 | return $output; |
| 239 | 313 | } |
| 240 | 314 | |
| 241 | 315 | /** |
| 242 | - * Prevents deleting all the translations of the default category | |
| 243 | - * | |
| 244 | - * @since 2.1 | |
| 245 | - * | |
| 246 | - * @param array $caps The user's actual capabilities. | |
| 247 | - * @param string $cap Capability name. | |
| 248 | - * @param int $user_id The user ID. | |
| 249 | - * @param array $args Adds the context to the cap. The category id. | |
| 250 | - * @return array | |
| 251 | - */ | |
| 252 | - public function fix_delete_default_category( $caps, $cap, $user_id, $args ) { | |
| 253 | - if ( 'delete_term' === $cap ) { | |
| 254 | - $term = get_term( reset( $args ) ); // Since WP 4.4, we can get the term to get the taxonomy | |
| 255 | - if ( $term instanceof WP_Term ) { | |
| 256 | - $default_cat = get_option( 'default_' . $term->taxonomy ); | |
| 257 | - if ( $default_cat && array_intersect( $args, $this->model->term->get_translations( $default_cat ) ) ) { | |
| 258 | - $caps[] = 'do_not_allow'; | |
| 259 | - } | |
| 260 | - } | |
| 261 | - } | |
| 262 | - | |
| 263 | - return $caps; | |
| 264 | - } | |
| 265 | - | |
| 266 | - /** | |
| 267 | 316 | * Translates the site title in emails sent to the user (change email, reset password) |
| 268 | 317 | * It is necessary to filter the email because WP evaluates the site title before calling switch_to_locale() |
| 269 | 318 | * |
| 270 | 319 | * @since 2.1.3 |
| 271 | 320 | * |
| 272 | - * @param array $email | |
| 273 | - * @return array | |
| 321 | + * @param string[] $email Email contents. | |
| 322 | + * @return string[] Translated email contents. | |
| 274 | 323 | */ |
| 275 | 324 | public function translate_user_email( $email ) { |
| 276 | 325 | $blog_name = wp_specialchars_decode( pll__( get_option( 'blogname' ) ), ENT_QUOTES ); |
| 277 | 326 | $email['subject'] = sprintf( $email['subject'], $blog_name ); |
| @@ -318,9 +367,9 @@ | ||
| 318 | 367 | * |
| 319 | 368 | * @since 2.3.6 |
| 320 | 369 | * |
| 321 | 370 | * @param array $exporters Personal data exporters |
| 322 | - * @retun array | |
| 371 | + * @return array | |
| 323 | 372 | */ |
| 324 | 373 | public function register_personal_data_exporter( $exporters ) { |
| 325 | 374 | $exporters[] = array( |
| 326 | 375 | 'exporter_friendly_name' => __( 'Translated user descriptions', 'polylang' ), |
| @@ -366,6 +415,34 @@ | ||
| 366 | 415 | return array( |
| 367 | 416 | 'data' => $data_to_export, |
| 368 | 417 | 'done' => true, |
| 369 | 418 | ); |
| 419 | + } | |
| 420 | + | |
| 421 | + /** | |
| 422 | + * Filters default query arguments for checking if a term exists. | |
| 423 | + * In `term_exists()`, WP 6.0 uses `get_terms()`, which is filtered by language by Polylang. | |
| 424 | + * This filter prevents `term_exists()` to be filtered by language. | |
| 425 | + * | |
| 426 | + * @since 3.2 | |
| 427 | + * | |
| 428 | + * @param array $defaults An array of arguments passed to get_terms(). | |
| 429 | + * @param int|string $term The term to check. Accepts term ID, slug, or name. | |
| 430 | + * @param string $taxonomy The taxonomy name to use. An empty string indicates the search is against all taxonomies. | |
| 431 | + * @return array | |
| 432 | + */ | |
| 433 | + public function term_exists_default_query_args( $defaults, $term, $taxonomy ) { | |
| 434 | + if ( ! empty( $taxonomy ) && ! $this->model->is_translated_taxonomy( $taxonomy ) ) { | |
| 435 | + return $defaults; | |
| 436 | + } | |
| 437 | + | |
| 438 | + if ( ! is_array( $defaults ) ) { | |
| 439 | + $defaults = array(); | |
| 440 | + } | |
| 441 | + | |
| 442 | + if ( ! isset( $defaults['lang'] ) ) { | |
| 443 | + $defaults['lang'] = ''; | |
| 444 | + } | |
| 445 | + | |
| 446 | + return $defaults; | |
| 370 | 447 | } |
| 371 | 448 | } |
