[ 
https://issues.apache.org/jira/browse/DBUTILS-91?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13408051#comment-13408051
 ] 

Simone Tripodi edited comment on DBUTILS-91 at 7/6/12 3:10 PM:
---------------------------------------------------------------

Patch applied, see r1358253, thanks for contributing.

Please, for future patches, remind the following:

 * respect the original file format - no tabs;
 * javadoc was not complete (the new constructor didn't have the argument 
documented);
 * when adding new methods/constructors, add the {{@since}} tag;
 * the constructor didn't shield the class from potential {{null}} maps, which 
could have caused NPEs at runtime.

thanks
                
      was (Author: simone.tripodi):
    Patch applied, see r1358253, thanks for contributing.

Please, for future patches, remind the following:

 * respect the original file format - no tabs;
 * javadoc was not complete (the new constructor didn't have the argument 
documented);
 * when adding new methods/constructors, add the {{@since}} tag

thanks
                  
> Enhance BasicRowProcessor to have row mapping easier to configure
> -----------------------------------------------------------------
>
>                 Key: DBUTILS-91
>                 URL: https://issues.apache.org/jira/browse/DBUTILS-91
>             Project: Commons DbUtils
>          Issue Type: Improvement
>    Affects Versions: 1.4
>            Reporter: Stevo Slavic
>            Assignee: Simone Tripodi
>             Fix For: 1.5
>
>         Attachments: DBUTILS-91.patch
>
>
> {{BasicRowProcessor}} by default makes use of {{BeanProcessor}} for mapping 
> result set row columns to bean property names. {{BeanProcessor}} uses bean 
> property names, and performs case-insensitive matching of those names to 
> column names.
> Currently {{BasicRowProcessor}} can be configured with custom 
> {{BeanProcessor}} extension to customize the row mapping, but it gets rather 
> ugly.
> It would be great if {{BeanProcessor}} was configurable with column name to 
> bean property name strategy. Besides current strategy, DbUtils should also 
> bundle strategy implementation configurable with a property name to column 
> name (or other way around) map.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to