FINERACT-822 enable JdkObsolete error

fineract-provider/build.gradle

@@ -307,7 +307,6 @@ tasks.withType(JavaCompile) {
                 "UnusedVariable",
                 "InconsistentCapitalization",
                 "ClassCanBeStatic",
-                "OperatorPrecedence",
                 "MixedMutabilityReturnType",
                 "SameNameButDifferent",
                 "ReferenceEquality",
@@ -325,7 +324,6 @@ tasks.withType(JavaCompile) {
                 "DoubleBraceInitialization",
                 "UnusedNestedClass",
                 "UndefinedEquals",
-                "JdkObsolete",
         )
         //TODO gradually enable these checks
         error(
@@ -368,6 +366,7 @@ tasks.withType(JavaCompile) {
                 "MissingCasesInEnumSwitch",
                 "OperatorPrecedence",
                 "EqualsGetClass",
+                "JdkObsolete",
 //                "ReturnMissingNullable",
 //                "InconsistentOverloads",
 //                "MethodCanBeStatic",

fineract-provider/src/main/java/org/apache/fineract/accounting/glaccount/domain/GLAccount.java

@@ -18,8 +18,8 @@
  */
 package org.apache.fineract.accounting.glaccount.domain;
 
+import java.util.ArrayList;
 import java.util.LinkedHashMap;
-import java.util.LinkedList;
 import java.util.List;
 import java.util.Map;
 import javax.persistence.Column;
@@ -50,7 +50,7 @@ public class GLAccount extends AbstractPersistableCustom {
 
     @OneToMany(fetch = FetchType.LAZY)
     @JoinColumn(name = "parent_id")
-    private List<GLAccount> children = new LinkedList<>();
+    private List<GLAccount> children = new ArrayList<>();
 
     @Column(name = "name", nullable = false, length = 45)
     private String name;
@@ -245,4 +245,4 @@ public class GLAccount extends AbstractPersistableCustom {
         this.parent = parentAccount;
         generateHierarchy();
     }
-}
+}

fineract-provider/src/main/java/org/apache/fineract/organisation/office/domain/Office.java

@@ -19,9 +19,9 @@
 package org.apache.fineract.organisation.office.domain;
 
 import java.io.Serializable;
+import java.util.ArrayList;
 import java.util.Date;
 import java.util.LinkedHashMap;
-import java.util.LinkedList;
 import java.util.List;
 import java.util.Map;
 import javax.persistence.Column;
@@ -48,7 +48,7 @@ public class Office extends AbstractPersistableCustom implements Serializable {
 
     @OneToMany(fetch = FetchType.LAZY)
     @JoinColumn(name = "parent_id")
-    private List<Office> children = new LinkedList<>();
+    private List<Office> children = new ArrayList<>();
 
     @ManyToOne(fetch = FetchType.LAZY)
     @JoinColumn(name = "parent_id")

fineract-provider/src/main/java/org/apache/fineract/portfolio/group/domain/Group.java

@@ -22,7 +22,6 @@ import java.util.ArrayList;
 import java.util.Date;
 import java.util.HashSet;
 import java.util.LinkedHashMap;
-import java.util.LinkedList;
 import java.util.List;
 import java.util.Map;
 import java.util.Set;
@@ -107,7 +106,7 @@ public final class Group extends AbstractPersistableCustom {
 
     @OneToMany(fetch = FetchType.EAGER)
     @JoinColumn(name = "parent_id")
-    private List<Group> groupMembers = new LinkedList<>();
+    private List<Group> groupMembers = new ArrayList<>();
 
     @ManyToMany
     @JoinTable(name = "m_group_client", joinColumns = @JoinColumn(name = "group_id"), inverseJoinColumns = @JoinColumn(name = "client_id"))