commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Niall Pemberton (JIRA)" <j...@apache.org>
Subject [jira] Commented: (DBUTILS-72) BasicRowProcessor's CaseInsentitiveHashMap does not work with reflection
Date Wed, 22 Sep 2010 22:47:34 GMT

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

Niall Pemberton commented on DBUTILS-72:
----------------------------------------

IMO this is a bug in the taglib and how its using reflection. If a private class implements
a public interface then the method should be accessed through the public interface and this
issue won't occur. Commons BeanUtils does this, searching the interface hierarchy if the class
is not public.

I believe that taglib is in the attic - but you should be able to get the source code and
you could replace the whole of that method with one line:

{code}
    return PropertyUtils.getProperty(bean name);
{code}

Anyway, I don't think DBUtils should change to accomodate a bug in taglibs

> BasicRowProcessor's CaseInsentitiveHashMap does not work with reflection
> ------------------------------------------------------------------------
>
>                 Key: DBUTILS-72
>                 URL: https://issues.apache.org/jira/browse/DBUTILS-72
>             Project: Commons DbUtils
>          Issue Type: Bug
>    Affects Versions: 1.3
>            Reporter: Michael Osipov
>            Priority: Minor
>
> The map has been declared class private, this causes problems when someone tries to invoke
the get method via reflection. I do use the results in the Jakarta Input Taglib which invokes
the get via reflection and it fails with illegalaccessexception.
> The class has to be made public or decoupled from the row processor.

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


Mime
View raw message