default exception handler WAWalkbackErrorHandler only present in development, not runtime

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

default exception handler WAWalkbackErrorHandler only present in development, not runtime

Thomas Brodt-2
Hi

in Seaside (3.2.0 - JohanBrichau.85 1 in the Visualworks 8.3 port) class
WAEnvironment installs WAWalkbackErrorHandler as default
exceptionHandler in method
#configureApplicationExceptionHandlingDefaults called from initialize.
But WAWalkbackErrorHandler is included in package Seaside-Development,
so if you build runtimes without Seaside development packages, this
fails because WAWalkbackErrorHandler is not present in such runtime images.

Shouldn't rather WAHtmlErrorHandler be used as default exception handler
which is in the same package as WAEnvironment? And then maybe replaced
by WAWalkbackErrorHandler if you additionally load development?

Any thoughts?

Thomas

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

Re: default exception handler WAWalkbackErrorHandler only present in development, not runtime

Philippe Marschall
Hello Thomas

Sorry for not replying. This has been corrected in 3.3 [1] which is
not released yet.

 [1] https://github.com/SeasideSt/Seaside/issues/959

Cheers
Philippe
On Wed, Feb 14, 2018 at 5:14 PM Thomas Brodt
<[hidden email]> wrote:

>
> Hi
>
> in Seaside (3.2.0 - JohanBrichau.85 1 in the Visualworks 8.3 port) class
> WAEnvironment installs WAWalkbackErrorHandler as default
> exceptionHandler in method
> #configureApplicationExceptionHandlingDefaults called from initialize.
> But WAWalkbackErrorHandler is included in package Seaside-Development,
> so if you build runtimes without Seaside development packages, this
> fails because WAWalkbackErrorHandler is not present in such runtime images.
>
> Shouldn't rather WAHtmlErrorHandler be used as default exception handler
> which is in the same package as WAEnvironment? And then maybe replaced
> by WAWalkbackErrorHandler if you additionally load development?
>
> Any thoughts?
>
> Thomas
>
> _______________________________________________
> seaside mailing list
> [hidden email]
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
_______________________________________________
seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: default exception handler WAWalkbackErrorHandler only present in development, not runtime

Thomas Brodt-2
So already fixed along time ago, that‘s great.

Thank you Philippe for this extra informal support at esug!

Now 3.3 has to come out and then be available for visualworks and there you go.
Thomas

> Am 11.09.2018 um 10:34 schrieb Philippe Marschall <[hidden email]>:
>
> Hello Thomas
>
> Sorry for not replying. This has been corrected in 3.3 [1] which is
> not released yet.
>
> [1] https://github.com/SeasideSt/Seaside/issues/959
>
> Cheers
> Philippe
> On Wed, Feb 14, 2018 at 5:14 PM Thomas Brodt
> <[hidden email]> wrote:
>>
>> Hi
>>
>> in Seaside (3.2.0 - JohanBrichau.85 1 in the Visualworks 8.3 port) class
>> WAEnvironment installs WAWalkbackErrorHandler as default
>> exceptionHandler in method
>> #configureApplicationExceptionHandlingDefaults called from initialize.
>> But WAWalkbackErrorHandler is included in package Seaside-Development,
>> so if you build runtimes without Seaside development packages, this
>> fails because WAWalkbackErrorHandler is not present in such runtime images.
>>
>> Shouldn't rather WAHtmlErrorHandler be used as default exception handler
>> which is in the same package as WAEnvironment? And then maybe replaced
>> by WAWalkbackErrorHandler if you additionally load development?
>>
>> Any thoughts?
>>
>> Thomas
>>
>> _______________________________________________
>> seaside mailing list
>> [hidden email]
>> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
> _______________________________________________
> seaside mailing list
> [hidden email]
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside

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