Changeset 46564 for trunk/tests/phpunit/tests/formatting/MakeClickable.php

Timestamp:

10/21/2019 08:39:16 PM (7 years ago)

Author:

SergeyBiryukov

Message:

Comments: Add rel="nofollow ugc" attribute when converting plain URLs to <a> tags in comments via make_clickable().

Introduce make_clickable_rel filter for the rel value that is added to URL matches converted to links.

This is a follow-up to [46349], which added the rel="nofollow ugc" attribute to existing <a> tags in comments via wp_rel_ugc().

UGC stands for User Generated Content, and the ugc attribute value is recommended for links within user generated content, such as comments and forum posts.

See ​https://webmasters.googleblog.com/2019/09/evolving-nofollow-new-ways-to-identify.html.

Props blogginglife, SergeyBiryukov.
Reviewed by desrosj, audrasjb.
Fixes #48022.

File:

• trunk/tests/phpunit/tests/formatting/MakeClickable.php (modified) (2 diffs)

trunk/tests/phpunit/tests/formatting/MakeClickable.php

@@ -376 +376 @@
 
         /**
+         * @ticket 30162
          * @dataProvider data_script_and_style_tags
-         * @ticket 30162
          */
         public function test_dont_link_script_and_style_tags( $tag ) {
@@ -400 +400 @@
         }
 
+        /**
+         * @ticket 48022
+         * @dataProvider data_add_rel_ugc_in_comments
+         */
+        public function test_add_rel_ugc_in_comments( $content, $expected ) {
+                $comment_id = self::factory()->comment->create(
+                        array(
+                                'comment_content' => $content,
+                        )
+                );
+
+                ob_start();
+                comment_text( $comment_id );
+                $comment_text = ob_get_clean();
+
+                $this->assertContains( $expected, make_clickable( $comment_text ) );
+        }
+
+        public function data_add_rel_ugc_in_comments() {
+                return array(
+                        array(
+                                'http://wordpress.org',
+                                '<a href="http://wordpress.org" rel="nofollow ugc">http://wordpress.org</a>',
+                        ),
+                        array(
+                                'www.wordpress.org',
+                                '<p><a href="http://www.wordpress.org" rel="nofollow ugc">http://www.wordpress.org</a>',
+                        ),
+                );
+        }
+
 }