You are viewing a plain text version of this content. The canonical link for it is here.
Posted to axis-cvs@ws.apache.org by de...@apache.org on 2007/06/27 13:13:27 UTC
svn commit: r551123 -
/webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java
Author: deepal
Date: Wed Jun 27 04:13:26 2007
New Revision: 551123
URL: http://svn.apache.org/viewvc?view=rev&rev=551123
Log:
dont we need to add the operation local name to the operationaliasmap
Modified:
webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java
Modified: webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java?view=diff&rev=551123&r1=551122&r2=551123
==============================================================================
--- webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java (original)
+++ webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java Wed Jun 27 04:13:26 2007
@@ -741,6 +741,8 @@
}
} else {
operationsAliasesMap.put(action, axisOperation);
+ //Adding operation name to the mapping table
+ operationsAliasesMap.put(axisOperation.getName().getLocalPart(), axisOperation);
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-cvs-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-cvs-help@ws.apache.org
Re: svn commit: r551123 - /webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java
Posted by Deepal Jayasinghe <de...@opensource.lk>.
David Illsley wrote:
> Um, no, I don't believe we need this, for 2 reasons:
> 1. This is a totally non-standard, non interoperable thing you're
> trying to support.
> 2. It's already supported by the code:
> AxisOperation axisOperation = (AxisOperation)
> getChild(operationName);
Hmm , seems like dispatchers can not find that.
The best thing is I will attach a test case to demonstrate the issue.
Thanks
Deepal
>
> David
>
> On 27/06/07, deepal@apache.org <de...@apache.org> wrote:
>> Author: deepal
>> Date: Wed Jun 27 04:13:26 2007
>> New Revision: 551123
>>
>> URL: http://svn.apache.org/viewvc?view=rev&rev=551123
>> Log:
>> dont we need to add the operation local name to the operationaliasmap
>>
>> Modified:
>>
>> webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java
>>
>>
>> Modified:
>> webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java
>>
>> URL:
>> http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java?view=diff&rev=551123&r1=551122&r2=551123
>>
>> ==============================================================================
>>
>> ---
>> webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java
>> (original)
>> +++
>> webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java
>> Wed Jun 27 04:13:26 2007
>> @@ -741,6 +741,8 @@
>> }
>> } else {
>> operationsAliasesMap.put(action, axisOperation);
>> + //Adding operation name to the mapping table
>> +
>> operationsAliasesMap.put(axisOperation.getName().getLocalPart(),
>> axisOperation);
>> }
>> }
>>
>>
>>
>>
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-cvs-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-cvs-help@ws.apache.org
Re: svn commit: r551123 - /webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java
Posted by Deepal Jayasinghe <de...@opensource.lk>.
David Illsley wrote:
> Um, no, I don't believe we need this, for 2 reasons:
> 1. This is a totally non-standard, non interoperable thing you're
> trying to support.
> 2. It's already supported by the code:
> AxisOperation axisOperation = (AxisOperation)
> getChild(operationName);
Hmm , seems like dispatchers can not find that.
The best thing is I will attach a test case to demonstrate the issue.
Thanks
Deepal
>
> David
>
> On 27/06/07, deepal@apache.org <de...@apache.org> wrote:
>> Author: deepal
>> Date: Wed Jun 27 04:13:26 2007
>> New Revision: 551123
>>
>> URL: http://svn.apache.org/viewvc?view=rev&rev=551123
>> Log:
>> dont we need to add the operation local name to the operationaliasmap
>>
>> Modified:
>>
>> webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java
>>
>>
>> Modified:
>> webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java
>>
>> URL:
>> http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java?view=diff&rev=551123&r1=551122&r2=551123
>>
>> ==============================================================================
>>
>> ---
>> webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java
>> (original)
>> +++
>> webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java
>> Wed Jun 27 04:13:26 2007
>> @@ -741,6 +741,8 @@
>> }
>> } else {
>> operationsAliasesMap.put(action, axisOperation);
>> + //Adding operation name to the mapping table
>> +
>> operationsAliasesMap.put(axisOperation.getName().getLocalPart(),
>> axisOperation);
>> }
>> }
>>
>>
>>
>>
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-cvs-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-cvs-help@ws.apache.org
Re: svn commit: r551123 - /webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java
Posted by David Illsley <da...@gmail.com>.
Um, no, I don't believe we need this, for 2 reasons:
1. This is a totally non-standard, non interoperable thing you're
trying to support.
2. It's already supported by the code:
AxisOperation axisOperation = (AxisOperation) getChild(operationName);
David
On 27/06/07, deepal@apache.org <de...@apache.org> wrote:
> Author: deepal
> Date: Wed Jun 27 04:13:26 2007
> New Revision: 551123
>
> URL: http://svn.apache.org/viewvc?view=rev&rev=551123
> Log:
> dont we need to add the operation local name to the operationaliasmap
>
> Modified:
> webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java
>
> Modified: webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java
> URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java?view=diff&rev=551123&r1=551122&r2=551123
> ==============================================================================
> --- webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java (original)
> +++ webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java Wed Jun 27 04:13:26 2007
> @@ -741,6 +741,8 @@
> }
> } else {
> operationsAliasesMap.put(action, axisOperation);
> + //Adding operation name to the mapping table
> + operationsAliasesMap.put(axisOperation.getName().getLocalPart(), axisOperation);
> }
> }
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: axis-cvs-unsubscribe@ws.apache.org
> For additional commands, e-mail: axis-cvs-help@ws.apache.org
>
>
--
David Illsley - IBM Web Services Development
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-cvs-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-cvs-help@ws.apache.org
Re: svn commit: r551123 - /webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java
Posted by David Illsley <da...@gmail.com>.
Um, no, I don't believe we need this, for 2 reasons:
1. This is a totally non-standard, non interoperable thing you're
trying to support.
2. It's already supported by the code:
AxisOperation axisOperation = (AxisOperation) getChild(operationName);
David
On 27/06/07, deepal@apache.org <de...@apache.org> wrote:
> Author: deepal
> Date: Wed Jun 27 04:13:26 2007
> New Revision: 551123
>
> URL: http://svn.apache.org/viewvc?view=rev&rev=551123
> Log:
> dont we need to add the operation local name to the operationaliasmap
>
> Modified:
> webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java
>
> Modified: webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java
> URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java?view=diff&rev=551123&r1=551122&r2=551123
> ==============================================================================
> --- webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java (original)
> +++ webservices/axis2/trunk/java/modules/kernel/src/org/apache/axis2/description/AxisService.java Wed Jun 27 04:13:26 2007
> @@ -741,6 +741,8 @@
> }
> } else {
> operationsAliasesMap.put(action, axisOperation);
> + //Adding operation name to the mapping table
> + operationsAliasesMap.put(axisOperation.getName().getLocalPart(), axisOperation);
> }
> }
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: axis-cvs-unsubscribe@ws.apache.org
> For additional commands, e-mail: axis-cvs-help@ws.apache.org
>
>
--
David Illsley - IBM Web Services Development
---------------------------------------------------------------------
To unsubscribe, e-mail: axis-cvs-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-cvs-help@ws.apache.org