[
https://issues.apache.org/jira/browse/WODEN-56?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12561366#action_12561366
]
John Kaputin commented on WODEN-56:
-----------------------------------
Jeremy,
what does this issue relate to specifically? Is it the use of the Attr impl
classes in org.apache.woden.internal.xml when registering extension attributes
in the ExtensionRegistry? These were created to as part of the woden
implementation, for use by Woden's implementation of its SOAP and HTTP
extension APIs. Is the issue that users will benefit from reusing these classes
too, in which case they should be API public? If so, it's worth noting that
they are not exhaustive - they just cover the attribute types defined in the
Part 2 adjuncts spec. User may require other types of attributes, but I guess
they can define those as part of their own extensions.
Or was it to do with the SOAP and HTTP ExtensionsImpl classes extending
ComponentExtensionsImpl? Again, ComponentExtensionsImpl was created to refactor
reusable code for use within the Woden implementation (e.g. in the
implementation of the SOAP and HTTP extensions). However, we could perhaps make
this an API public abstract super class, instead of an Interface, so that
user-defined extensions can inherit from it. Is this what you had in mind?
> Extensions' impl classes shouldn't have to use *internal* classes in their
> implementation
> -----------------------------------------------------------------------------------------
>
> Key: WODEN-56
> URL: https://issues.apache.org/jira/browse/WODEN-56
> Project: Woden
> Issue Type: Improvement
> Components: Parser
> Reporter: Jeremy Hughes
> Assignee: John Kaputin
> Fix For: M8
>
>
> An extension writer shouldn't be expected to use any classes internal to
> Woden (ie ones that have 'internal' in their package name). Everthing an
> extension implementation (as well as interface) needs to get to should be on
> an external package.
> We should eat our own food by changing the SOAP and HTTP extensions to do
> this.
--
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]