You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by mi...@apache.org on 2005/10/23 19:02:23 UTC

svn commit: r327810 - /jakarta/tapestry/trunk/framework/src/java/org/apache/tapestry/util/text/MarkupCharacterTranslator.java

Author: mindbridge
Date: Sun Oct 23 10:02:18 2005
New Revision: 327810

URL: http://svn.apache.org/viewcvs?rev=327810&view=rev
Log:
[TAPESTRY-713] Fixing "Problem encoding extended ASCII Characters"
IE and Firefox do indeed behave stangely with Unicode character
in the range 128-159. Those characters are now quoted as a result.

Modified:
    jakarta/tapestry/trunk/framework/src/java/org/apache/tapestry/util/text/MarkupCharacterTranslator.java

Modified: jakarta/tapestry/trunk/framework/src/java/org/apache/tapestry/util/text/MarkupCharacterTranslator.java
URL: http://svn.apache.org/viewcvs/jakarta/tapestry/trunk/framework/src/java/org/apache/tapestry/util/text/MarkupCharacterTranslator.java?rev=327810&r1=327809&r2=327810&view=diff
==============================================================================
--- jakarta/tapestry/trunk/framework/src/java/org/apache/tapestry/util/text/MarkupCharacterTranslator.java (original)
+++ jakarta/tapestry/trunk/framework/src/java/org/apache/tapestry/util/text/MarkupCharacterTranslator.java Sun Oct 23 10:02:18 2005
@@ -72,7 +72,9 @@
 	 * @see org.apache.tapestry.util.text.IMarkupCharacterTranslator#translateAttribute(char)
 	 */
 	public String translate(char ch) {
-		if (ch >= 128 && !_encodeNonAscii)
+		// IE and Firefox do not handle characters between 128 and 159 well, 
+		// so they have to be quoted as well 
+		if (ch >= 160 && !_encodeNonAscii) 
 			return null;
 		
 		if (_safeMatcher.matches(ch))



---------------------------------------------------------------------
To unsubscribe, e-mail: tapestry-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: tapestry-dev-help@jakarta.apache.org