Seaside-Core-NickAger.720

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

Seaside-Core-NickAger.720

Philippe Marschall
Hi

I think this methods needs to be deprecated. I fear it would break too
much code that sets up a RequestContext. IIRC there was a thread about
Seaside-Email where somebody would set up a RequestContext.

Cheers
Philippe
_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
Reply | Threaded
Open this post in threaded view
|

Re: Seaside-Core-NickAger.720

Julian Fitzell-2
I did wonder about that after we committed... you're probably right.

Nick can you commit another version with the original method deprecated?

On Tue, Jul 12, 2011 at 9:26 PM, Philippe Marschall
<[hidden email]> wrote:

> Hi
>
> I think this methods needs to be deprecated. I fear it would break too
> much code that sets up a RequestContext. IIRC there was a thread about
> Seaside-Email where somebody would set up a RequestContext.
>
> Cheers
> Philippe
> _______________________________________________
> seaside-dev mailing list
> [hidden email]
> http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
>
_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
Reply | Threaded
Open this post in threaded view
|

Re: Seaside-Core-NickAger.720

Nick


On 13 July 2011 09:10, Julian Fitzell <[hidden email]> wrote:
I did wonder about that after we committed... you're probably right.

Nick can you commit another version with the original method deprecated?

Done:

Name: Seaside-Core-NickAger.721
Author: NickAger
Time: 14 July 2011, 8:09:27 am
UUID: 38778695-c02c-418b-a5c9-2c819195d93b
Ancestors: Seaside-Core-NickAger.720

re-added:
   WARequestContext>>#push:while:
as a deprecated API calling:
   WARequestContext>>#push:during:  

_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev