Spring/spring-framework
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

SPR-8652 Fixing queryForLong and queryForInt implementation to use correct expected type

Browse Source
This commit is contained in:
Thomas Risberg
2011-10-06 23:58:38 +00:00
parent 0920f2d7d5
commit 118ee3fce8
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
org.springframework.jdbc/src/main/java/org/springframework/jdbc/core/namedparam/NamedParameterJdbcTemplate.java
View File

@@ -206,7 +206,7 @@ public class NamedParameterJdbcTemplate implements NamedParameterJdbcOperations
} }
public long queryForLong(String sql, SqlParameterSource paramSource) throws DataAccessException { public long queryForLong(String sql, SqlParameterSource paramSource) throws DataAccessException {
Number number = queryForObject(sql, paramSource, Number.class); Number number = queryForObject(sql, paramSource, Long.class);
return (number != null ? number.longValue() : 0); return (number != null ? number.longValue() : 0);
} }
@@ -215,7 +215,7 @@ public class NamedParameterJdbcTemplate implements NamedParameterJdbcOperations
} }
public int queryForInt(String sql, SqlParameterSource paramSource) throws DataAccessException { public int queryForInt(String sql, SqlParameterSource paramSource) throws DataAccessException {
Number number = queryForObject(sql, paramSource, Number.class); Number number = queryForObject(sql, paramSource, Integer.class);
return (number != null ? number.intValue() : 0); return (number != null ? number.intValue() : 0);
} }

4
org.springframework.jdbc/src/test/java/org/springframework/jdbc/core/namedparam/NamedParameterQueryTests.java
View File

@@ -518,7 +518,7 @@ public class NamedParameterQueryTests extends AbstractJdbcTests {
ctrlResultSet.setReturnValue(mockResultSetMetaData); ctrlResultSet.setReturnValue(mockResultSetMetaData);
mockResultSet.next(); mockResultSet.next();
ctrlResultSet.setReturnValue(true); ctrlResultSet.setReturnValue(true);
mockResultSet.getDouble(1); mockResultSet.getInt(1);
ctrlResultSet.setReturnValue(22.0d); ctrlResultSet.setReturnValue(22.0d);
mockResultSet.wasNull(); mockResultSet.wasNull();
ctrlResultSet.setReturnValue(false); ctrlResultSet.setReturnValue(false);
@@ -561,7 +561,7 @@ public class NamedParameterQueryTests extends AbstractJdbcTests {
ctrlResultSet.setReturnValue(mockResultSetMetaData); ctrlResultSet.setReturnValue(mockResultSetMetaData);
mockResultSet.next(); mockResultSet.next();
ctrlResultSet.setReturnValue(true); ctrlResultSet.setReturnValue(true);
mockResultSet.getDouble(1); mockResultSet.getLong(1);
ctrlResultSet.setReturnValue(87.0d); ctrlResultSet.setReturnValue(87.0d);
mockResultSet.wasNull(); mockResultSet.wasNull();
ctrlResultSet.setReturnValue(false); ctrlResultSet.setReturnValue(false);