Community
Participate
Working Groups
I20081028-0100 Throwing exceptions in the constructor gracefully disables the according cleanup ui and present the problem to the user. Exceptions in other methods (eg getCleanUpCount) are catched but only logged without the knowledge of the user. I think all exceptions in the contributed cleanup ui should be handled by disabling the UI of this cleanup. If not this could lead to real problems as the cleanup uses options the user may not have chosen.
The count and the selection count are really just for user information and there's no harm if this is wrong. Also, if there's a one time exception - maybe due to a corrupt file - we should not disable but let the user decide (like we do with failing completion processors).