Ticket #30162: 30162.diff

src/wp-includes/formatting.php

@@ -1872 +1872 @@
         $nested_code_pre = 0; // Keep track of how many levels link is nested inside <pre> or <code>
         foreach ( $textarr as $piece ) {
 
-                if ( preg_match( '|^<code[\s>]|i', $piece ) || preg_match( '|^<pre[\s>]|i', $piece ) )
+                if ( preg_match( '|^<code[\s>]|i', $piece ) || preg_match( '|^<pre[\s>]|i', $piece ) || preg_match( '|^<script[\s>]|i', $piece ) || preg_match( '|^<style[\s>]|i', $piece ) )
                         $nested_code_pre++;
-                elseif ( ( '</code>' === strtolower( $piece ) || '</pre>' === strtolower( $piece ) ) && $nested_code_pre )
+                elseif ( $nested_code_pre && ( '</code>' === strtolower( $piece ) || '</pre>' === strtolower( $piece ) || '</script>' === strtolower( $piece ) || '</style>' === strtolower( $piece ) ) )
                         $nested_code_pre--;
 
                 if ( $nested_code_pre || empty( $piece ) || ( $piece[0] === '<' && ! preg_match( '|^<\s*[\w]{1,20}+://|', $piece ) ) ) {

tests/phpunit/tests/formatting/MakeClickable.php

@@ -372 +372 @@
                                   href='mailto:someone@example.com'>someone@example.com</a>";
                 $this->assertEquals( $html, make_clickable( $html ) );
         }
+
+        /**
+         * @ticket 30162
+         */
+        function test_dont_link_script_and_style_tags() {
+                $before = array(
+                        '<script>http://wordpress.org</script>',
+                        '<style>http://wordpress.org</style>',
+                        '<script type="text/javascript">http://wordpress.org</script>',
+                        '<style type="text/css">http://wordpress.org</style>',
+                );
+
+                $expected = array(
+                        '<script>http://wordpress.org</script>',
+                        '<style>http://wordpress.org</style>',
+                        '<script type="text/javascript">http://wordpress.org</script>',
+                        '<style type="text/css">http://wordpress.org</style>',
+                );
+
+                foreach ( $before as $key => $url ) {
+                        $this->assertEquals( $expected[ $key ], make_clickable( $url ) );
+                }
+        }
 }