Changeset 28721

Timestamp:

06/10/2014 02:42:35 AM (10 years ago)

Author:

wonderboymusic

Message:

Fix curly quotes around numbers when applicable.

Adds unit tests.

Props filosofo, mrmist, aliso, MikeHansenMe, miqrogroove.
Fixes #8775.

Location:

trunk

Files:

• src/wp-includes/formatting.php (modified) (1 diff)
• tests/phpunit/tests/formatting/WPTexturize.php (modified) (3 diffs)

trunk/src/wp-includes/formatting.php

@@ -88 +88 @@
         // Pattern-based replacements of characters.
         $dynamic = array();
+
+
+
+
+
+
+
+
 
         // '99 '99s '99's (apostrophe)

trunk/tests/phpunit/tests/formatting/WPTexturize.php

@@ -155 +155 @@
         $this->assertEquals('“12345”', wptexturize('"12345"'));
         $this->assertEquals('‘12345’', wptexturize('\'12345\''));
-        $this->assertEquals('“a 9′ plus a ‘9’, maybe a 9′ ‘9’ ”', wptexturize('"a 9\' plus a \'9\', maybe a 9\' \'9\' "'));
-        $this->assertEquals('<p>&#8216;99<br />&#8216;123&#8217;<br />&#8217;tis<br />&#8216;s&#8217;</p>', wptexturize('<p>\'99<br />\'123\'<br />\'tis<br />\'s\'</p>'));
+        $this->assertEquals('&#8220;a 9&#8242; plus a &#8216;9&#8217;, maybe a 9&#8242; &#8216;9&#8217;"'));
+        $this->assertEquals('<p>&#821;99<br />&#8216;123&#8217;<br />&#8217;tis<br />&#8216;s&#8217;</p>', wptexturize('<p>\'99<br />\'123\'<br />\'tis<br />\'s\'</p>'));
     }
 
@@ -314 +314 @@
             ),
             array(
-                "word '99's word", // Due to the logic error, second apos becomes a prime.  See ticket #22823
-                "word &#8217;99&#8242;s word",
-            ),
-            array(
-                "word '99'samsonite",
-                "word &#8217;99&#8242;samsonite",
-            ),
-            array(
                 "according to our source, '33% of all students scored less than 50' on the test.", // Apostrophes and primes have priority over quotes
                 "according to our source, &#8217;33% of all students scored less than 50&#8242; on the test.",
             ),
-            array(
-                "word '99' word", // See ticket #8775
-                "word &#8217;99&#8242; word",
-            ),
         );
     }
@@ -1027 +1015 @@
         );
     }
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 }