[jira] Commented: (GERONIMO-4216) Examine setters methods in connector builder

["Rex Wang (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/GERONIMO-4216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12752897#action_12752897 ] 

Rex Wang commented on GERONIMO-4216:
------------------------------------

I would like to re-discuss on this improvement, the previous way to set up dynamic gbean in connector builder is, use java reflection to find out all the public getters and setters. And the ones both have public getter and setter method is a "valid" attribute and will build a GAttributeInfo for them.

But the GERONIMO-3874 hope examine the setters (because in a spring ra, the getter is protected. And what is more, there might be no getter in a RA class definiation), so the purpose of this jira.

The difficulty is how to determine the type of an attribute. And that is also the reason why this jira opens such a long time. The current code logic is using the retune type of getter as the type of attribute, That is reasonalbe absolutely. But if no getter exist, how to "guess" the type?

I am not sure if the following code logic is acceptable:
1. if both have setter and getter, use the return type of getter.
2. if no getter, check if there is an attribute declaration. if exist, use its type.
3. if no getter and attribute declaration, use the setter arguments type as the attibute type. (always primative and string)

Any thoughts?

-Rex

> Examine setters methods in connector builder
> --------------------------------------------
>
>                 Key: GERONIMO-4216
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-4216
>             Project: Geronimo
>          Issue Type: Improvement
>      Security Level: public(Regular issues) 
>          Components: connector
>    Affects Versions: 2.2
>            Reporter: Jarek Gawor
>            Assignee: Rex Wang
>         Attachments: GERONIMO-4216-b21-updated.patch
>
>
> Currently, the connector builder uses the getter methods to figure out the types for the attributes for the dynamic gbeans created for the various connector classes. The connector builder should use the setter methods instead (for example, in cases where there is a write-only property, i.e. setter method is present but getter is not).
> This is related to [GERONIMO-4131].

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