[ 
https://issues.apache.org/jira/browse/WSCOMMONS-134?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12483835
 ] 

Glen Daniels commented on WSCOMMONS-134:
----------------------------------------

Hi Alexander!  Your patch was outdated, so I did what I think is basically what 
you're looking for... let me know.

So I added the createOMElement(QName) API.  I didn't take out the "throws 
OMException" as you did in a couple of places because I think there actually 
might be problems that we'd want to deal with, but I'm open to changing that if 
we're sure.  I also opted not to add the createOMElement(localName, 
namespaceURI, prefix, parent) API because we've already got 
createOMElement(qname, parent) and I didn't want to clutter the interface any 
further.

I added a test for the QName API, but still need to take a closer look at the 
tests you introduced - I'll check those in today.

Please let us know if this work resolves your issue.  Thanks!

> The interface org.apache.axiom.om.OMFactory seems to be incomplete
> ------------------------------------------------------------------
>
>                 Key: WSCOMMONS-134
>                 URL: https://issues.apache.org/jira/browse/WSCOMMONS-134
>             Project: WS-Commons
>          Issue Type: Improvement
>            Reporter: Alexander Veit
>         Assigned To: Glen Daniels
>            Priority: Blocker
>         Attachments: axiom-patch.zip
>
>
> The interface misses some createOMElement(QName qname, ...) methods as 
> counterparts of the createOMElement(String localName, ...) methods.
> Currently, the only method that takes a QName as it's first argument is 
> createOMElement(QName qname, OMContainer parent).

-- 
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]

Reply via email to