[OFBIZ-12061] Comment out HideUtilityClassConstructor in checkstyle.xml

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Implemented
• Sprint: OFBiz Community Day (Nov 2020)
• Assignee: Jacques Le Roux
• Reporter: Jacques Le Roux
• Created: 18/Nov/20 15:57
• Updated: 02/Dec/20 14:10
• Resolved: 02/Dec/20 14:10

Description

Currently Checkstyle returns 222 occurrences of the message "Utility classes should not have a public or default constructor."

This is due to https://checkstyle.sourceforge.io/config_design.html#HideUtilityClassConstructor
I don't think we need to follow the "prevents calls from subclass" suggestion there.

I rather suggest to comment out the related class in checkstyle.xml:

ie <!-- <module name="HideUtilityClassConstructor"/> -->

Then should remain only 280 errors and hopefully most will be easy to fix. At least they will be easier to spot...