Changeset 33064 for trunk/tests/phpunit/tests/formatting/EscUrl.php

Timestamp:

07/03/2015 02:27:11 PM (10 years ago)

Author:

jorbin

Message:

Don't strip newline in esc_url() when protocol is mailto:

The mailto protocol is a bit different than the other protocols in that new lines are something you might realistically want to include. Includes tests to make sure that http protocol urls that contain mailto: aren't affected. Tests for stripping newlines in general already exist.

Fixes #31632
Props danielbachhuber

File:

• trunk/tests/phpunit/tests/formatting/EscUrl.php (modified) (1 diff)

trunk/tests/phpunit/tests/formatting/EscUrl.php

@@ -69 +69 @@
         $this->assertEquals( '//example.com/foo?foo=abc:def', esc_url( '//example.com/foo?foo=abc:def' ) );
     }
+
+    /**
+     * @ticket 31632
+     */
+    function test_mailto_with_newline() {
+        $body = <<<EOT
+Hi there,
+
+I thought you might want to sign up for this newsletter
+EOT;
+        $email_link = 'mailto:?body=' . rawurlencode( $body );
+        $email_link = esc_url( $email_link );
+        $this->assertEquals( 'mailto:?body=Hi%20there%2C%0A%0AI%20thought%20you%20might%20want%20to%20sign%20up%20for%20this%20newsletter', $email_link );
+    }
+    /**
+     * @ticket 31632
+     */
+    function test_mailto_in_http_url_with_newline() {
+        $body = <<<EOT
+Hi there,
+
+I thought you might want to sign up for this newsletter
+EOT;
+        $email_link = 'http://example.com/mailto:?body=' . rawurlencode( $body );
+        $email_link = esc_url( $email_link );
+        $this->assertEquals( 'http://example.com/mailto:?body=Hi%20there%2CI%20thought%20you%20might%20want%20to%20sign%20up%20for%20this%20newsletter', $email_link );
+    }
+
 }