RE: [Axis2] Proposal to chage the method signature of set/get documentation
Sorry for the late follow up. Out last week. +1 for proposal below. +0 for getDocumentationAsString() -- Tom Jordahl From: keith chapman [mailto:[EMAIL PROTECTED] Sent: Wednesday, November 07, 2007 12:34 AM To: axis-dev@ws.apache.org Subject: Re: [Axis2] Proposal to chage the method signature of set/get documentation So shall we settle for public OMNode getDocumentation(); public void setDocumentation(OMNode documentation); public void setDocumentation(String documentation); and do we need public String getDocumentationAsString();? Thanks, Keith. On 11/1/07, keith chapman [EMAIL PROTECTED] wrote: We can have the setter method for both but not the getter. So we can have public void setDocumentation(String documentation); but we cant have public String getDocumentation(); cause we would already have public OMNode getDocumentation(); Thanks, Keith. On 11/1/07, Sanjiva Weerawarana [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] wrote: +1 that we'd be making the keep-the-simple-case-simple rule by losing the String method. Keith, what's the harm in having two methods? That is, do both. Sanjiva. Tom Jordahl wrote: Removing String as an argument type seems like a bad idea to me for two reasons: 1. Breaking API compatibility, if it is public, is not nice. 2. If I want to put a string there (which I think you are saying goes in the documentation elements of the WSDL) you are making me create another object. Sure it's easy, but it doesn't make me like the API much. I think putting a string in this element is the 90+% use case, right? Who is putting XML in here? Is there a public WSDL that you can point to that does this? -- Tom Jordahl *From:* keith chapman [mailto: [EMAIL PROTECTED] *Sent:* Friday, October 26, 2007 8:16 AM *To:* axis-dev@ws.apache.org *Subject:* Re: [Axis2] Proposal to chage the method signature of set/get documentation +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] mailto:[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] mailto: [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] mailto: [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] -- Keith Chapman WSO2 Inc. Oxygen for Web Services Developers. http://wso2.org/ -- Sanjiva Weerawarana, Ph.D. Founder Director; Lanka Software Foundation; http://www.opensource.lk/ Founder, Chairman CEO; WSO2, Inc.; http://www.wso2.com/ Member; Apache Software Foundation; http://www.apache.org/ Visiting Lecturer; University of Moratuwa; http://www.cse.mrt.ac.lk/ - 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/ -- Keith Chapman WSO2 Inc. Oxygen for Web Services Developers. http://wso2.org/
Re: [Axis2] Proposal to chage the method signature of set/get documentation
So shall we settle for public OMNode getDocumentation(); public void setDocumentation(OMNode documentation); public void setDocumentation(String documentation); and do we need public String getDocumentationAsString();? Thanks, Keith. On 11/1/07, keith chapman [EMAIL PROTECTED] wrote: We can have the setter method for both but not the getter. So we can have public void setDocumentation(String documentation); but we cant have public String getDocumentation(); cause we would already have public OMNode getDocumentation(); Thanks, Keith. On 11/1/07, Sanjiva Weerawarana [EMAIL PROTECTED] wrote: +1 that we'd be making the keep-the-simple-case-simple rule by losing the String method. Keith, what's the harm in having two methods? That is, do both. Sanjiva. Tom Jordahl wrote: Removing String as an argument type seems like a bad idea to me for two reasons: 1. Breaking API compatibility, if it is public, is not nice. 2. If I want to put a string there (which I think you are saying goes in the documentation elements of the WSDL) you are making me create another object. Sure it's easy, but it doesn't make me like the API much. I think putting a string in this element is the 90+% use case, right? Who is putting XML in here? Is there a public WSDL that you can point to that does this? -- Tom Jordahl *From:* keith chapman [mailto: [EMAIL PROTECTED] *Sent:* Friday, October 26, 2007 8:16 AM *To:* axis-dev@ws.apache.org *Subject:* Re: [Axis2] Proposal to chage the method signature of set/get documentation +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] mailto:[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] mailto:[EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] -- Keith Chapman WSO2 Inc. Oxygen for Web Services Developers. http://wso2.org/ -- Sanjiva Weerawarana, Ph.D. Founder Director; Lanka Software Foundation; http://www.opensource.lk/ Founder, Chairman CEO; WSO2, Inc.; http://www.wso2.com/ Member; Apache Software Foundation; http://www.apache.org/ Visiting Lecturer; University of Moratuwa; http://www.cse.mrt.ac.lk/ - 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/ -- Keith Chapman WSO2 Inc. Oxygen for Web Services Developers. http://wso2.org/
Re: [Axis2] Proposal to chage the method signature of set/get documentation
+1 that we'd be making the keep-the-simple-case-simple rule by losing the String method. Keith, what's the harm in having two methods? That is, do both. Sanjiva. Tom Jordahl wrote: Removing String as an argument type seems like a bad idea to me for two reasons: 1. Breaking API compatibility, if it is public, is not nice. 2. If I want to put a string there (which I think you are saying goes in the documentation elements of the WSDL) you are making me create another object. Sure it’s “easy”, but it doesn’t make me like the API much. I think putting a string in this element is the 90+% use case, right? Who is putting XML in here? Is there a public WSDL that you can point to that does this? -- Tom Jordahl *From:* keith chapman [mailto:[EMAIL PROTECTED] *Sent:* Friday, October 26, 2007 8:16 AM *To:* axis-dev@ws.apache.org *Subject:* Re: [Axis2] Proposal to chage the method signature of set/get documentation +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] mailto:[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] mailto:[EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] -- Keith Chapman WSO2 Inc. Oxygen for Web Services Developers. http://wso2.org/ -- Sanjiva Weerawarana, Ph.D. Founder Director; Lanka Software Foundation; http://www.opensource.lk/ Founder, Chairman CEO; WSO2, Inc.; http://www.wso2.com/ Member; Apache Software Foundation; http://www.apache.org/ Visiting Lecturer; University of Moratuwa; http://www.cse.mrt.ac.lk/ - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [Axis2] Proposal to chage the method signature of set/get documentation
We can have the setter method for both but not the getter. So we can have public void setDocumentation(String documentation); but we cant have public String getDocumentation(); cause we would already have public OMNode getDocumentation(); Thanks, Keith. On 11/1/07, Sanjiva Weerawarana [EMAIL PROTECTED] wrote: +1 that we'd be making the keep-the-simple-case-simple rule by losing the String method. Keith, what's the harm in having two methods? That is, do both. Sanjiva. Tom Jordahl wrote: Removing String as an argument type seems like a bad idea to me for two reasons: 1. Breaking API compatibility, if it is public, is not nice. 2. If I want to put a string there (which I think you are saying goes in the documentation elements of the WSDL) you are making me create another object. Sure it's easy, but it doesn't make me like the API much. I think putting a string in this element is the 90+% use case, right? Who is putting XML in here? Is there a public WSDL that you can point to that does this? -- Tom Jordahl *From:* keith chapman [mailto:[EMAIL PROTECTED] *Sent:* Friday, October 26, 2007 8:16 AM *To:* axis-dev@ws.apache.org *Subject:* Re: [Axis2] Proposal to chage the method signature of set/get documentation +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] mailto:[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] mailto:[EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] mailto:[EMAIL PROTECTED] -- Keith Chapman WSO2 Inc. Oxygen for Web Services Developers. http://wso2.org/ -- Sanjiva Weerawarana, Ph.D. Founder Director; Lanka Software Foundation; http://www.opensource.lk/ Founder, Chairman CEO; WSO2, Inc.; http://www.wso2.com/ Member; Apache Software Foundation; http://www.apache.org/ Visiting Lecturer; University of Moratuwa; http://www.cse.mrt.ac.lk/ - 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/
Re: [Axis2] Proposal to chage the method signature of set/get documentation
Hi Tom, Yes its not very common to have xml is tour documentation but there are occations where such information can be helpful. Here is a sample wsdl which has documentation as XML. http://mashups.wso2.org/services/nationalgeographic?wsdl2annotation=true If you look at the source of it you will see how ugly the documentation looks. Thanks, Keith. On 10/27/07, Tom Jordahl [EMAIL PROTECTED] wrote: Removing String as an argument type seems like a bad idea to me for two reasons: 1. Breaking API compatibility, if it is public, is not nice. 2. If I want to put a string there (which I think you are saying goes in the documentation elements of the WSDL) you are making me create another object. Sure it's easy, but it doesn't make me like the API much. I think putting a string in this element is the 90+% use case, right? Who is putting XML in here? Is there a public WSDL that you can point to that does this? -- Tom Jordahl -- *From:* keith chapman [mailto:[EMAIL PROTECTED] *Sent:* Friday, October 26, 2007 8:16 AM *To:* axis-dev@ws.apache.org *Subject:* Re: [Axis2] Proposal to chage the method signature of set/get documentation +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/ -- Keith Chapman WSO2 Inc. Oxygen for Web Services Developers. http://wso2.org/
Re: [Axis2] Proposal to chage the method signature of set/get documentation
BTW I forgot to mention that this site is powered by axis2. Thanks, Keith. On 10/31/07, keith chapman [EMAIL PROTECTED] wrote: Hi Tom, Yes its not very common to have xml is tour documentation but there are occations where such information can be helpful. Here is a sample wsdl which has documentation as XML. http://mashups.wso2.org/services/nationalgeographic?wsdl2annotation=trueIf you look at the source of it you will see how ugly the documentation looks. Thanks, Keith. On 10/27/07, Tom Jordahl [EMAIL PROTECTED] wrote: Removing String as an argument type seems like a bad idea to me for two reasons: 1. Breaking API compatibility, if it is public, is not nice. 2. If I want to put a string there (which I think you are saying goes in the documentation elements of the WSDL) you are making me create another object. Sure it's easy, but it doesn't make me like the API much. I think putting a string in this element is the 90+% use case, right? Who is putting XML in here? Is there a public WSDL that you can point to that does this? -- Tom Jordahl -- *From:* keith chapman [mailto:[EMAIL PROTECTED] *Sent:* Friday, October 26, 2007 8:16 AM *To:* axis-dev@ws.apache.org *Subject:* Re: [Axis2] Proposal to chage the method signature of set/get documentation +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/ -- Keith Chapman WSO2 Inc. Oxygen for Web Services Developers. http://wso2.org/ -- Keith Chapman WSO2 Inc. Oxygen for Web Services Developers. http://wso2.org/
Re: [Axis2] Proposal to chage the method signature of set/get documentation
+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/ -- Sanjiva Weerawarana, Ph.D. Founder Director; Lanka Software Foundation; http://www.opensource.lk/ Founder, Chairman CEO; WSO2, Inc.; http://www.wso2.com/ Member; Apache Software Foundation; http://www.apache.org/ Visiting Lecturer; University of Moratuwa; http://www.cse.mrt.ac.lk/ - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [Axis2] Proposal to chage the method signature of set/get documentation
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]
Re: [Axis2] Proposal to chage the method signature of set/get documentation
+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/
RE: [Axis2] Proposal to chage the method signature of set/get documentation
Removing String as an argument type seems like a bad idea to me for two reasons: 1. Breaking API compatibility, if it is public, is not nice. 2. If I want to put a string there (which I think you are saying goes in the documentation elements of the WSDL) you are making me create another object. Sure it's easy, but it doesn't make me like the API much. I think putting a string in this element is the 90+% use case, right? Who is putting XML in here? Is there a public WSDL that you can point to that does this? -- Tom Jordahl From: keith chapman [mailto:[EMAIL PROTECTED] Sent: Friday, October 26, 2007 8:16 AM To: axis-dev@ws.apache.org Subject: Re: [Axis2] Proposal to chage the method signature of set/get documentation +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/
Re: [Axis2] Proposal to chage the method signature of set/get documentation
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 +1 from me. If some one calls the earlier methods, you can serialize the OMNode and return, which i think is the easier to do. Chinthaka 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/ -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.6 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org iD8DBQFHIWucjON2uBzUhh8RAtmCAKCabVIEiPB/qtLSnzhw2PvW5Jj+GwCaA17r xWy2A3una8h4LSgppYDjGYM= =2Dyf -END PGP SIGNATURE- - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [Axis2] Proposal to chage the method signature of set/get documentation
On 10/26/07, Eran Chinthaka [EMAIL PROTECTED] wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 +1 from me. If some one calls the earlier methods, you can serialize the OMNode and return, which i think is the easier to do. Sure. Thanks, Keith. Chinthaka 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/ -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.6 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org iD8DBQFHIWucjON2uBzUhh8RAtmCAKCabVIEiPB/qtLSnzhw2PvW5Jj+GwCaA17r xWy2A3una8h4LSgppYDjGYM= =2Dyf -END PGP SIGNATURE- - 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/