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

Tim McConnell updated GERONIMO-2874:
------------------------------------

    Attachment: GERONIMO-2874.patch

Fixed all annotation code that checks for existing DD entries by adding 
.getStringValue() to the comparison (as Jarek suggested). Hope we can get this 
committed into both trunk and the new branch. 

> ResourceAnnotationHelper performs invalid check for existing deployment 
> descriptor entries
> ------------------------------------------------------------------------------------------
>
>                 Key: GERONIMO-2874
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-2874
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>            Reporter: Jarek Gawor
>         Assigned To: Tim McConnell
>         Attachments: GERONIMO-2874.patch
>
>
> ResourceAnnotationHelper checks for existing deployment descriptor entries by 
> doing:
>  
> String resourceName = ...;
>  if ( serviceRef.getServiceRefName().equals(resourceName) ) { .. }
> However, serviceRef.getServiceRefName() (and similar calls) returns an xbean 
> object and not a String, therefore the check will always return false.

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

Reply via email to