More protected methods are needed in FullAjaxExceptionHandler

[omnifaces]

More protected methods are needed in FullAjaxExceptionHandler

[BalusC]

I wonder what exactly you'd like to do in resetResponse() and renderErrorPageView() instead as changing them would violate the purpose of FullAjaxExceptionHandler of showing an <error-page> exactly the same way as during a synchronous request.

Those constants are since OmniFaces 2.0 replaced by Servlet 3.0 ones.

[ova2]

I can not remember about resetResponse (have to look into the project code), but renderErrorPageView is e.g. needed to be able to show an error on the same page. We only show unexpected errors on a separate page. Business errors are shown on the same page by invoking navigate in the NavigationHandler with the outcome null. Business errors are thrown across the whole web app and we don't need to change the view for them because the user can continue his/her work. I try to summarize: all exceptions should be treated in the same ExceptionHandler, but the handling differs in exception's type - some exceptions are shown on the current page and some on the separate page.

[BalusC]

Closing off as this actually defeats purpose of FullAjaxExceptionHandler.

To implement the desired functionaliry, better override shouldHandleExceptionRootCause().

protected boolean shouldHandleExceptionRootCause(FacesContext context, Throwable exception) {
    if (isContinuableBusinessException(exception)) { 
        return true;
    }
    else {
        // Do your specific thing here. Show a faces message?
        // ...
        return false;
    }
}

[meprabu]

Hi I tried the above logic to show my business exception on the same page but i dont the values that i set in the context in the xhtml. This is the response that i get.

class gov.ms.mdes.access.framework.exception.BaseRunTimeException

Please see the code here
http://stackoverflow.com/questions/34144536/omnifaces-fullajaxexceptionhandler