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

ant elder closed TUSCANY-3971.
------------------------------

    Resolution: Fixed

Fixed by updating as suggested
                
> JMSBindingProcessor.writeOperationProperties should trim() properties prior 
> to writing them & update logic to write JMS attributes even if properties are 
> null.
> ---------------------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: TUSCANY-3971
>                 URL: https://issues.apache.org/jira/browse/TUSCANY-3971
>             Project: Tuscany
>          Issue Type: Bug
>          Components: SCA Java Runtime
>    Affects Versions: Java-SCA-2.x
>            Reporter: Jennifer A Thompson
>             Fix For: Java-SCA-2.x
>
>
> In the JMSBindingProcessor.writeOperationProperties() method there are two 
> corrections which need to be made.
> 1) Update logic for writing properties. Currently the logic is
> if (operationProperties != null){
>       if ((jmsType != null && jmsType.length() > 0) || 
>                     (jmsCorrelationId != null && jmsCorrelationId.length() > 
> 0) || 
>                     jmsDeliveryMode != null || jmsTimeToLive != null || 
>                     jmsPriority != null) {
> }
> So this will only write jmsType, jmsDeliveryMode and jmsPriority only if 
> there actually are operation properties. However, this may not always be the 
> case. So the logic should be updated to 1 if statement connected with "or" 
> clauses. 
> 2) Also trim() should be added to 0 length checks, and add possibly added 
> before actually writing the value.  
> So the update code would be:
> if ( (operationProperties != null && !operationProperties.isEmpty()) || 
> (jmsType != null && jmsType.trim().length() > 0) || 
>                     jmsDeliveryMode != null || jmsTimeToLive != null || 
>                     jmsPriority != null) {
> .....
>       if (jmsType != null && jmsType.trim().length() > 0) {
>                         writer.writeAttribute("type", jmsType.trim());
>       }
> ........
> }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to