Can't this wrapping be done on the code that calls detach?

On Mon, Dec 15, 2014 at 8:41 AM, Martin Grigorov <[email protected]>
wrote:
>
> Hi,
>
> https://issues.apache.org/jira/browse/WICKET-5776 suggests to try/catch
> and
> wrap exceptions in Component#detach() so that it is easier for the
> application developer to identify the problematic component.
> The reporter suggests code like:
>
> void detach() {
>   try {
>     // detachChildren  } catch (ComponentDetachException cde) {
>     throw cde;
>   } catch (Exception e) {
>     throw new ComponentDetachException("Error detaching component " +
> getPath() +
>       (getDefaultModel() != null ? " with a " +
> getDefaultModel().getClass().getName() + " model" : "") + ": " + e,
> e);
>   }
> }
>
>
> In a comment in the ticket I explain my concerns.
>
> What do you think about this suggestion ?
>
> Martin Grigorov
> Wicket Training and Consulting
> https://twitter.com/mtgrigorov
>


-- 
Regards - Ernesto Reinaldo Barreiro

Reply via email to