diff --git a/fineract-provider/build.gradle b/fineract-provider/build.gradle index 4e4b4d8df..9c5dfaa89 100644 --- a/fineract-provider/build.gradle +++ b/fineract-provider/build.gradle @@ -395,7 +395,6 @@ tasks.withType(JavaCompile) { "InvalidParam", "ReturnFromVoid", "AlmostJavadoc", - "ProtectedMembersInFinalClass", "InvalidBlockTag", ) //TODO gradually enable these checks @@ -458,6 +457,7 @@ tasks.withType(JavaCompile) { "InvalidInlineTag", "MutablePublicArray", "StaticAssignmentInConstructor", + "ProtectedMembersInFinalClass", "OperatorPrecedence", "EqualsGetClass", "EqualsUnsafeCast", diff --git a/fineract-provider/src/main/java/org/apache/fineract/accounting/provisioning/service/ProvisioningEntriesReadPlatformServiceImpl.java b/fineract-provider/src/main/java/org/apache/fineract/accounting/provisioning/service/ProvisioningEntriesReadPlatformServiceImpl.java index 915971d1c..69ec72865 100644 --- a/fineract-provider/src/main/java/org/apache/fineract/accounting/provisioning/service/ProvisioningEntriesReadPlatformServiceImpl.java +++ b/fineract-provider/src/main/java/org/apache/fineract/accounting/provisioning/service/ProvisioningEntriesReadPlatformServiceImpl.java @@ -67,7 +67,7 @@ public class ProvisioningEntriesReadPlatformServiceImpl implements ProvisioningE private final StringBuilder sqlQuery; - protected LoanProductProvisioningEntryMapper(String formattedDate) { + private LoanProductProvisioningEntryMapper(String formattedDate) { sqlQuery = new StringBuilder().append( "select if(loan.loan_type_enum=1, mclient.office_id, mgroup.office_id) as office_id, loan.loan_type_enum, pcd.criteria_id as criteriaid, loan.product_id,loan.currency_code,") .append("GREATEST(datediff(").append(formattedDate) diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/domain/Report.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/domain/Report.java index 8dbd633ec..b885361b2 100644 --- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/domain/Report.java +++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/domain/Report.java @@ -111,7 +111,7 @@ public final class Report extends AbstractPersistableCustom { return new Report(reportName, reportType, reportSubType, reportCategory, description, useReport, reportSql, reportTypes); } - protected Report() { + Report() { // } diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/domain/ReportParameterUsage.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/domain/ReportParameterUsage.java index 2cdc460a9..d7d2f3aef 100644 --- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/domain/ReportParameterUsage.java +++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/domain/ReportParameterUsage.java @@ -40,7 +40,7 @@ public final class ReportParameterUsage extends AbstractPersistableCustom { @Column(name = "report_parameter_name") private String reportParameterName; - protected ReportParameterUsage() { + ReportParameterUsage() { // } diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/documentmanagement/domain/Image.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/documentmanagement/domain/Image.java index b82b49b71..373900f87 100644 --- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/documentmanagement/domain/Image.java +++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/documentmanagement/domain/Image.java @@ -38,7 +38,7 @@ public final class Image extends AbstractPersistableCustom { this.storageType = storageType.getValue(); } - protected Image() { + Image() { } diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/domain/Client.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/domain/Client.java index cd218e28a..097a537f0 100644 --- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/domain/Client.java +++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/domain/Client.java @@ -279,7 +279,7 @@ public final class Client extends AbstractPersistableCustom { savingsAccountId, dataOfBirth, gender, clientType, clientClassification, legalForm, isStaff); } - protected Client() { + Client() { this.setLegalForm(null); } diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/group/domain/Group.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/group/domain/Group.java index 746b311a0..e318d4cea 100644 --- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/group/domain/Group.java +++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/group/domain/Group.java @@ -148,7 +148,7 @@ public final class Group extends AbstractPersistableCustom { private List glimLoan; // JPA default constructor for entity - protected Group() { + Group() { this.name = null; this.externalId = null; this.clientMembers = new HashSet<>(); diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/LoanRepaymentScheduleInstallment.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/LoanRepaymentScheduleInstallment.java index 7172f58b2..b81757121 100644 --- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/LoanRepaymentScheduleInstallment.java +++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/LoanRepaymentScheduleInstallment.java @@ -132,7 +132,7 @@ public final class LoanRepaymentScheduleInstallment extends AbstractAuditableCus @JoinColumn(name = "loan_repayment_schedule_id", referencedColumnName = "id", nullable = false) private Set loanCompoundingDetails = new HashSet<>(); - protected LoanRepaymentScheduleInstallment() { + LoanRepaymentScheduleInstallment() { this.installmentNumber = null; this.fromDate = null; this.dueDate = null; diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/LoanSummary.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/LoanSummary.java index 0eb7da607..bf5ca2f21 100644 --- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/LoanSummary.java +++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/LoanSummary.java @@ -121,7 +121,7 @@ public final class LoanSummary { return new LoanSummary(totalFeeChargesDueAtDisbursement); } - protected LoanSummary() { + LoanSummary() { // } diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/paymentdetail/domain/PaymentDetail.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/paymentdetail/domain/PaymentDetail.java index 0eaadb30d..8c0cbaecf 100644 --- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/paymentdetail/domain/PaymentDetail.java +++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/paymentdetail/domain/PaymentDetail.java @@ -55,7 +55,7 @@ public final class PaymentDetail extends AbstractPersistableCustom { @Column(name = "bank_number", length = 50) private String bankNumber; - protected PaymentDetail() { + PaymentDetail() { } diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/FixedDepositProductData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/FixedDepositProductData.java index 7b580ebfa..2c61643e1 100644 --- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/FixedDepositProductData.java +++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/FixedDepositProductData.java @@ -39,17 +39,17 @@ public final class FixedDepositProductData extends DepositProductData { // additional fields private boolean preClosurePenalApplicable; - protected BigDecimal preClosurePenalInterest; - protected EnumOptionData preClosurePenalInterestOnType; - protected Integer minDepositTerm; - protected Integer maxDepositTerm; + private BigDecimal preClosurePenalInterest; + private EnumOptionData preClosurePenalInterestOnType; + private Integer minDepositTerm; + private Integer maxDepositTerm; private EnumOptionData minDepositTermType; private EnumOptionData maxDepositTermType; - protected Integer inMultiplesOfDepositTerm; - protected EnumOptionData inMultiplesOfDepositTermType; - protected BigDecimal minDepositAmount; - protected BigDecimal depositAmount; - protected BigDecimal maxDepositAmount; + private Integer inMultiplesOfDepositTerm; + private EnumOptionData inMultiplesOfDepositTermType; + private BigDecimal minDepositAmount; + private BigDecimal depositAmount; + private BigDecimal maxDepositAmount; private Collection preClosurePenalInterestOnTypeOptions; private Collection periodFrequencyTypeOptions; diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/SavingsAccountSummary.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/SavingsAccountSummary.java index a15debd19..9bc437bff 100644 --- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/SavingsAccountSummary.java +++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/SavingsAccountSummary.java @@ -82,7 +82,7 @@ public final class SavingsAccountSummary { @Column(name = "last_interest_calculation_date") private Date lastInterestCalculationDate; - protected SavingsAccountSummary() { + SavingsAccountSummary() { // } diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/SavingsAccountTransaction.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/SavingsAccountTransaction.java index bee444f50..bb7269d96 100644 --- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/SavingsAccountTransaction.java +++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/SavingsAccountTransaction.java @@ -130,7 +130,7 @@ public final class SavingsAccountTransaction extends AbstractPersistableCustom { @JoinColumn(name = "savings_account_transaction_id", referencedColumnName = "id") private List notes = new ArrayList<>(); - protected SavingsAccountTransaction() { + SavingsAccountTransaction() { this.dateOf = null; this.typeOf = null; this.createdDate = null;