Try/catch and wrap exceptions in Component (maybe also in MarkupContainer) for better diagnostics

[Martin Grigorov]

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