[prev in list] [next in list] [prev in thread] [next in thread] 

List:       jakarta-commons-dev
Subject:    [jira] [Commented] (DBUTILS-150) BeanProcessor populateBean does not work when PropertyDescriptor is
From:       "Miguel Costa (Jira)" <jira () apache ! org>
Date:       2023-02-27 10:05:00
Message-ID: JIRA.13524554.1676327461000.107031.1677492300007 () Atlassian ! JIRA
[Download RAW message or body]


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

Miguel Costa commented on DBUTILS-150:
--------------------------------------

Hi [~thecarlhall] your points are very fair.

To be honest I don't use any of the "Indexed" features the only thing that I needed \
was the PropertyDescriptor itself. Because I don't even use the setters and getters \
method, I implemented that myself with an override. And also yes you are correct, I'm \
just doing a directly "set" of the full list as a blob, and not iterating one entry \
at a time. So basically I get the Blob from the DB as a Json of a Map, convert into a \
HashMap and invoke the putAll over reflection 

> BeanProcessor populateBean does not work when PropertyDescriptor is of type 
> ----------------------------------------------------------------------------
> 
> Key: DBUTILS-150
> URL: https://issues.apache.org/jira/browse/DBUTILS-150
> Project: Commons DbUtils
> Issue Type: Improvement
> Affects Versions: 1.7
> Reporter: Miguel Costa
> Assignee: Carl Franklin Hall
> Priority: Major
> Fix For: 1.8
> 
> Attachments: 2023-02-14 10_14_48-Window.png, 2023-02-14 10_17_28-Window.png, \
> IndexedPropertyTestClass.java, TestCopy.java 
> 
> I've been using DB utils for a while to help me handle some custom beans \
> (Protobufs) Anyway while debugging an issue I'm facing I noticed that the function  \
>  BeanProcessor.populateBean is not able to handle  
> PropertyDescriptor when their class is IndexedPropertyDescriptor instead of a \
> normal PropertyDescriptor. IndexedPropertyDescriptor   extends PropertyDescriptor \
> so I would expect this to work without any issue, but currently it does not 
> I can provide a PR for review with my best approach for it



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic