[ 
http://issues.apache.org/jira/browse/BEEHIVE-179?page=comments#action_57866 ]
     
Lawrence Jones commented on BEEHIVE-179:
----------------------------------------

That sounds good and it looks like there's already an externalConfig member on 
PropertySet - so maybe I can set TypesJarName to be a PropertySet but to have 
externalConfig = false and hasSetters = false. That would address my concerns.

I would still need Kyle's original suggestion of an additional 'optional' 
member on PropertySet such that getControlPropertySet() on that annotation 
would return null if the annotation was not present.


> context.getControlPropertySet() should return null for non-existent annotation
> ------------------------------------------------------------------------------
>
>          Key: BEEHIVE-179
>          URL: http://issues.apache.org/jira/browse/BEEHIVE-179
>      Project: Beehive
>         Type: Bug
>   Components: Controls
>     Versions: V1Beta
>  Environment: Windows
>     Reporter: Lawrence Jones
>     Assignee: Kyle Marvin
>     Priority: Critical

>
> I call
> ServiceControl.TypesJarName typesJarNameAnn =
>                 (ServiceControl.TypesJarName)context.
>                     getControlPropertySet(ServiceControl.TypesJarName.class);
> in the Impl of my control (ServiceControl). TypesJarName is a valid 
> annotation which is defined in the public interface  but not used in either 
> the public interface nor in the JCX.
> I expected to receive null but instead I get a non-null object with defaulted 
> values for its members.
> This is at odds with the javadoc for the API 
> ControlBeanContext.getControlPropertySet() and in any case you need a way to 
> be able to tell whether an annotation is present or not (especially if you 
> have marker annotations).

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira

Reply via email to