FINERACT-821 Added and Enforced EmptyLineSeparator Checkstyle

fineract-provider/config/checkstyle/checkstyle.xml

@@ -93,6 +93,11 @@
             <property name="format" value="(^[A-Z][0-9]?)$|([A-Z][a-zA-Z0-9]*[T]$)"/>
             <message key="name.invalidPattern"
              value="Method type name ''{0}'' must match pattern ''{1}''."/>
+        </module>
+        <module name="EmptyLineSeparator">
+            <property name="allowNoEmptyLineBetweenFields" value="true"/>
+            <!-- No PACKAGE_DEF token, as we like our license header to be glue to "package" statement, to save 1 line screen space -->
+            <property name="tokens" value="IMPORT, CLASS_DEF, INTERFACE_DEF, ENUM_DEF, STATIC_INIT, INSTANCE_INIT, METHOD_DEF, CTOR_DEF, VARIABLE_DEF"/>
         </module>
           <module name="PackageName">
             <property name="format" value="^[a-z]+(\.[a-z][a-z0-9_]*)*$"/>
@@ -177,11 +182,7 @@
 
         <module name="ModifierOrder"/>
         <module name="RedundantModifier"/>
-        <module name="EmptyLineSeparator">
-            <property name="allowNoEmptyLineBetweenFields" value="true"/>
-            < ! - - No PACKAGE_DEF token, as we like our license header to be glue to "package" statement, to save 1 line screen space - - >
-            <property name="tokens" value="IMPORT, CLASS_DEF, INTERFACE_DEF, ENUM_DEF, STATIC_INIT, INSTANCE_INIT, METHOD_DEF, CTOR_DEF, VARIABLE_DEF"/>
-        </module>
+
         <module name="SeparatorWrap">
             <property name="tokens" value="DOT"/>
             <property name="option" value="nl"/>

fineract-provider/src/main/java/org/apache/fineract/portfolio/calendar/service/CalendarUtils.java

@@ -57,6 +57,7 @@ import org.slf4j.LoggerFactory;
 public class CalendarUtils {
 
     private final static Logger LOG = LoggerFactory.getLogger(CalendarUtils.class);
+
     static {
         System.setProperty("net.fortuna.ical4j.timezone.date.floating", "true");
     }

fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/LoanTopupDetails.java

@@ -43,7 +43,7 @@ public class LoanTopupDetails extends AbstractPersistableCustom {
     @Column(name = "topup_amount", nullable = true)
     private BigDecimal topupAmount;
 
-    protected LoanTopupDetails() {};
+    protected LoanTopupDetails() {}
 
     public LoanTopupDetails(final Loan loan, final Long loanIdToClose) {
         this.loan = loan;