You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ibatis.apache.org by cb...@apache.org on 2010/05/01 22:53:52 UTC
svn commit: r940108 - in
/ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis:
builder/annotation/ builder/xml/ builder/xml/dynamic/ executor/loader/
executor/resultset/ jdbc/ mapping/ metadata/ reflection/
Author: cbegin
Date: Sat May 1 20:53:51 2010
New Revision: 940108
URL: http://svn.apache.org/viewvc?rev=940108&view=rev
Log:
ibatis-759 ensure uppercase uses english characters.
Modified:
ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/builder/annotation/MapperAnnotationBuilder.java
ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/builder/xml/XMLMapperEntityResolver.java
ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/builder/xml/XMLStatementBuilder.java
ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/builder/xml/dynamic/TrimSqlNode.java
ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/executor/loader/ResultLoaderMap.java
ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/executor/resultset/FastResultSetHandler.java
ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/jdbc/SqlRunner.java
ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/mapping/ResultMap.java
ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/metadata/Database.java
ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/metadata/Table.java
ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/reflection/Reflector.java
Modified: ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/builder/annotation/MapperAnnotationBuilder.java
URL: http://svn.apache.org/viewvc/ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/builder/annotation/MapperAnnotationBuilder.java?rev=940108&r1=940107&r2=940108&view=diff
==============================================================================
--- ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/builder/annotation/MapperAnnotationBuilder.java (original)
+++ ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/builder/annotation/MapperAnnotationBuilder.java Sat May 1 20:53:51 2010
@@ -285,7 +285,7 @@ public class MapperAnnotationBuilder {
} else if (type == DeleteProvider.class) {
type = Delete.class;
}
- return SqlCommandType.valueOf(type.getSimpleName().toUpperCase());
+ return SqlCommandType.valueOf(type.getSimpleName().toUpperCase(Locale.ENGLISH));
}
return SqlCommandType.UNKNOWN;
}
Modified: ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/builder/xml/XMLMapperEntityResolver.java
URL: http://svn.apache.org/viewvc/ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/builder/xml/XMLMapperEntityResolver.java?rev=940108&r1=940107&r2=940108&view=diff
==============================================================================
--- ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/builder/xml/XMLMapperEntityResolver.java (original)
+++ ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/builder/xml/XMLMapperEntityResolver.java Sat May 1 20:53:51 2010
@@ -8,6 +8,7 @@ import org.xml.sax.SAXException;
import java.io.IOException;
import java.io.InputStream;
import java.util.HashMap;
+import java.util.Locale;
import java.util.Map;
/**
@@ -21,11 +22,11 @@ public class XMLMapperEntityResolver imp
private static final Map<String, String> doctypeMap = new HashMap<String, String>();
static {
- doctypeMap.put("http://ibatis.apache.org/dtd/ibatis-3-config.dtd".toUpperCase(), MAPPER_CONFIG_DTD_RESOURCE);
- doctypeMap.put("-//ibatis.apache.org//DTD Config 3.0//EN".toUpperCase(), MAPPER_CONFIG_DTD_RESOURCE);
+ doctypeMap.put("http://ibatis.apache.org/dtd/ibatis-3-config.dtd".toUpperCase(Locale.ENGLISH), MAPPER_CONFIG_DTD_RESOURCE);
+ doctypeMap.put("-//ibatis.apache.org//DTD Config 3.0//EN".toUpperCase(Locale.ENGLISH), MAPPER_CONFIG_DTD_RESOURCE);
- doctypeMap.put("http://ibatis.apache.org/dtd/ibatis-3-mapper.dtd".toUpperCase(), MAPPER_DTD_RESOURCE);
- doctypeMap.put("-//ibatis.apache.org//DTD Mapper 3.0//EN".toUpperCase(), MAPPER_DTD_RESOURCE);
+ doctypeMap.put("http://ibatis.apache.org/dtd/ibatis-3-mapper.dtd".toUpperCase(Locale.ENGLISH), MAPPER_DTD_RESOURCE);
+ doctypeMap.put("-//ibatis.apache.org//DTD Mapper 3.0//EN".toUpperCase(Locale.ENGLISH), MAPPER_DTD_RESOURCE);
}
/**
@@ -39,8 +40,8 @@ public class XMLMapperEntityResolver imp
public InputSource resolveEntity(String publicId, String systemId)
throws SAXException {
- if (publicId != null) publicId = publicId.toUpperCase();
- if (systemId != null) systemId = systemId.toUpperCase();
+ if (publicId != null) publicId = publicId.toUpperCase(Locale.ENGLISH);
+ if (systemId != null) systemId = systemId.toUpperCase(Locale.ENGLISH);
InputSource source = null;
try {
Modified: ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/builder/xml/XMLStatementBuilder.java
URL: http://svn.apache.org/viewvc/ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/builder/xml/XMLStatementBuilder.java?rev=940108&r1=940107&r2=940108&view=diff
==============================================================================
--- ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/builder/xml/XMLStatementBuilder.java (original)
+++ ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/builder/xml/XMLStatementBuilder.java Sat May 1 20:53:51 2010
@@ -14,10 +14,7 @@ import org.apache.ibatis.session.Configu
import org.w3c.dom.Node;
import org.w3c.dom.NodeList;
-import java.util.ArrayList;
-import java.util.HashMap;
-import java.util.List;
-import java.util.Map;
+import java.util.*;
public class XMLStatementBuilder extends BaseBuilder {
@@ -49,7 +46,7 @@ public class XMLStatementBuilder extends
MixedSqlNode rootSqlNode = new MixedSqlNode(contents);
SqlSource sqlSource = new DynamicSqlSource(configuration, rootSqlNode);
String nodeName = context.getNode().getNodeName();
- SqlCommandType sqlCommandType = SqlCommandType.valueOf(nodeName.toUpperCase());
+ SqlCommandType sqlCommandType = SqlCommandType.valueOf(nodeName.toUpperCase(Locale.ENGLISH));
boolean isSelect = sqlCommandType == SqlCommandType.SELECT;
boolean flushCache = context.getBooleanAttribute("flushCache", !isSelect);
boolean useCache = context.getBooleanAttribute("useCache", isSelect);
Modified: ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/builder/xml/dynamic/TrimSqlNode.java
URL: http://svn.apache.org/viewvc/ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/builder/xml/dynamic/TrimSqlNode.java?rev=940108&r1=940107&r2=940108&view=diff
==============================================================================
--- ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/builder/xml/dynamic/TrimSqlNode.java (original)
+++ ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/builder/xml/dynamic/TrimSqlNode.java Sat May 1 20:53:51 2010
@@ -35,7 +35,7 @@ public class TrimSqlNode implements SqlN
return new ArrayList<String>() {
{
while (parser.hasMoreTokens()) {
- add(parser.nextToken().toUpperCase());
+ add(parser.nextToken().toUpperCase(Locale.ENGLISH));
}
}
};
@@ -59,7 +59,7 @@ public class TrimSqlNode implements SqlN
public void applyAll() {
sqlBuffer = new StringBuilder(sqlBuffer.toString().trim());
- String trimmedUppercaseSql = sqlBuffer.toString().toUpperCase();
+ String trimmedUppercaseSql = sqlBuffer.toString().toUpperCase(Locale.ENGLISH);
if (trimmedUppercaseSql.length() > 0) {
applyPrefix(sqlBuffer, trimmedUppercaseSql);
applySuffix(sqlBuffer, trimmedUppercaseSql);
Modified: ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/executor/loader/ResultLoaderMap.java
URL: http://svn.apache.org/viewvc/ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/executor/loader/ResultLoaderMap.java?rev=940108&r1=940107&r2=940108&view=diff
==============================================================================
--- ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/executor/loader/ResultLoaderMap.java (original)
+++ ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/executor/loader/ResultLoaderMap.java Sat May 1 20:53:51 2010
@@ -5,6 +5,7 @@ import org.apache.ibatis.reflection.Meta
import java.io.Serializable;
import java.sql.SQLException;
import java.util.HashMap;
+import java.util.Locale;
import java.util.Map;
import java.util.Set;
@@ -22,11 +23,11 @@ public class ResultLoaderMap implements
}
public boolean hasLoader(String methodName) throws SQLException {
- return loaderMap.containsKey(methodName.toUpperCase());
+ return loaderMap.containsKey(methodName.toUpperCase(Locale.ENGLISH));
}
public boolean load(String property) throws SQLException {
- LoadPair pair = loaderMap.remove(property.toUpperCase());
+ LoadPair pair = loaderMap.remove(property.toUpperCase(Locale.ENGLISH));
if (pair != null) {
pair.load();
return true;
@@ -44,7 +45,7 @@ public class ResultLoaderMap implements
private static String getUppercaseFirstProperty(String property) {
String[] parts = property.split("\\.");
- return parts[0].toUpperCase();
+ return parts[0].toUpperCase(Locale.ENGLISH);
}
private static class LoadPair {
Modified: ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/executor/resultset/FastResultSetHandler.java
URL: http://svn.apache.org/viewvc/ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/executor/resultset/FastResultSetHandler.java?rev=940108&r1=940107&r2=940108&view=diff
==============================================================================
--- ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/executor/resultset/FastResultSetHandler.java (original)
+++ ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/executor/resultset/FastResultSetHandler.java Sat May 1 20:53:51 2010
@@ -17,11 +17,7 @@ import org.apache.ibatis.type.TypeHandle
import org.apache.ibatis.type.TypeHandlerRegistry;
import java.sql.*;
-import java.util.ArrayList;
-import java.util.HashMap;
-import java.util.HashSet;
-import java.util.List;
-import java.util.Set;
+import java.util.*;
public class FastResultSetHandler implements ResultSetHandler {
@@ -221,7 +217,7 @@ public class FastResultSetHandler implem
final List<ResultMapping> propertyMappings = resultMap.getPropertyResultMappings();
for (ResultMapping propertyMapping : propertyMappings) {
final String column = propertyMapping.getColumn();
- if (propertyMapping.isCompositeResult() || (column != null && mappedColumnNames.contains(column.toUpperCase()))) {
+ if (propertyMapping.isCompositeResult() || (column != null && mappedColumnNames.contains(column.toUpperCase(Locale.ENGLISH)))) {
Object value = getPropertyMappingValue(rs, metaObject, propertyMapping, lazyLoader);
if (value != null) {
final String property = propertyMapping.getProperty();
@@ -271,7 +267,7 @@ public class FastResultSetHandler implem
final Set<String> mappedColumns = resultMap.getMappedColumns();
for (int i = 1; i <= columnCount; i++) {
final String columnName = configuration.isUseColumnLabel() ? rsmd.getColumnLabel(i) : rsmd.getColumnName(i);
- final String upperColumnName = columnName.toUpperCase();
+ final String upperColumnName = columnName.toUpperCase(Locale.ENGLISH);
if (mappedColumns.contains(upperColumnName)) {
mappedColumnNames.add(upperColumnName);
mappedColumnNames.add(columnName);
Modified: ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/jdbc/SqlRunner.java
URL: http://svn.apache.org/viewvc/ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/jdbc/SqlRunner.java?rev=940108&r1=940107&r2=940108&view=diff
==============================================================================
--- ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/jdbc/SqlRunner.java (original)
+++ ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/jdbc/SqlRunner.java Sat May 1 20:53:51 2010
@@ -212,7 +212,7 @@ public class SqlRunner {
for (int i = 0, n = columns.size(); i < n; i++) {
String name = columns.get(i);
TypeHandler handler = typeHandlers.get(i);
- row.put(name.toUpperCase(), handler.getResult(rs, name));
+ row.put(name.toUpperCase(Locale.ENGLISH), handler.getResult(rs, name));
}
list.add(row);
}
Modified: ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/mapping/ResultMap.java
URL: http://svn.apache.org/viewvc/ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/mapping/ResultMap.java?rev=940108&r1=940107&r2=940108&view=diff
==============================================================================
--- ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/mapping/ResultMap.java (original)
+++ ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/mapping/ResultMap.java Sat May 1 20:53:51 2010
@@ -46,12 +46,12 @@ public class ResultMap {
resultMap.hasNestedResultMaps = resultMap.hasNestedResultMaps || resultMapping.getNestedResultMapId() != null;
final String column = resultMapping.getColumn();
if (column != null) {
- resultMap.mappedColumns.add(column.toUpperCase());
+ resultMap.mappedColumns.add(column.toUpperCase(Locale.ENGLISH));
} else if (resultMapping.isCompositeResult()) {
for (ResultMapping compositeResultMapping : resultMapping.getComposites()) {
final String compositeColumn = compositeResultMapping.getColumn();
if (compositeColumn != null) {
- resultMap.mappedColumns.add(compositeColumn.toUpperCase());
+ resultMap.mappedColumns.add(compositeColumn.toUpperCase(Locale.ENGLISH));
}
}
}
Modified: ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/metadata/Database.java
URL: http://svn.apache.org/viewvc/ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/metadata/Database.java?rev=940108&r1=940107&r2=940108&view=diff
==============================================================================
--- ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/metadata/Database.java (original)
+++ ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/metadata/Database.java Sat May 1 20:53:51 2010
@@ -1,6 +1,7 @@
package org.apache.ibatis.metadata;
import java.util.HashMap;
+import java.util.Locale;
import java.util.Map;
public class Database {
@@ -24,11 +25,11 @@ public class Database {
}
public void addTable(Table table) {
- tables.put(table.getName().toUpperCase(), table);
+ tables.put(table.getName().toUpperCase(Locale.ENGLISH), table);
}
public Table getTable(String name) {
- return (Table) tables.get(name.toUpperCase());
+ return (Table) tables.get(name.toUpperCase(Locale.ENGLISH));
}
public String[] getTableNames() {
Modified: ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/metadata/Table.java
URL: http://svn.apache.org/viewvc/ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/metadata/Table.java?rev=940108&r1=940107&r2=940108&view=diff
==============================================================================
--- ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/metadata/Table.java (original)
+++ ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/metadata/Table.java Sat May 1 20:53:51 2010
@@ -1,6 +1,7 @@
package org.apache.ibatis.metadata;
import java.util.HashMap;
+import java.util.Locale;
import java.util.Map;
public class Table {
@@ -35,11 +36,11 @@ public class Table {
}
public void addColumn(Column col) {
- columns.put(col.getName().toUpperCase(), col);
+ columns.put(col.getName().toUpperCase(Locale.ENGLISH), col);
}
public Column getColumn(String name) {
- return (Column) columns.get(name.toUpperCase());
+ return (Column) columns.get(name.toUpperCase(Locale.ENGLISH));
}
public String[] getColumnNames() {
Modified: ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/reflection/Reflector.java
URL: http://svn.apache.org/viewvc/ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/reflection/Reflector.java?rev=940108&r1=940107&r2=940108&view=diff
==============================================================================
--- ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/reflection/Reflector.java (original)
+++ ibatis/java/ibatis-3/trunk/src/main/java/org/apache/ibatis/reflection/Reflector.java Sat May 1 20:53:51 2010
@@ -39,10 +39,10 @@ public class Reflector {
readablePropertyNames = getMethods.keySet().toArray(new String[getMethods.keySet().size()]);
writeablePropertyNames = setMethods.keySet().toArray(new String[setMethods.keySet().size()]);
for (String propName : readablePropertyNames) {
- caseInsensitivePropertyMap.put(propName.toUpperCase(), propName);
+ caseInsensitivePropertyMap.put(propName.toUpperCase(Locale.ENGLISH), propName);
}
for (String propName : writeablePropertyNames) {
- caseInsensitivePropertyMap.put(propName.toUpperCase(), propName);
+ caseInsensitivePropertyMap.put(propName.toUpperCase(Locale.ENGLISH), propName);
}
}
@@ -377,7 +377,7 @@ public class Reflector {
}
public String findPropertyName(String name) {
- return caseInsensitivePropertyMap.get(name.toUpperCase());
+ return caseInsensitivePropertyMap.get(name.toUpperCase(Locale.ENGLISH));
}
/**