Context Navigation

← Previous Change
Next Change →

feedLinksExtra.php

Timestamp:

04/11/2022 05:01:04 AM (4 years ago)

Author:

peterwilsoncc

Message:

Feeds: Remove comment feed HTML headers when empty.

Remove the link[rel=alternate] element from the HTML header when the comment feeds are disabled. Previously the HTML element was output with an empty href attribute.

The element is removed if get_post_comments_feed_link() returns an empty string or the feed is disabled via the feed_links_show_comments_feed filter.

Props barryceelen, audrasjb, costdev, rachelbaker, Boniu91.
Fixes #54703.

File:

: 1 edited

trunk/tests/phpunit/tests/general/feedLinksExtra.php (modified) (1 diff)

Legend:

: Unmodified
: Added
: Removed

trunk/tests/phpunit/tests/general/feedLinksExtra.php

-              r53033
+              r53125
                 $this->assertSame( $expected, get_echo( 'feed_links_extra' ) );
+        }
+        /**
+         * @ticket 54703
+         */
+        public function test_feed_links_extra_should_output_nothing_when_show_comments_feed_filter_returns_false() {
+                add_filter( 'feed_links_show_comments_feed', '__return_false' );
+                $this->go_to( get_the_permalink( self::$post_with_comment_id ) );
+                $this->assertEmpty( get_echo( 'feed_links_extra' ) );
+        }
+        /**
+         * @dataProvider data_feed_links_extra_should_output_nothing_when_post_comments_feed_link_is_falsy
+         *
+         * @ticket 54703
+         *
+         * @param string $callback The callback to use for the 'post_comments_feed_link' filter.
+         */
+        public function test_feed_links_extra_should_output_nothing_when_post_comments_feed_link_is_falsy( $callback ) {
+                add_filter( 'post_comments_feed_link', $callback );
+                $this->go_to( get_the_permalink( self::$post_with_comment_id ) );
+                $this->assertEmpty( get_echo( 'feed_links_extra' ) );
+        }
+        /**
+         * Data provider.
+         *
+         * @return array
+         */
+        public function data_feed_links_extra_should_output_nothing_when_post_comments_feed_link_is_falsy() {
+                return array(
+                        'empty string' => array( 'callback' => '__return_empty_string' ),
+                        'empty array'  => array( 'callback' => '__return_empty_array' ),
+                        'zero int'     => array( 'callback' => '__return_zero' ),
+                        'zero float'   => array( 'callback' => array( $this, 'cb_return_zero_float' ) ),
+                        'zero string'  => array( 'callback' => array( $this, 'cb_return_zero_string' ) ),
+                        'null'         => array( 'callback' => '__return_null' ),
+                        'false'        => array( 'callback' => '__return_false' ),
+                );
+        }
+        /**
+         * Callback that returns 0.0.
+         *
+         * @return float 0.0.
+         */
+        public function cb_return_zero_float() {
+                return 0.0;
+        }
+        /**
+         * Callback that returns '0'.
+         *
+         * @return string '0'.
+         */
+        public function cb_return_zero_string() {
+                return '0';
+        }
+        /**
+         * @ticket 54703
+         */
+        public function test_feed_links_extra_should_output_the_comments_feed_link_when_show_comments_feed_filter_returns_true() {
+                add_filter( 'feed_links_show_comments_feed', '__return_true' );
+                $this->go_to( get_the_permalink( self::$post_with_comment_id ) );
+                $this->assertNotEmpty( get_echo( 'feed_links_extra' ) );
+        }
+}

Note: See TracChangeset for help on using the changeset viewer.

Make WordPress Core

Context Navigation

Changeset 53125 for trunk/tests/phpunit/tests/general/feedLinksExtra.php

Legend:

trunk/tests/phpunit/tests/general/feedLinksExtra.php

Download in other formats: