You are viewing a plain text version of this content. The canonical link for it is here.
Posted to woden-dev@ws.apache.org by "Lawrence Mandel (JIRA)" <ji...@apache.org> on 2007/05/02 21:54:15 UTC

[jira] Created: (WODEN-159) InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive

InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive
-----------------------------------------------------------------------------------------

                 Key: WODEN-159
                 URL: https://issues.apache.org/jira/browse/WODEN-159
             Project: Woden
          Issue Type: Bug
          Components: Parser
    Affects Versions: M7a
            Reporter: Lawrence Mandel


Jacek Kopecky <ja...@deri.org> reported the following Woden bug on the W3C mailing list. See http://lists.w3.org/Archives/Public/www-ws-desc/2007May/0001.html. 

InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are
not quite recursive, they only go one level of interface extension deep.
This manifests in the flickr test-suite WSDL file when trying to access
the InterfaceOperation of the BindingOperation for
"flickr.groups.pools.add" - the method getInterfaceOperation() returns
null because the actual operation is too deep in the hierarchy of
extended interfaces.

Fix: in both InterfaceImpl.getDerivedInterface*, change getInterface* to
getAllInterface* - this can get into an infinite loop if a circular
interface extension is not detected as an error first.

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


---------------------------------------------------------------------
To unsubscribe, e-mail: woden-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: woden-dev-help@ws.apache.org


[jira] Updated: (WODEN-159) InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive

Posted by "John Kaputin (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WODEN-159?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

John Kaputin updated WODEN-159:
-------------------------------

    Comment: was deleted

> InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive
> -----------------------------------------------------------------------------------------
>
>                 Key: WODEN-159
>                 URL: https://issues.apache.org/jira/browse/WODEN-159
>             Project: Woden
>          Issue Type: Bug
>          Components: Parser
>    Affects Versions: M7a
>            Reporter: Lawrence Mandel
>            Assignee: John Kaputin
>             Fix For: M8
>
>
> Jacek Kopecky <ja...@deri.org> reported the following Woden bug on the W3C mailing list. See http://lists.w3.org/Archives/Public/www-ws-desc/2007May/0001.html. 
> InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are
> not quite recursive, they only go one level of interface extension deep.
> This manifests in the flickr test-suite WSDL file when trying to access
> the InterfaceOperation of the BindingOperation for
> "flickr.groups.pools.add" - the method getInterfaceOperation() returns
> null because the actual operation is too deep in the hierarchy of
> extended interfaces.
> Fix: in both InterfaceImpl.getDerivedInterface*, change getInterface* to
> getAllInterface* - this can get into an infinite loop if a circular
> interface extension is not detected as an error first.

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


---------------------------------------------------------------------
To unsubscribe, e-mail: woden-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: woden-dev-help@ws.apache.org


[jira] Commented: (WODEN-159) InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive

Posted by "John Kaputin (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/WODEN-159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12517231 ] 

John Kaputin commented on WODEN-159:
------------------------------------

The recursion through the interface extension tree has been fixed via JIRA WODEN-172. A further fix will be applied to InterfaceImpl to detect the circular interface reference and avoid an infinite loop when processing interface operations and faults.  I have also created a new testcase, Interface-6B, and will contribute this to the W3C WSDL 2.0 test suite.

> InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive
> -----------------------------------------------------------------------------------------
>
>                 Key: WODEN-159
>                 URL: https://issues.apache.org/jira/browse/WODEN-159
>             Project: Woden
>          Issue Type: Bug
>          Components: Parser
>    Affects Versions: M7a
>            Reporter: Lawrence Mandel
>            Assignee: John Kaputin
>             Fix For: M8
>
>
> Jacek Kopecky <ja...@deri.org> reported the following Woden bug on the W3C mailing list. See http://lists.w3.org/Archives/Public/www-ws-desc/2007May/0001.html. 
> InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are
> not quite recursive, they only go one level of interface extension deep.
> This manifests in the flickr test-suite WSDL file when trying to access
> the InterfaceOperation of the BindingOperation for
> "flickr.groups.pools.add" - the method getInterfaceOperation() returns
> null because the actual operation is too deep in the hierarchy of
> extended interfaces.
> Fix: in both InterfaceImpl.getDerivedInterface*, change getInterface* to
> getAllInterface* - this can get into an infinite loop if a circular
> interface extension is not detected as an error first.

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


---------------------------------------------------------------------
To unsubscribe, e-mail: woden-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: woden-dev-help@ws.apache.org


[jira] Commented: (WODEN-159) InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive

Posted by "Lawrence Mandel (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/WODEN-159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12493271 ] 

Lawrence Mandel commented on WODEN-159:
---------------------------------------

Further info from Jacek. See http://lists.w3.org/Archives/Public/www-ws-desc/2007May/0007.html.

The fix for 3 (159) is easy - in both InterfaceImpl.getDerivedInterface*
methods, change getInterface* to getAllInterface* - but this can get
into an infinite loop if a circular interface extension is not detected
as an error first, which I haven't checked. The WSDL 2.0 test case
FlickrHTTP-1G has enough interface extension to trigger this.

> InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive
> -----------------------------------------------------------------------------------------
>
>                 Key: WODEN-159
>                 URL: https://issues.apache.org/jira/browse/WODEN-159
>             Project: Woden
>          Issue Type: Bug
>          Components: Parser
>    Affects Versions: M7a
>            Reporter: Lawrence Mandel
>
> Jacek Kopecky <ja...@deri.org> reported the following Woden bug on the W3C mailing list. See http://lists.w3.org/Archives/Public/www-ws-desc/2007May/0001.html. 
> InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are
> not quite recursive, they only go one level of interface extension deep.
> This manifests in the flickr test-suite WSDL file when trying to access
> the InterfaceOperation of the BindingOperation for
> "flickr.groups.pools.add" - the method getInterfaceOperation() returns
> null because the actual operation is too deep in the hierarchy of
> extended interfaces.
> Fix: in both InterfaceImpl.getDerivedInterface*, change getInterface* to
> getAllInterface* - this can get into an infinite loop if a circular
> interface extension is not detected as an error first.

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


---------------------------------------------------------------------
To unsubscribe, e-mail: woden-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: woden-dev-help@ws.apache.org


[jira] Updated: (WODEN-159) InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive

Posted by "John Kaputin (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WODEN-159?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

John Kaputin updated WODEN-159:
-------------------------------

    Attachment: Interface-6B.patch.txt

A new testcase for interface extension (assertion Interface-1009) which includes operations, faults and bindings, so any attempt to process these will result in an infinite loop if the circular reference is not correctly handled. Testcase to be contributed to W3C WSDL 2.0 test suite.

> InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive
> -----------------------------------------------------------------------------------------
>
>                 Key: WODEN-159
>                 URL: https://issues.apache.org/jira/browse/WODEN-159
>             Project: Woden
>          Issue Type: Bug
>          Components: Parser
>    Affects Versions: M7a
>            Reporter: Lawrence Mandel
>            Assignee: John Kaputin
>             Fix For: M8
>
>
> Jacek Kopecky <ja...@deri.org> reported the following Woden bug on the W3C mailing list. See http://lists.w3.org/Archives/Public/www-ws-desc/2007May/0001.html. 
> InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are
> not quite recursive, they only go one level of interface extension deep.
> This manifests in the flickr test-suite WSDL file when trying to access
> the InterfaceOperation of the BindingOperation for
> "flickr.groups.pools.add" - the method getInterfaceOperation() returns
> null because the actual operation is too deep in the hierarchy of
> extended interfaces.
> Fix: in both InterfaceImpl.getDerivedInterface*, change getInterface* to
> getAllInterface* - this can get into an infinite loop if a circular
> interface extension is not detected as an error first.

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


---------------------------------------------------------------------
To unsubscribe, e-mail: woden-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: woden-dev-help@ws.apache.org


[jira] Updated: (WODEN-159) InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive

Posted by "John Kaputin (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WODEN-159?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

John Kaputin updated WODEN-159:
-------------------------------

    Attachment:     (was: Interface-6B.patch.txt)

> InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive
> -----------------------------------------------------------------------------------------
>
>                 Key: WODEN-159
>                 URL: https://issues.apache.org/jira/browse/WODEN-159
>             Project: Woden
>          Issue Type: Bug
>          Components: Parser
>    Affects Versions: M7a
>            Reporter: Lawrence Mandel
>            Assignee: John Kaputin
>             Fix For: M8
>
>
> Jacek Kopecky <ja...@deri.org> reported the following Woden bug on the W3C mailing list. See http://lists.w3.org/Archives/Public/www-ws-desc/2007May/0001.html. 
> InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are
> not quite recursive, they only go one level of interface extension deep.
> This manifests in the flickr test-suite WSDL file when trying to access
> the InterfaceOperation of the BindingOperation for
> "flickr.groups.pools.add" - the method getInterfaceOperation() returns
> null because the actual operation is too deep in the hierarchy of
> extended interfaces.
> Fix: in both InterfaceImpl.getDerivedInterface*, change getInterface* to
> getAllInterface* - this can get into an infinite loop if a circular
> interface extension is not detected as an error first.

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


---------------------------------------------------------------------
To unsubscribe, e-mail: woden-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: woden-dev-help@ws.apache.org


[jira] Resolved: (WODEN-159) InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive

Posted by "John Kaputin (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WODEN-159?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

John Kaputin resolved WODEN-159.
--------------------------------

    Resolution: Fixed

r562092
Fix to detect circular reference and avoid infinite loop.

> InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive
> -----------------------------------------------------------------------------------------
>
>                 Key: WODEN-159
>                 URL: https://issues.apache.org/jira/browse/WODEN-159
>             Project: Woden
>          Issue Type: Bug
>          Components: Parser
>    Affects Versions: M7a
>            Reporter: Lawrence Mandel
>            Assignee: John Kaputin
>             Fix For: M8
>
>
> Jacek Kopecky <ja...@deri.org> reported the following Woden bug on the W3C mailing list. See http://lists.w3.org/Archives/Public/www-ws-desc/2007May/0001.html. 
> InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are
> not quite recursive, they only go one level of interface extension deep.
> This manifests in the flickr test-suite WSDL file when trying to access
> the InterfaceOperation of the BindingOperation for
> "flickr.groups.pools.add" - the method getInterfaceOperation() returns
> null because the actual operation is too deep in the hierarchy of
> extended interfaces.
> Fix: in both InterfaceImpl.getDerivedInterface*, change getInterface* to
> getAllInterface* - this can get into an infinite loop if a circular
> interface extension is not detected as an error first.

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


---------------------------------------------------------------------
To unsubscribe, e-mail: woden-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: woden-dev-help@ws.apache.org


[jira] Updated: (WODEN-159) InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive

Posted by "John Kaputin (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WODEN-159?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

John Kaputin updated WODEN-159:
-------------------------------

    Fix Version/s: M8

> InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive
> -----------------------------------------------------------------------------------------
>
>                 Key: WODEN-159
>                 URL: https://issues.apache.org/jira/browse/WODEN-159
>             Project: Woden
>          Issue Type: Bug
>          Components: Parser
>    Affects Versions: M7a
>            Reporter: Lawrence Mandel
>            Assignee: John Kaputin
>             Fix For: M8
>
>
> Jacek Kopecky <ja...@deri.org> reported the following Woden bug on the W3C mailing list. See http://lists.w3.org/Archives/Public/www-ws-desc/2007May/0001.html. 
> InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are
> not quite recursive, they only go one level of interface extension deep.
> This manifests in the flickr test-suite WSDL file when trying to access
> the InterfaceOperation of the BindingOperation for
> "flickr.groups.pools.add" - the method getInterfaceOperation() returns
> null because the actual operation is too deep in the hierarchy of
> extended interfaces.
> Fix: in both InterfaceImpl.getDerivedInterface*, change getInterface* to
> getAllInterface* - this can get into an infinite loop if a circular
> interface extension is not detected as an error first.

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


---------------------------------------------------------------------
To unsubscribe, e-mail: woden-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: woden-dev-help@ws.apache.org


[jira] Assigned: (WODEN-159) InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive

Posted by "John Kaputin (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/WODEN-159?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

John Kaputin reassigned WODEN-159:
----------------------------------

    Assignee: John Kaputin

> InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive
> -----------------------------------------------------------------------------------------
>
>                 Key: WODEN-159
>                 URL: https://issues.apache.org/jira/browse/WODEN-159
>             Project: Woden
>          Issue Type: Bug
>          Components: Parser
>    Affects Versions: M7a
>            Reporter: Lawrence Mandel
>            Assignee: John Kaputin
>
> Jacek Kopecky <ja...@deri.org> reported the following Woden bug on the W3C mailing list. See http://lists.w3.org/Archives/Public/www-ws-desc/2007May/0001.html. 
> InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are
> not quite recursive, they only go one level of interface extension deep.
> This manifests in the flickr test-suite WSDL file when trying to access
> the InterfaceOperation of the BindingOperation for
> "flickr.groups.pools.add" - the method getInterfaceOperation() returns
> null because the actual operation is too deep in the hierarchy of
> extended interfaces.
> Fix: in both InterfaceImpl.getDerivedInterface*, change getInterface* to
> getAllInterface* - this can get into an infinite loop if a circular
> interface extension is not detected as an error first.

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


---------------------------------------------------------------------
To unsubscribe, e-mail: woden-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: woden-dev-help@ws.apache.org


[jira] Commented: (WODEN-159) InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive

Posted by "Arthur Ryman (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/WODEN-159?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12568717#action_12568717 ] 

Arthur Ryman commented on WODEN-159:
------------------------------------

John, I committed the test case to the W3C test suite.
See http://www.w3.org/Bugs/Public/show_bug.cgi?id=5477

> InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive
> -----------------------------------------------------------------------------------------
>
>                 Key: WODEN-159
>                 URL: https://issues.apache.org/jira/browse/WODEN-159
>             Project: Woden
>          Issue Type: Bug
>          Components: Parser
>    Affects Versions: M7a
>            Reporter: Lawrence Mandel
>            Assignee: John Kaputin
>             Fix For: M8
>
>
> Jacek Kopecky <ja...@deri.org> reported the following Woden bug on the W3C mailing list. See http://lists.w3.org/Archives/Public/www-ws-desc/2007May/0001.html. 
> InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are
> not quite recursive, they only go one level of interface extension deep.
> This manifests in the flickr test-suite WSDL file when trying to access
> the InterfaceOperation of the BindingOperation for
> "flickr.groups.pools.add" - the method getInterfaceOperation() returns
> null because the actual operation is too deep in the hierarchy of
> extended interfaces.
> Fix: in both InterfaceImpl.getDerivedInterface*, change getInterface* to
> getAllInterface* - this can get into an infinite loop if a circular
> interface extension is not detected as an error first.

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


---------------------------------------------------------------------
To unsubscribe, e-mail: woden-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: woden-dev-help@ws.apache.org