From b859df894145b8a41609958917cf394fc46b8b10 Mon Sep 17 00:00:00 2001
From: Derek Jones <derek.jones@ellislab.com>
Date: Tue, 18 Nov 2008 15:24:20 +0000
Subject: Smart curly quotes!  Spaces trump all, then worry about \W, you can't
 convert them all at once or either your opening or your ending (whichever you
 process first) is going to be too greedy.

---
 system/libraries/Typography.php | 37 ++++++++++++++++++++++++-------------
 1 file changed, 24 insertions(+), 13 deletions(-)

(limited to 'system')

diff --git a/system/libraries/Typography.php b/system/libraries/Typography.php
index 808a76953..27fa42192 100644
--- a/system/libraries/Typography.php
+++ b/system/libraries/Typography.php
@@ -235,26 +235,37 @@ class CI_Typography {
 		
 		if ( ! isset($table))
 		{
-	        $table = array(					
+			$table = array(					
 							// nested smart quotes, opening and closing
 							// note that rules for grammar (English) allow only for two levels deep
 							// and that single quotes are _supposed_ to always be on the outside
 							// but we'll accommodate both
-							'/(^|\W|\s)\'"/'				=> '$1&#8216;&#8220;',
-							'/\'"(\s|\W|$)/'				=> '&#8217;&#8221;$1',
-							'/(^|\W|\s)"\'/'				=> '$1&#8220;&#8216;',
-							'/"\'(\s|\W|$)/'				=> '&#8221;&#8217;$1',
+							// Note that in all cases, whitespace is the primary determining factor
+							// on which direction to curl, with non-word characters like punctuation
+							// being a secondary factor only after whitespace is addressed.
+							'/\'"(\s|$)/'					=> '&#8217;&#8221;$1',
+							'/(^|\s)\'"/'					=> '$1&#8216;&#8220;',
+							'/\'"(\W)/'						=> '&#8217;&#8221;$1',
+							'/(\W)\'"/'						=> '$1&#8216;&#8220;',
+							'/"\'(\s|$)/'					=> '&#8221;&#8217;$1',
+							'/(^|\s)"\'/'					=> '$1&#8220;&#8216;',
+							'/"\'(\W)/'						=> '&#8221;&#8217;$1',
+							'/(\W)"\'/'						=> '$1&#8220;&#8216;',
 
 							// single quote smart quotes
-							'/(^|\W|\s)\'/'					=> '$1&#8216;',
-							'/\'(\s|\W|$)/'					=> '&#8217;$1',
+							'/\'(\s|$)/'					=> '&#8217;$1',
+							'/(^|\s)\'/'					=> '$1&#8216;',
+							'/\'(\W)/'						=> '&#8217;$1',
+							'/(\W)\'/'						=> '$1&#8216;',
 
 							// double quote smart quotes
-							'/(^|\W|\s)"/'					=> '$1&#8220;',
-							'/"(\s|\W|$)/'					=> '&#8221;$1',
-							
+							'/"(\s|$)/'						=> '&#8221;$1',
+							'/(^|\s)"/'						=> '$1&#8220;',
+							'/"(\W)/'						=> '&#8221;$1',
+							'/(\W)"/'						=> '$1&#8220;',
+
 							// apostrophes
-							"/(\w)'(\w)/"       	    	=> '$1&#8217;$2',
+							"/(\w)'(\w)/"					=> '$1&#8217;$2',
 
 							// Em dash and ellipses dots
 							'/\s?\-\-\s?/'					=> '&#8212;',
@@ -265,8 +276,8 @@ class CI_Typography {
 
 							// ampersands, if not a character entity
 							'/&(?!#?[a-zA-Z0-9]{2,};)/'		=> '&amp;'
-	        			);			
-		}	
+						);
+		}
 
 		return preg_replace(array_keys($table), $table, $str);
 	}
-- 
cgit v1.2.3-24-g4f1b