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

defaulting primitive property when receiving null value from result in BeanPropertyRowMapper (SPR-5588)

Browse Source
This commit is contained in:
Thomas Risberg
2009-03-25 12:12:42 +00:00
parent 665b284443
commit 360434d0d1
1 changed files with 18 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
org.springframework.jdbc/src/main/java/org/springframework/jdbc/core/BeanPropertyRowMapper.java
View File

@@ -32,6 +32,7 @@ import org.springframework.beans.BeanUtils;
import org.springframework.beans.BeanWrapper;
import org.springframework.beans.NotWritablePropertyException;
import org.springframework.beans.PropertyAccessorFactory;
import org.springframework.beans.TypeMismatchException;
import org.springframework.dao.DataRetrievalFailureException;
import org.springframework.dao.InvalidDataAccessApiUsageException;
import org.springframework.jdbc.support.JdbcUtils;
@@ -54,6 +55,11 @@ import org.springframework.util.Assert;
* <p>To facilitate mapping between columns and fields that don't have matching names,
* try using column aliases in the SQL statement like "select fname as first_name from customer".
*
* <p>For 'null' values read from the databasem, we will attempt to call the setter, but in the case of
* primitives, this causes a TypeMismatchException. We will trap this exception and log a warning message.
* Be aware that if you use the values from the generated bean to update the database the primitive value
* will have been set to the primitive's default value instead of null.
*
* <p>Please note that this class is designed to provide convenience rather than high performance.
* For best performance consider using a custom RowMapper.
*
@@ -219,7 +225,18 @@ public class BeanPropertyRowMapper<T> implements RowMapper<T> {
logger.debug("Mapping column '" + column + "' to property '" +
pd.getName() + "' of type " + pd.getPropertyType());
}
bw.setPropertyValue(pd.getName(), value);
try {
bw.setPropertyValue(pd.getName(), value);
}
catch (TypeMismatchException e) {
logger.warn("Intercepted TypeMismatchException for row " + rowNumber +
" and column '" + column + "' with value " + value +
" when setting property '" + pd.getName() + "' of type " + pd.getPropertyType() +
" on object: " + mappedObject);
if (value != null) {
throw e;
}
}
if (populatedProperties != null) {
populatedProperties.add(pd.getName());
}