WordPress.org

Make WordPress Core

Ticket #36905: 36905.6.patch

File 36905.6.patch, 5.7 KB (added by pcfreak30, 11 months ago)

patch refresh

  • tests/phpunit/tests/post/getPageByTitle.php

    IDEA additional info:
    Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
    <+>UTF-8
     
     1<?php
     2
     3/**
     4 * @group post
     5 */
     6class Tests_Post_GetPageByTitle extends WP_UnitTestCase {
     7
     8        /**
     9         * @ticket 36905
     10         */
     11        public function test_get_page_by_title() {
     12                $page_id       = self::factory()->post->create_object( array( 'post_title' => 'Some Page', 'post_type' => 'page' ) );
     13                $page = get_page_by_title( 'some Page' );
     14                $this->assertEquals( $page_id, $page->ID );
     15        }
     16
     17        /**
     18         * @ticket 36905
     19         */
     20        public function test_should_not_care_about_cases() {
     21                $page_id       = self::factory()->post->create_object( array( 'post_title' => 'Some Page', 'post_type' => 'page' ) );
     22                $page = get_page_by_title( 'soMe paGE' );
     23                $this->assertEquals( $page_id, $page->ID );
     24        }
     25
     26        /**
     27         * @ticket 36905
     28         */
     29        public function test_should_match_exact_title() {
     30                $page_id       = self::factory()->post->create_object( array( 'post_title' => 'Some Page', 'post_type' => 'page' ) );
     31
     32                $page = get_page_by_title( 'Some Page Yeah' );
     33                $this->assertNull( $page );
     34
     35                $page = get_page_by_title( 'Yeah Some Page' );
     36                $this->assertNull( $page );
     37        }
     38
     39        /**
     40         * @ticket 36905
     41         */
     42        public function test_should_return_one_result_if_some_posts_has_the_same_title() {
     43                $post_id_1       = self::factory()->post->create_object( array( 'post_title' => 'Some Title', 'post_type' => 'page' ) );
     44                $post_id_2       = self::factory()->post->create_object( array( 'post_title' => 'Some Title', 'post_type' => 'page' ) );
     45                $post = get_page_by_title( 'Some Title' );
     46                $this->assertEquals( $post_id_1, $post->ID );
     47        }
     48
     49        /**
     50         * @ticket 36905
     51         */
     52        public function test_should_obey_post_type() {
     53                register_post_type( 'my-cpt' );
     54                $post_id       = self::factory()->post->create_object( array( 'post_title' => 'Some CPT', 'post_type' => 'my-cpt' ) );
     55
     56                $post = get_page_by_title( 'Some CPT', OBJECT, 'my-cpt' );
     57                $this->assertEquals( $post_id, $post->ID );
     58
     59                $post = get_page_by_title( 'Some CPT', OBJECT, 'page' );
     60                $this->assertNull( $post );
     61        }
     62
     63        /**
     64         * @ticket 36905
     65         */
     66        public function test_should_get_different_post_types() {
     67                register_post_type( 'my-cpt' );
     68                $post_id_1       = self::factory()->post->create_object( array( 'post_title' => 'Some CPT', 'post_type' => 'my-cpt' ) );
     69                $post_id_2       = self::factory()->post->create_object( array( 'post_title' => 'Some Page', 'post_type' => 'page' ) );
     70                $post = get_page_by_title( 'Some Page', OBJECT, array( 'my-cpt', 'page' ) );
     71                $this->assertEquals( $post_id_2, $post->ID );
     72        }
     73
     74        /**
     75         * @ticket 36905
     76         */
     77        public function test_should_get_different_post_statuses() {
     78                register_post_type( 'my-cpt' );
     79
     80                $post_statuses = get_post_stati();
     81                foreach ( $post_statuses as $post_status ) {
     82                        $title = sprintf( 'Some %s post', $post_status );
     83                        $post_id       = self::factory()->post->create_object( array( 'post_title' => $title, 'post_type' => 'page', 'post_status' => $post_status ) );
     84                        $found = get_page_by_title( $title );
     85                        $this->assertEquals( $post_id, $found->ID );
     86                }
     87        }
     88
     89        /**
     90         * @ticket 36905
     91         */
     92        public function test_output_param_should_be_obeyed() {
     93                $post_id       = self::factory()->post->create_object( array( 'post_title' => 'Some Page', 'post_type' => 'page' ) );
     94
     95                $found = get_page_by_title( 'Some Page' );
     96                $this->assertInternalType( 'object', $found );
     97                $this->assertSame( $post_id, $found->ID );
     98
     99                $found = get_page_by_title( 'Some Page', OBJECT );
     100                $this->assertInternalType( 'object', $found );
     101                $this->assertSame( $post_id, $found->ID );
     102
     103                $found = get_page_by_title( 'Some Page', ARRAY_N );
     104                $this->assertInternalType( 'array', $found );
     105                $this->assertSame( $post_id, $found[0] );
     106
     107                $found = get_page_by_title( 'Some Page', ARRAY_A );
     108                $this->assertInternalType( 'array', $found );
     109                $this->assertSame( $post_id, $found['ID'] );
     110        }
     111
     112
     113
     114}
  • wp-includes/post.php

    IDEA additional info:
    Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
    <+>UTF-8
     
    47094709 * @return WP_Post|array|null WP_Post (or array) on success, or null on failure.
    47104710 */
    47114711function get_page_by_title( $page_title, $output = OBJECT, $post_type = 'page' ) {
    4712         global $wpdb;
     4712        // Make sure that we even query post statuses that are excluded from search
     4713        $post_status = array_values( get_post_stati( array( 'exclude_from_search' => true ) ) );
    47134714
    4714         if ( is_array( $post_type ) ) {
    4715                 $post_type           = esc_sql( $post_type );
    4716                 $post_type_in_string = "'" . implode( "','", $post_type ) . "'";
    4717                 $sql                 = $wpdb->prepare(
    4718                         "
    4719                         SELECT ID
    4720                         FROM $wpdb->posts
    4721                         WHERE post_title = %s
    4722                         AND post_type IN ($post_type_in_string)
    4723                 ",
    4724                         $page_title
    4725                 );
    4726         } else {
    4727                 $sql = $wpdb->prepare(
    4728                         "
    4729                         SELECT ID
    4730                         FROM $wpdb->posts
    4731                         WHERE post_title = %s
    4732                         AND post_type = %s
    4733                 ",
    4734                         $page_title,
    4735                         $post_type
    4736                 );
    4737         }
    4738 
    4739         $page = $wpdb->get_var( $sql );
     4715        $query = new WP_Query(
     4716                array(
     4717                        'title'          => $page_title,
     4718                        'post_type'      => $post_type,
     4719                        'posts_per_page' => 1,
     4720                        'post_status'    => array_merge( $post_status, array( 'any' ) ),
     4721                        'orderby'        => 'none',
     4722                        'fields'         => 'ids',
     4723                )
     4724        );
     4725        $posts = $query->get_posts();
    47404726
    4741         if ( $page ) {
    4742                 return get_post( $page, $output );
     4727        if ( $posts ) {
     4728                return get_post( $posts[0], $output );
    47434729        }
     4730
     4731        return null;
    47444732}
    47454733
    47464734/**