Hi Dims,

It seems that the PhaseRuleTests.java is not running with the tests, so I
have added this testcase to the PreDispatchPhaseRuleTest.java and here
attach with is the patch to the test.

Due to this issue phaseLast="true" also doesn't work properly, (if you
specify any handler to be after a particular handler in that phase, then
that handler will be added after the phaseLast handler) and the second
addition causes this error.

Hope this will be fixed on the axis2-1.4

Thanks,
Ruwan

On Mon, Apr 21, 2008 at 5:34 PM, Davanum Srinivas <[EMAIL PROTECTED]> wrote:

> Ah. ok. that makes me feel slightly better. Please add the test here.
>
> modules\kernel\test\org\apache\axis2\phaserule\PhaseRuleTests.java
>
> -- dims
>
>
>
> On Mon, Apr 21, 2008 at 7:36 AM, Ruwan Linton <[EMAIL PROTECTED]>
> wrote:
> > Dims,
> >
> > Well we did track the axis2 trunk, but till rampart folks added this
> handler
> > with phaseLast set to true we couldn't find the issue.
> >
> > As far as I can see, without this fix there is no meaning of the after
> > attribute of the handlers in a module.xml
> >
> > Thanks,
> > Ruwan
> >
> >
> >
> > On Mon, Apr 21, 2008 at 5:02 PM, Davanum Srinivas <[EMAIL PROTECTED]>
> wrote:
> > > Deepal,
> > >
> > > Am afraid at making this change so late in the game..after 5 RC's.
> > > From what i recall, The phase code is so sensitive to JDK's. Can
> > > Synapse guys ship this class with their other axis2 classes?
> > >
> > > thanks,
> > > dims
> > >
> > > PS: Wish Syanpse trunk tracked Axis2 trunk. That would have avoided
> > > this issue entirely. But that would be asking for too much i guess..
> > >
> > >
> > >
> > >
> > > On Mon, Apr 21, 2008 at 6:42 AM, Deepal jayasinghe <[EMAIL PROTECTED]>
> > wrote:
> > > > Hi Dims,
> > > >
> > > >  I have attached a patch for
> > > > https://issues.apache.org/jira/browse/SYNAPSE-271 , which I consider
> as
> > > > critical for them. If you can please go though that and try to apply
> to
> > the
> > > > branch.
> > > >
> > > >  Thank you!
> > > >  Deepal
> > > >
> > > >
> > > > > Folks,
> > > > >
> > > > > Jarek (many thanks!) ran the JAXWS 2.1 TCK a little while ago and
> > > > everything looks good. Please don't check into 1.4
> > > > > branch till further notice.
> > > > >
> > > > > If you need something to get into 1.4 Final, please submit a diff
> > against
> > > > the branch and raise a JIRA with fix version
> > > > > to 1.4 and priority as blocker.
> > > > >
> > > > > Thanks,
> > > > > dims
> > > > >
> > > > > PS: Please remember there's a code freeze on
> axiom/xmlschema/neethi
> > > > trunk(s) as well.
> > > > >
> > > >
> > > >
> > > >
>  ---------------------------------------------------------------------
> > > >  To unsubscribe, e-mail: [EMAIL PROTECTED]
> > > >  For additional commands, e-mail: [EMAIL PROTECTED]
> > > >
> > > >
> > >
> > >
> > >
> > > --
> > > Davanum Srinivas :: http://davanum.wordpress.com
> > >
> > >
> > >
> > >
> > > ---------------------------------------------------------------------
> > > To unsubscribe, e-mail: [EMAIL PROTECTED]
> > > For additional commands, e-mail: [EMAIL PROTECTED]
> > >
> > >
> >
> >
> >
> > --
> >
> > Ruwan Linton
> > http://www.wso2.org - "Oxygenating the Web Services Platform"
>
>
>
> --
> Davanum Srinivas :: http://davanum.wordpress.com
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>


-- 
Ruwan Linton
http://www.wso2.org - "Oxygenating the Web Services Platform"

Reply via email to