Mailing-List: contact issues-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: issues@commons.apache.org
Message-ID: <69303267.1248591796664.JavaMail.jira@brutus>
Date: Sun, 26 Jul 2009 00:03:16 -0700 (PDT)
From: "Henri Yandell (JIRA)" <jira@apache.org>
To: issues@commons.apache.org
Subject: [jira] Updated: (BEANUTILS-344) Method createDynaProperty of
 JDBCDynaClass should first look for column label instead of column name in
 ResultSetMetadata object..
In-Reply-To: <688877576.1238915232910.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


     [ https://issues.apache.org/jira/browse/BEANUTILS-344?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Henri Yandell updated BEANUTILS-344:
------------------------------------

    Fix Version/s:     (was: 1.8.0-BETA)

> Method createDynaProperty of JDBCDynaClass should first look for column label instead of column name in ResultSetMetadata object..
> ----------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: BEANUTILS-344
>                 URL: https://issues.apache.org/jira/browse/BEANUTILS-344
>             Project: Commons BeanUtils
>          Issue Type: Bug
>          Components: DynaBean
>    Affects Versions: 1.8.0-BETA
>            Reporter: Viral
>         Attachments: columnName-vs-columnLabel-fix.patch
>
>   Original Estimate: 5h
>  Remaining Estimate: 5h
>
> Method  *createDynaProperty( ResultSetMetaData metadata, int i)*  of class *JDBCDynaClass* should read *column label* instead of *column name* from ResultSetMetadata object.
> For example..
> If query is something like *'select user_id as userId, user_name as userName....'* than *columName* variable should have the value *userId* instead of *user_id*
> Fix is ..
> *Original*
> {code:title=JDBCDynaClass.java|borderStyle=solid}
> protected DynaProperty createDynaProperty(
>                                     ResultSetMetaData metadata,
>                                     int i)
>                                     throws SQLException {
> //This is reading ColumnName but not ColumnLabel which returns value 'user_id' ..not 'userId' 
> String columnName = metadata.getColumnName(i); 
> ...
> ...
> ...
> }
> {code} 
> *It should be like following*
> {code:title=JDBCDynaClass.java|borderStyle=solid}
> protected DynaProperty createDynaProperty(
>                                     ResultSetMetaData metadata,
>                                     int i)
>                                     throws SQLException {
>  //First read ColumnLabel ..which sould be 'userId' in our case
>  String columnName = metadata.getColumnLabel(i);  
>  // If ColumnLabel is 'null' or 'empty' read ColumnName...
>  if(columnName == null || columnName.trim().equals(""))  {  
>      columnName = metadata.getColumnName(i);
>  }
> ...
> ...
> ...
> {code} 
> Let me know if this is not how it suppose to be or I am missing something here.
> Thanks,
> Viral Patel

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.