You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@synapse.apache.org by Ruwan Linton <ru...@gmail.com> on 2008/06/30 06:39:20 UTC

Re: svn commit: r672694 - /synapse/branches/1.2/modules/core/src/main/java/org/apache/synapse/mediators/ext/ClassMediator.java

Asankha,

Any reason for this commit to go into the 1.2 branch? Or is this just a
mistake?

Thanks,
Ruwan

On 6/30/08, asankha@apache.org <as...@apache.org> wrote:
>
> Author: asankha
> Date: Sun Jun 29 20:48:41 2008
> New Revision: 672694
>
> URL: http://svn.apache.org/viewvc?rev=672694&view=rev
> Log:
> let custom mediators throw their exceptions through class mediator
>
> Modified:
>
>     synapse/branches/1.2/modules/core/src/main/java/org/apache/synapse/mediators/ext/ClassMediator.java
>
> Modified:
> synapse/branches/1.2/modules/core/src/main/java/org/apache/synapse/mediators/ext/ClassMediator.java
> URL:
> http://svn.apache.org/viewvc/synapse/branches/1.2/modules/core/src/main/java/org/apache/synapse/mediators/ext/ClassMediator.java?rev=672694&r1=672693&r2=672694&view=diff
>
> ==============================================================================
> ---
> synapse/branches/1.2/modules/core/src/main/java/org/apache/synapse/mediators/ext/ClassMediator.java
> (original)
> +++
> synapse/branches/1.2/modules/core/src/main/java/org/apache/synapse/mediators/ext/ClassMediator.java
> Sun Jun 29 20:48:41 2008
> @@ -76,10 +76,13 @@
>
>          try {
>                         result = mediator.mediate(synCtx);
> +        } catch (SynapseException e) {
> +            // throw SynapseExceptions as they are..
> +            throw e;
>          } catch (Exception e) {
> -            // throw Synapse Exception for any exception in class meditor
> +            // throw a new SynapseException wrapping original, for any
> exception in class meditor
>              // so that the fault handler will be invoked
> -            throw new SynapseException("Error occured in the mediation of
> the class mediator", e);
> +            throw new SynapseException(e.getMessage(), e);
>          }
>
>          if (traceOrDebugOn) {
>
>
>


-- 
Ruwan Linton
http://wso2.org - "Oxygenating the Web Services Platform"
http://ruwansblog.blogspot.com/

Re: svn commit: r672694 - /synapse/branches/1.2/modules/core/src/main/java/org/apache/synapse/mediators/ext/ClassMediator.java

Posted by Ruwan Linton <ru...@gmail.com>.
On 6/30/08, Asankha C. Perera <as...@wso2.com> wrote:
>
>  Yep... its a mistake I noticed after I committed :(.. anyway we have the
> tag untouched.. so will leave this as it is for now
>

+1

Ruwan

asankha
>
> Ruwan Linton wrote:
>
> Asankha,
>
> Any reason for this commit to go into the 1.2 branch? Or is this just a
> mistake?
>
> Thanks,
> Ruwan
>
> On 6/30/08, asankha@apache.org <as...@apache.org> wrote:
>>
>> Author: asankha
>> Date: Sun Jun 29 20:48:41 2008
>> New Revision: 672694
>>
>> URL: http://svn.apache.org/viewvc?rev=672694&view=rev
>> Log:
>> let custom mediators throw their exceptions through class mediator
>>
>> Modified:
>>
>>     synapse/branches/1.2/modules/core/src/main/java/org/apache/synapse/mediators/ext/ClassMediator.java
>>
>>
>


-- 
Ruwan Linton
http://wso2.org - "Oxygenating the Web Services Platform"
http://ruwansblog.blogspot.com/

Re: svn commit: r672694 - /synapse/branches/1.2/modules/core/src/main/java/org/apache/synapse/mediators/ext/ClassMediator.java

Posted by "Asankha C. Perera" <as...@wso2.com>.
Yep... its a mistake I noticed after I committed :(.. anyway we have the 
tag untouched.. so will leave this as it is for now

asankha

Ruwan Linton wrote:
> Asankha,
>
> Any reason for this commit to go into the 1.2 branch? Or is this just 
> a mistake?
>
> Thanks,
> Ruwan
>
> On 6/30/08, *asankha@apache.org <ma...@apache.org>* 
> <asankha@apache.org <ma...@apache.org>> wrote:
>
>     Author: asankha
>     Date: Sun Jun 29 20:48:41 2008
>     New Revision: 672694
>
>     URL: http://svn.apache.org/viewvc?rev=672694&view=rev
>     <http://svn.apache.org/viewvc?rev=672694&view=rev>
>     Log:
>     let custom mediators throw their exceptions through class mediator
>
>     Modified:
>         synapse/branches/1.2/modules/core/src/main/java/org/apache/synapse/mediators/ext/ClassMediator.java
>