[TUSCANY-3256] PolicyConfigurationException raised incorrectly when using operation level policy

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: Java-SCA-1.5.1
• Fix Version/s: Java-SCA-1.5.1, Java-SCA-1.x
• Component/s: Java SCA Policy
• Patch Info: Patch Available
• Assignee: Luciano Resende
• Reporter: Greg Dritschler
• Created: 03/Sep/09 01:00
• Updated: 19/Sep/10 11:14
• Resolved: 17/Sep/09 19:30

Description

A PolicyConfigurationException may be raised incorrectly when using operation policy as shown in the example below.

<composite ...>
<component ... requires='A'>
<implementation.java class="test.MyServiceImpl"/>
<service name="MyService">
<operation name="setXYZZY" requires="B"/>
</service>
</component>
</composite>

Let's say intent 'A' constrains an implementation rather than a binding. Constraints aren't examined until the final step of propagating the intents into the binding. So 'A' gets inherited by the service and then the service operation. Now the code in PolicyComputationUtil.addInheritedOpConfOnBindings gets control to inherit the service operation intents to the binding. If it can find a matching operation element under the binding, it goes through logic that applies the constraint (among among things). This would keep 'A' from being inherited by the binding level. However if it can't find an operation element under the binding, it simply adds a new operation element without doing any checking on it. This leads to a PolicyConfigurationException because there is no policy that applies to the binding and satisfies 'A'.

I am attaching a patch that changes the logic in PolicyComputationUtil.addInheritedOpConfOnBindings so that added binding operations go through the same logic as existing operations.