+1 for changing getDocumentation to return OMNode (no deprecation, same will have to go for setDocumentation). Although this is an API method its mostly used from within for generating the WSDL.
Thanks, Keith. On 10/26/07, Glen Daniels <[EMAIL PROTECTED]> wrote: > > Well, +1 except for the fact that you can't overload getDocumentation() > and just change the return type - so there's no way to deprecate that > one without adding another method like getDocumentationOM(). We could > either do that and avoid breaking existing stuff, or just change > getDocumentation() and accept the incompatible change. > > --Glen > > Sanjiva Weerawarana wrote: > > +1. > > > > keith chapman wrote: > >> Hi Devs, > >> > >> Currently the method signature for det/get documentation in > >> AxisDescriptio is as follows. > >> > >> public String getDocumentation(); > >> public void setDocumentation(String documentation); > >> > >> As you can see it treats the documentation as a string. There are > >> occasions where the documentation can be XML though. If the > >> documentation is XML what we do currently is wrap it in CDATA tags. > >> This works, but its not the best sollution. > >> > >> I propose deprecating the above methods and introducing the following > >> methods which that documentation as an OMNode. > >> > >> public OMNode getDocumentation(); > >> public void setDocumentation(OMNode documentation); > >> > >> This will enable us to set the documentation as an OMText or OMElement. > >> > >> If needed we can retain the old methods without deprecating them (Just > >> leave them as a convenience method), where they will call into the new > >> method. > >> > >> The above proposal is targeted at improving the wsdl served by Axis2. > >> With what we have currently documentation which is XML is wrapped in > >> CDATA tags in the WSDL. > >> > >> Thanks, > >> Keith. > >> > >> -- > >> Keith Chapman > >> WSO2 Inc. > >> Oxygen for Web Services Developers. > >> http://wso2.org/ > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > -- Keith Chapman WSO2 Inc. Oxygen for Web Services Developers. http://wso2.org/
