[#WODEN-135] Change lookup strategy for interface and binding fault reference elements to check extended interfaces

Woden

Change lookup strategy for interface and binding fault reference elements to check extended interfaces

Created: 29/Jan/07 06:51 PM Updated: 06/Aug/07 12:13 PM

Return to search

Component/s:

Parser

Affects Version/s:

Fix Version/s:

Time Tracking:

Not Specified

File Attachments:

File Name	Date Attached	Attached By	Size
BindingFaultImpl.java.patch	2007-07-25 11:40 AM	Dan Harvey	0.9 kB
InterfaceFaultReferenceImpl.java.patch	2007-07-25 11:40 AM	Dan Harvey	1 kB

Issue Links:

Dependants

This issue blocks:

~~WODEN-74~~ Validation does not report a QName resolution error for the {interface fault} property of InterfaceFaultReference

Resolution Date:

06/Aug/07 12:13 PM

Description

« Hide

The Element API is document scoped, however BindingOperationElement.getInterfaceOperationElement() checks for operations inherited via interface extension. For consistentcy the following methods should be implemented using the same strategy:
* InterfaceFaultReferenceElement.getInterfaceFaultElement()
* BindingFaultElement.getInterfaceFaultElement()

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

[ Permlink | « Hide ]

John Kaputin added a comment - 18/Jul/07 03:45 PM

code changes required are:

The implementation of BindingFaultElement.getInterfaceFaultElement() should use Interface.getFromAllInterfaceFaults(QName) instead of InterfaceElement.getInterfaceFaultElement(QName).

The implementation of InterfaceFaultReferenceElement.getInterfaceFaultElement() should use Interface.getFromAllInterfaceFaults(QName) instead of InterfaceElement.getInterfaceFaultElement(QName).

See BindingOperationElement.getInterfaceOperationElement() for an example.

[ Permlink | « Hide ]

John Kaputin added a comment - 02/Aug/07 12:37 PM

I have asked Dan to look into this.

[ Permlink | « Hide ]

John Kaputin added a comment - 06/Aug/07 12:13 PM

r563120
Fix as described above. Thanks to Dan Harvey for contributing it.