| 283 | /** |
| 284 | * Unfiltered search queries for attachment post types should not inlcude file |
| 285 | * names to ensure the postmeta JOINs don't happen on the front end. |
| 286 | * |
| 287 | * @ticket 22744 |
| 288 | */ |
| 289 | public function test_exclude_file_names_in_attachment_search_by_default() { |
| 290 | $attachment = self::factory()->post->create( array( |
| 291 | 'post_type' => 'attachment', |
| 292 | 'post_status' => 'publish', |
| 293 | 'post_title' => 'bar foo', |
| 294 | 'post_content' => 'foo bar', |
| 295 | 'post_excerpt' => 'This post has foo', |
| 296 | ) ); |
| 297 | |
| 298 | add_post_meta( $attachment, '_wp_attached_file', 'some-image2.png', true ); |
| 299 | |
| 300 | // Pass post_type an array value. |
| 301 | $q = new WP_Query( array( |
| 302 | 's' => 'image2', |
| 303 | 'fields' => 'ids', |
| 304 | 'post_type' => 'attachment', |
| 305 | 'post_status' => 'inherit', |
| 306 | ) ); |
| 307 | |
| 308 | $this->assertNotEquals( array( $attachment ), $q->posts ); |
| 309 | } |
| 310 | |
| 311 | /** |
| 312 | * @ticket 22744 |
| 313 | */ |
| 314 | public function test_include_file_names_in_attachment_search_as_string() { |
| 315 | $attachment = self::factory()->post->create( array( |
| 316 | 'post_type' => 'attachment', |
| 317 | 'post_status' => 'publish', |
| 318 | 'post_title' => 'bar foo', |
| 319 | 'post_content' => 'foo bar', |
| 320 | 'post_excerpt' => 'This post has foo', |
| 321 | ) ); |
| 322 | |
| 323 | add_post_meta( $attachment, '_wp_attached_file', 'some-image1.png', true ); |
| 324 | add_filter( 'posts_clauses', '_filter_query_attachment_filenames' ); |
| 325 | |
| 326 | // Pass post_type a string value. |
| 327 | $q = new WP_Query( array( |
| 328 | 's' => 'image1', |
| 329 | 'fields' => 'ids', |
| 330 | 'post_type' => 'attachment', |
| 331 | 'post_status' => 'inherit', |
| 332 | ) ); |
| 333 | |
| 334 | $this->assertSame( array( $attachment ), $q->posts ); |
| 335 | } |
| 336 | |
| 337 | /** |
| 338 | * @ticket 22744 |
| 339 | */ |
| 340 | public function test_include_file_names_in_attachment_search_as_array() { |
| 341 | $attachment = self::factory()->post->create( array( |
| 342 | 'post_type' => 'attachment', |
| 343 | 'post_status' => 'publish', |
| 344 | 'post_title' => 'bar foo', |
| 345 | 'post_content' => 'foo bar', |
| 346 | 'post_excerpt' => 'This post has foo', |
| 347 | ) ); |
| 348 | |
| 349 | add_post_meta( $attachment, '_wp_attached_file', 'some-image2.png', true ); |
| 350 | add_filter( 'posts_clauses', '_filter_query_attachment_filenames' ); |
| 351 | |
| 352 | // Pass post_type an array value. |
| 353 | $q = new WP_Query( array( |
| 354 | 's' => 'image2', |
| 355 | 'fields' => 'ids', |
| 356 | 'post_type' => array( 'attachment' ), |
| 357 | 'post_status' => 'inherit', |
| 358 | ) ); |
| 359 | |
| 360 | $this->assertSame( array( $attachment ), $q->posts ); |
| 361 | } |
| 362 | |
| 363 | /** |
| 364 | * @ticket 22744 |
| 365 | */ |
| 366 | public function test_exclude_attachment_file_names_in_general_searches() { |
| 367 | $attachment = self::factory()->post->create( array( |
| 368 | 'post_type' => 'attachment', |
| 369 | 'post_status' => 'publish', |
| 370 | 'post_title' => 'bar foo', |
| 371 | 'post_content' => 'foo bar', |
| 372 | 'post_excerpt' => 'This post has foo', |
| 373 | ) ); |
| 374 | |
| 375 | add_post_meta( $attachment, '_wp_attached_file', 'some-image3.png', true ); |
| 376 | |
| 377 | $q = new WP_Query( array( |
| 378 | 's' => 'image3', |
| 379 | 'fields' => 'ids', |
| 380 | 'post_type' => array( 'post', 'page', 'attachment' ), |
| 381 | 'post_status' => 'inherit', |
| 382 | ) ); |
| 383 | |
| 384 | $this->assertNotEquals( array( $attachment ), $q->posts ); |
| 385 | } |
| 386 | |