fineract-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From vorbur...@apache.org
Subject [fineract] branch develop updated: FINERACT-821 Added and Enforced FinalClass checkstyle
Date Thu, 18 Jun 2020 07:31:25 GMT
This is an automated email from the ASF dual-hosted git repository.

vorburger pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/fineract.git


The following commit(s) were added to refs/heads/develop by this push:
     new 74b04cd  FINERACT-821 Added and Enforced FinalClass checkstyle
74b04cd is described below

commit 74b04cd4d350e4426cf30cdcc4e47941f7a5136c
Author: thesmallstar <manthan.surkar@gmail.com>
AuthorDate: Thu Jun 18 02:38:34 2020 +0530

    FINERACT-821 Added and Enforced FinalClass checkstyle
---
 fineract-provider/config/checkstyle/checkstyle.xml                    | 3 ++-
 .../java/org/apache/fineract/integrationtests/common/BatchHelper.java | 2 +-
 .../org/apache/fineract/integrationtests/common/CenterDomain.java     | 2 +-
 .../org/apache/fineract/integrationtests/common/CurrencyDomain.java   | 2 +-
 .../org/apache/fineract/integrationtests/common/OfficeDomain.java     | 2 +-
 .../apache/fineract/integrationtests/common/PaymentTypeDomain.java    | 2 +-
 .../apache/fineract/integrationtests/common/funds/FundsHelper.java    | 2 +-
 .../accounting/journalentry/data/OfficeOpeningBalancesData.java       | 2 +-
 .../src/main/java/org/apache/fineract/adhocquery/domain/AdHoc.java    | 2 +-
 .../main/java/org/apache/fineract/batch/command/CommandContext.java   | 4 ++--
 .../fineract/infrastructure/bulkimport/data/BulkImportEvent.java      | 2 +-
 .../org/apache/fineract/infrastructure/bulkimport/data/Count.java     | 2 +-
 .../apache/fineract/infrastructure/bulkimport/data/ImportData.java    | 2 +-
 .../java/org/apache/fineract/infrastructure/cache/data/CacheData.java | 2 +-
 .../infrastructure/campaigns/email/data/EmailCampaignData.java        | 2 +-
 .../infrastructure/campaigns/email/data/EmailConfigurationData.java   | 2 +-
 .../campaigns/email/data/EmailConfigurationValidator.java             | 2 +-
 .../fineract/infrastructure/campaigns/email/data/EmailData.java       | 2 +-
 .../campaigns/email/data/EmailMessageWithAttachmentData.java          | 2 +-
 .../campaigns/email/service/EmailMessageJobEmailServiceImpl.java      | 2 +-
 .../fineract/infrastructure/campaigns/sms/data/SmsCampaignData.java   | 2 +-
 .../java/org/apache/fineract/infrastructure/codes/data/CodeData.java  | 2 +-
 .../fineract/infrastructure/core/data/PaginationParameters.java       | 2 +-
 .../creditbureau/data/CreditBureauConfigurationData.java              | 2 +-
 .../fineract/infrastructure/creditbureau/data/CreditBureauData.java   | 2 +-
 .../creditbureau/data/CreditBureauLoanProductMappingData.java         | 2 +-
 .../infrastructure/creditbureau/data/CreditBureauMasterData.java      | 2 +-
 .../infrastructure/creditbureau/data/CreditBureauProduct.java         | 2 +-
 .../creditbureau/data/OrganisationCreditBureauData.java               | 2 +-
 .../fineract/infrastructure/dataqueries/data/DatatableData.java       | 2 +-
 .../apache/fineract/infrastructure/dataqueries/data/ReportData.java   | 2 +-
 .../fineract/infrastructure/dataqueries/data/ReportParameterData.java | 2 +-
 .../infrastructure/dataqueries/data/ReportParameterJoinData.java      | 2 +-
 .../fineract/infrastructure/dataqueries/data/ResultsetRowData.java    | 2 +-
 .../entityaccess/data/FineractEntityToEntityMappingData.java          | 2 +-
 .../infrastructure/entityaccess/domain/FineractEntityAccessType.java  | 2 +-
 .../entityaccess/domain/FineractEntityRelationRepositoryWrapper.java  | 2 +-
 .../infrastructure/entityaccess/domain/FineractEntityType.java        | 2 +-
 .../apache/fineract/infrastructure/gcm/domain/DeviceRegistration.java | 2 +-
 .../fineract/infrastructure/gcm/domain/DeviceRegistrationData.java    | 2 +-
 .../java/org/apache/fineract/infrastructure/hooks/data/Event.java     | 2 +-
 .../java/org/apache/fineract/infrastructure/hooks/data/Field.java     | 2 +-
 .../java/org/apache/fineract/infrastructure/hooks/data/HookData.java  | 2 +-
 .../apache/fineract/infrastructure/hooks/data/HookTemplateData.java   | 2 +-
 .../reportmailingjob/data/ReportMailingJobConfigurationData.java      | 2 +-
 .../infrastructure/reportmailingjob/data/ReportMailingJobData.java    | 2 +-
 .../reportmailingjob/data/ReportMailingJobRunHistoryData.java         | 2 +-
 .../fineract/infrastructure/security/data/PlatformRequestLog.java     | 2 +-
 .../java/org/apache/fineract/infrastructure/sms/data/SmsData.java     | 2 +-
 .../fineract/infrastructure/survey/data/SurveyDataTableData.java      | 2 +-
 .../apache/fineract/interoperation/data/InteropQuoteResponseData.java | 2 +-
 .../interoperation/data/InteropTransactionRequestResponseData.java    | 2 +-
 .../fineract/interoperation/data/InteropTransferResponseData.java     | 2 +-
 .../fineract/organisation/office/data/OfficeTransactionData.java      | 2 +-
 .../organisation/provisioning/data/ProvisioningCriteriaData.java      | 2 +-
 .../fineract/organisation/staff/data/BulkTransferLoanOfficerData.java | 2 +-
 .../java/org/apache/fineract/organisation/staff/data/StaffData.java   | 2 +-
 .../apache/fineract/organisation/teller/domain/CashierTxnType.java    | 2 +-
 .../apache/fineract/portfolio/account/data/AccountTransferData.java   | 2 +-
 .../fineract/portfolio/account/data/StandingInstructionData.java      | 2 +-
 .../org/apache/fineract/portfolio/address/data/ClientAddressData.java | 2 +-
 .../fineract/portfolio/address/data/FieldConfigurationData.java       | 2 +-
 .../org/apache/fineract/portfolio/calendar/data/CalendarData.java     | 2 +-
 .../java/org/apache/fineract/portfolio/charge/data/ChargeData.java    | 2 +-
 .../org/apache/fineract/portfolio/client/data/ClientChargeData.java   | 2 +-
 .../java/org/apache/fineract/portfolio/client/data/ClientData.java    | 2 +-
 .../fineract/portfolio/client/data/ClientFamilyMembersData.java       | 2 +-
 .../apache/fineract/portfolio/client/data/ClientTransactionData.java  | 2 +-
 .../ClientFamilyMemberCommandFromApiJsonDeserializer.java             | 2 +-
 .../org/apache/fineract/portfolio/collateral/data/CollateralData.java | 2 +-
 .../fineract/portfolio/collectionsheet/data/IndividualClientData.java | 2 +-
 .../portfolio/collectionsheet/data/IndividualCollectionSheetData.java | 2 +-
 .../apache/fineract/portfolio/collectionsheet/data/JLGClientData.java | 2 +-
 .../portfolio/collectionsheet/data/JLGCollectionSheetData.java        | 2 +-
 .../apache/fineract/portfolio/collectionsheet/data/JLGGroupData.java  | 2 +-
 .../fineract/portfolio/collectionsheet/data/SavingsDueData.java       | 2 +-
 .../main/java/org/apache/fineract/portfolio/fund/data/FundData.java   | 2 +-
 .../java/org/apache/fineract/portfolio/group/data/CenterData.java     | 2 +-
 .../org/apache/fineract/portfolio/group/data/StaffCenterData.java     | 2 +-
 .../portfolio/interestratechart/data/InterestIncentiveData.java       | 2 +-
 .../portfolio/interestratechart/data/InterestRateChartData.java       | 2 +-
 .../portfolio/interestratechart/data/InterestRateChartSlabData.java   | 2 +-
 .../fineract/portfolio/loanaccount/data/GlimRepaymentTemplate.java    | 2 +-
 .../loanaccount/data/GroupLoanIndividualMonitoringAccountData.java    | 2 +-
 .../apache/fineract/portfolio/loanaccount/data/LoanAccountData.java   | 2 +-
 .../loanaccount/domain/GroupLoanIndividualMonitoringAccount.java      | 2 +-
 .../portfolio/loanaccount/guarantor/data/GuarantorFundingData.java    | 2 +-
 .../loanaccount/guarantor/data/GuarantorTransactionData.java          | 2 +-
 .../fineract/portfolio/loanaccount/guarantor/data/ObligeeData.java    | 2 +-
 .../portfolio/loanaccount/loanschedule/data/LoanScheduleDTO.java      | 2 +-
 .../portfolio/loanaccount/loanschedule/data/LoanScheduleParams.java   | 2 +-
 .../loanaccount/loanschedule/data/LoanSchedulePeriodData.java         | 2 +-
 .../loanaccount/rescheduleloan/data/LoanRescheduleRequestData.java    | 2 +-
 .../fineract/portfolio/loanproduct/data/LoanProductGuaranteeData.java | 2 +-
 .../portfolio/meeting/attendance/data/ClientAttendanceData.java       | 2 +-
 .../java/org/apache/fineract/portfolio/meeting/data/MeetingData.java  | 2 +-
 .../main/java/org/apache/fineract/portfolio/rate/data/RateData.java   | 2 +-
 .../fineract/portfolio/savings/data/ClosingOfSavingsAccounts.java     | 2 +-
 .../portfolio/savings/data/DepositAccountInterestIncentiveData.java   | 2 +-
 .../portfolio/savings/data/DepositAccountInterestRateChartData.java   | 2 +-
 .../savings/data/DepositAccountInterestRateChartSlabData.java         | 2 +-
 .../portfolio/savings/data/DepositAccountOnHoldTransactionData.java   | 2 +-
 .../fineract/portfolio/savings/data/FixedDepositAccountData.java      | 2 +-
 .../fineract/portfolio/savings/data/FixedDepositProductData.java      | 2 +-
 .../savings/data/GroupSavingsIndividualMonitoringAccountData.java     | 2 +-
 .../fineract/portfolio/savings/data/RecurringDepositAccountData.java  | 2 +-
 .../fineract/portfolio/savings/data/RecurringDepositProductData.java  | 2 +-
 .../fineract/portfolio/savings/data/SavingsAccountAnnualFeeData.java  | 2 +-
 .../apache/fineract/portfolio/savings/data/SavingsAccountData.java    | 2 +-
 .../portfolio/savings/data/SavingsAccountTransactionData.java         | 2 +-
 .../org/apache/fineract/portfolio/savings/data/SavingsActivation.java | 2 +-
 .../org/apache/fineract/portfolio/savings/data/SavingsApproval.java   | 2 +-
 .../apache/fineract/portfolio/savings/data/SavingsProductData.java    | 2 +-
 .../portfolio/savings/domain/GroupSavingsIndividualMonitoring.java    | 2 +-
 .../portfolio/savings/domain/interest/AnnualCompoundingPeriod.java    | 2 +-
 .../portfolio/savings/domain/interest/BiAnnualCompoundingPeriod.java  | 2 +-
 .../portfolio/savings/domain/interest/DailyCompoundingPeriod.java     | 2 +-
 .../portfolio/savings/domain/interest/MonthlyCompoundingPeriod.java   | 2 +-
 .../fineract/portfolio/savings/domain/interest/PostingPeriod.java     | 2 +-
 .../portfolio/savings/domain/interest/QuarterlyCompoundingPeriod.java | 2 +-
 .../apache/fineract/portfolio/search/data/AdHocSearchQueryData.java   | 2 +-
 .../portfolio/self/pockets/data/PocketAccountMappingData.java         | 2 +-
 .../fineract/portfolio/shareproducts/data/ShareProductData.java       | 2 +-
 .../java/org/apache/fineract/portfolio/tax/data/TaxComponentData.java | 2 +-
 .../java/org/apache/fineract/portfolio/tax/data/TaxGroupData.java     | 2 +-
 .../src/main/java/org/apache/fineract/spm/util/LookupTableMapper.java | 2 +-
 .../src/main/java/org/apache/fineract/spm/util/ScorecardMapper.java   | 2 +-
 .../src/main/java/org/apache/fineract/spm/util/SurveyMapper.java      | 2 +-
 .../src/main/java/org/apache/fineract/template/data/TemplateData.java | 2 +-
 .../java/org/apache/fineract/useradministration/data/AppUserData.java | 2 +-
 .../org/apache/fineract/useradministration/data/PermissionData.java   | 2 +-
 131 files changed, 133 insertions(+), 132 deletions(-)

diff --git a/fineract-provider/config/checkstyle/checkstyle.xml b/fineract-provider/config/checkstyle/checkstyle.xml
index 9788ab4..3519ad4 100644
--- a/fineract-provider/config/checkstyle/checkstyle.xml
+++ b/fineract-provider/config/checkstyle/checkstyle.xml
@@ -57,6 +57,7 @@
             <property name="ignoreComments" value="false" />
         </module>
         <module name="EqualsHashCode"/>
+        <module name="FinalClass"/>
         <module name="SeparatorWrap">
             <property name="tokens" value="COMMA"/>
             <property name="option" value="EOL"/>
@@ -177,7 +178,7 @@
             <property name="ignoreSetter" value="true" />
             <property name="setterCanReturnItsClass" value="true" />
         </module>
-        <module name="FinalClass"/>
+
         <module name="HideUtilityClassConstructor"/>
 
         < ! - - TODO Checks for Exception Handling Anti-Patterns - - >
diff --git a/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/BatchHelper.java b/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/BatchHelper.java
index 310157a..8bb347a 100644
--- a/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/BatchHelper.java
+++ b/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/BatchHelper.java
@@ -40,7 +40,7 @@ import org.slf4j.LoggerFactory;
  *
  * @see org.apache.fineract.integrationtests.BatchApiTest
  */
-public class BatchHelper {
+public final class BatchHelper {
 
     private final static Logger LOG = LoggerFactory.getLogger(BatchHelper.class);
     private static final String BATCH_API_URL = "/fineract-provider/api/v1/batches?" + Utils.TENANT_IDENTIFIER;
diff --git a/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/CenterDomain.java b/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/CenterDomain.java
index 22fc139..87ea6fc 100644
--- a/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/CenterDomain.java
+++ b/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/CenterDomain.java
@@ -28,7 +28,7 @@ public class CenterDomain implements Comparable<CenterDomain> {
 
     private final static Logger LOG = LoggerFactory.getLogger(CenterDomain.class);
 
-    public static class Builder {
+    public final static class Builder {
 
         private final Integer id;
         private final String accountNo;
diff --git a/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/CurrencyDomain.java b/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/CurrencyDomain.java
index a96e12b..a62994e 100644
--- a/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/CurrencyDomain.java
+++ b/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/CurrencyDomain.java
@@ -22,7 +22,7 @@ import com.google.gson.Gson;
 
 public class CurrencyDomain implements Comparable<CurrencyDomain> {
 
-    public static class Builder {
+    public final static class Builder {
 
         private String code;
         private String name;
diff --git a/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/OfficeDomain.java b/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/OfficeDomain.java
index d936d99..a522b46 100644
--- a/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/OfficeDomain.java
+++ b/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/OfficeDomain.java
@@ -23,7 +23,7 @@ import java.util.Arrays;
 
 public class OfficeDomain {
 
-    public static class Builder {
+    public final static class Builder {
 
         private int id;
         private String name;
diff --git a/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/PaymentTypeDomain.java b/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/PaymentTypeDomain.java
index 7437ec8..df780c2 100644
--- a/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/PaymentTypeDomain.java
+++ b/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/PaymentTypeDomain.java
@@ -18,7 +18,7 @@
  */
 package org.apache.fineract.integrationtests.common;
 
-public class PaymentTypeDomain {
+public final class PaymentTypeDomain {
 
     private Integer id;
     private String name;
diff --git a/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/funds/FundsHelper.java b/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/funds/FundsHelper.java
index 9ad0aa0..604b088 100644
--- a/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/funds/FundsHelper.java
+++ b/fineract-provider/src/integrationTest/java/org/apache/fineract/integrationtests/common/funds/FundsHelper.java
@@ -22,7 +22,7 @@ import com.google.gson.Gson;
 
 public class FundsHelper {
 
-    public static class Builder {
+    final public static class Builder {
 
         private String name;
         private String externalId;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/accounting/journalentry/data/OfficeOpeningBalancesData.java b/fineract-provider/src/main/java/org/apache/fineract/accounting/journalentry/data/OfficeOpeningBalancesData.java
index 9333fce..627dc8f 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/accounting/journalentry/data/OfficeOpeningBalancesData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/accounting/journalentry/data/OfficeOpeningBalancesData.java
@@ -22,7 +22,7 @@ import java.util.List;
 import org.apache.fineract.accounting.glaccount.data.GLAccountData;
 import org.joda.time.LocalDate;
 
-public class OfficeOpeningBalancesData {
+public final class OfficeOpeningBalancesData {
 
     @SuppressWarnings("unused")
     private final Long officeId;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/adhocquery/domain/AdHoc.java b/fineract-provider/src/main/java/org/apache/fineract/adhocquery/domain/AdHoc.java
index 84838f5..2f7782a 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/adhocquery/domain/AdHoc.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/adhocquery/domain/AdHoc.java
@@ -32,7 +32,7 @@ import org.apache.fineract.infrastructure.security.utils.SQLInjectionValidator;
 
 @Entity
 @Table(name = "m_adhoc")
-public class AdHoc extends AbstractAuditableCustom {
+public final class AdHoc extends AbstractAuditableCustom {
 
     @Column(name = "name", length = 100)
     private String name;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/batch/command/CommandContext.java b/fineract-provider/src/main/java/org/apache/fineract/batch/command/CommandContext.java
index 4f2275a..d8a8bb4 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/batch/command/CommandContext.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/batch/command/CommandContext.java
@@ -28,14 +28,14 @@ package org.apache.fineract.batch.command;
  *
  * @see org.apache.fineract.batch.service.BatchApiService
  */
-public class CommandContext {
+public final class CommandContext {
 
     /**
      * Static Builder class to provide a Build method for CommandContext.
      *
      * @author Rishabh Shukla
      */
-    public static class Builder {
+    public final static class Builder {
 
         private String resource;
         private String method;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/bulkimport/data/BulkImportEvent.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/bulkimport/data/BulkImportEvent.java
index 66dd11b..4996ddd 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/bulkimport/data/BulkImportEvent.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/bulkimport/data/BulkImportEvent.java
@@ -21,7 +21,7 @@ package org.apache.fineract.infrastructure.bulkimport.data;
 import org.apache.poi.ss.usermodel.Workbook;
 import org.springframework.context.ApplicationEvent;
 
-public class BulkImportEvent extends ApplicationEvent {
+public final class BulkImportEvent extends ApplicationEvent {
 
     private final String tenantIdentifier;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/bulkimport/data/Count.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/bulkimport/data/Count.java
index be73123..696b6fe 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/bulkimport/data/Count.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/bulkimport/data/Count.java
@@ -18,7 +18,7 @@
  */
 package org.apache.fineract.infrastructure.bulkimport.data;
 
-public class Count {
+public final class Count {
 
     private Integer successCount;
     private Integer errorCount;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/bulkimport/data/ImportData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/bulkimport/data/ImportData.java
index 8349930..5f6dae4 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/bulkimport/data/ImportData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/bulkimport/data/ImportData.java
@@ -20,7 +20,7 @@ package org.apache.fineract.infrastructure.bulkimport.data;
 
 import org.joda.time.LocalDate;
 
-public class ImportData {
+public final class ImportData {
 
     @SuppressWarnings("unused")
     private Long importId;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/cache/data/CacheData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/cache/data/CacheData.java
index 1a4d543..7f0240d 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/cache/data/CacheData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/cache/data/CacheData.java
@@ -20,7 +20,7 @@ package org.apache.fineract.infrastructure.cache.data;
 
 import org.apache.fineract.infrastructure.core.data.EnumOptionData;
 
-public class CacheData {
+public final class CacheData {
 
     @SuppressWarnings("unused")
     private final EnumOptionData cacheType;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/data/EmailCampaignData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/data/EmailCampaignData.java
index 76984ce..a648805 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/data/EmailCampaignData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/data/EmailCampaignData.java
@@ -22,7 +22,7 @@ import org.apache.fineract.infrastructure.core.data.EnumOptionData;
 import org.joda.time.DateTime;
 import org.joda.time.LocalDate;
 
-public class EmailCampaignData {
+public final class EmailCampaignData {
 
     @SuppressWarnings("unused")
     private Long id;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/data/EmailConfigurationData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/data/EmailConfigurationData.java
index 2cbc66c..c77522a 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/data/EmailConfigurationData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/data/EmailConfigurationData.java
@@ -21,7 +21,7 @@ package org.apache.fineract.infrastructure.campaigns.email.data;
 /**
  * Immutable data object representing an Email configuration.
  */
-public class EmailConfigurationData {
+public final class EmailConfigurationData {
 
     @SuppressWarnings("unused")
     private final Long id;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/data/EmailConfigurationValidator.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/data/EmailConfigurationValidator.java
index 1ce8554..75e7355 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/data/EmailConfigurationValidator.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/data/EmailConfigurationValidator.java
@@ -40,7 +40,7 @@ import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Component;
 
 @Component
-public class EmailConfigurationValidator {
+public final class EmailConfigurationValidator {
 
     private final FromJsonHelper fromApiJsonHelper;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/data/EmailData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/data/EmailData.java
index 54c1d63..2a7b7c5 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/data/EmailData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/data/EmailData.java
@@ -26,7 +26,7 @@ import org.joda.time.LocalDate;
 /**
  * Immutable data object representing a SMS message.
  */
-public class EmailData {
+public final class EmailData {
 
     private final Long id;
     private final Long groupId;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/data/EmailMessageWithAttachmentData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/data/EmailMessageWithAttachmentData.java
index 066cb11..a67ebf2 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/data/EmailMessageWithAttachmentData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/data/EmailMessageWithAttachmentData.java
@@ -21,7 +21,7 @@ package org.apache.fineract.infrastructure.campaigns.email.data;
 import java.io.File;
 import java.util.List;
 
-public class EmailMessageWithAttachmentData {
+public final class EmailMessageWithAttachmentData {
 
     private final String to;
     private final String text;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/service/EmailMessageJobEmailServiceImpl.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/service/EmailMessageJobEmailServiceImpl.java
index 9dd8b8d..3b28b45 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/service/EmailMessageJobEmailServiceImpl.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/email/service/EmailMessageJobEmailServiceImpl.java
@@ -35,7 +35,7 @@ import org.springframework.mail.javamail.MimeMessageHelper;
 import org.springframework.stereotype.Service;
 
 @Service
-public class EmailMessageJobEmailServiceImpl implements EmailMessageJobEmailService {
+public final class EmailMessageJobEmailServiceImpl implements EmailMessageJobEmailService {
 
     private final static Logger LOG = LoggerFactory.getLogger(EmailMessageJobEmailServiceImpl.class);
     private EmailConfigurationRepository emailConfigurationRepository;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/sms/data/SmsCampaignData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/sms/data/SmsCampaignData.java
index d22b14c..6d6e31b 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/sms/data/SmsCampaignData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/campaigns/sms/data/SmsCampaignData.java
@@ -24,7 +24,7 @@ import org.joda.time.DateTime;
 import org.joda.time.LocalDate;
 
 @SuppressWarnings("unused")
-public class SmsCampaignData {
+public final class SmsCampaignData {
 
     private Long id;
     private final String campaignName;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/codes/data/CodeData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/codes/data/CodeData.java
index f1088e8..7ebb89c 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/codes/data/CodeData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/codes/data/CodeData.java
@@ -23,7 +23,7 @@ import java.io.Serializable;
 /**
  * Immutable data object representing a code.
  */
-public class CodeData implements Serializable {
+public final class CodeData implements Serializable {
 
     private final Long id;
     @SuppressWarnings("unused")
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/core/data/PaginationParameters.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/core/data/PaginationParameters.java
index f80eaf5..de5915d 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/core/data/PaginationParameters.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/core/data/PaginationParameters.java
@@ -25,7 +25,7 @@ import org.apache.commons.lang3.StringUtils;
  * Immutable data object representing pagination parameter values.
  * </p>
  */
-public class PaginationParameters {
+public final class PaginationParameters {
 
     private final boolean paged;
     private final Integer offset;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/CreditBureauConfigurationData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/CreditBureauConfigurationData.java
index a54ea9b..64ef95c 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/CreditBureauConfigurationData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/CreditBureauConfigurationData.java
@@ -18,7 +18,7 @@
  */
 package org.apache.fineract.infrastructure.creditbureau.data;
 
-public class CreditBureauConfigurationData {
+public final class CreditBureauConfigurationData {
 
     private final long creditBureauConfigurationId;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/CreditBureauData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/CreditBureauData.java
index 31382af..8d24787 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/CreditBureauData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/CreditBureauData.java
@@ -18,7 +18,7 @@
  */
 package org.apache.fineract.infrastructure.creditbureau.data;
 
-public class CreditBureauData {
+public final class CreditBureauData {
 
     private final long creditBureauId;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/CreditBureauLoanProductMappingData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/CreditBureauLoanProductMappingData.java
index 9eb08ac..41394d7 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/CreditBureauLoanProductMappingData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/CreditBureauLoanProductMappingData.java
@@ -18,7 +18,7 @@
  */
 package org.apache.fineract.infrastructure.creditbureau.data;
 
-public class CreditBureauLoanProductMappingData {
+public final class CreditBureauLoanProductMappingData {
 
     private final long creditbureauLoanProductMappingId;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/CreditBureauMasterData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/CreditBureauMasterData.java
index bef1e61..06dcb37 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/CreditBureauMasterData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/CreditBureauMasterData.java
@@ -18,7 +18,7 @@
  */
 package org.apache.fineract.infrastructure.creditbureau.data;
 
-public class CreditBureauMasterData {
+public final class CreditBureauMasterData {
 
     private final long creditBureauId;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/CreditBureauProduct.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/CreditBureauProduct.java
index c2c42dd..1ebdaf5 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/CreditBureauProduct.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/CreditBureauProduct.java
@@ -18,7 +18,7 @@
  */
 package org.apache.fineract.infrastructure.creditbureau.data;
 
-public class CreditBureauProduct {
+public final class CreditBureauProduct {
 
     private final long creditBureauProductId;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/OrganisationCreditBureauData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/OrganisationCreditBureauData.java
index 409bc0e..beb7b07 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/OrganisationCreditBureauData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/creditbureau/data/OrganisationCreditBureauData.java
@@ -18,7 +18,7 @@
  */
 package org.apache.fineract.infrastructure.creditbureau.data;
 
-public class OrganisationCreditBureauData {
+public final class OrganisationCreditBureauData {
 
     private final long organisationCreditBureauId;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/data/DatatableData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/data/DatatableData.java
index a716f43..fb149ce 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/data/DatatableData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/data/DatatableData.java
@@ -24,7 +24,7 @@ import java.util.List;
 /**
  * Immutable data object representing datatable data.
  */
-public class DatatableData implements Serializable {
+public final class DatatableData implements Serializable {
 
     @SuppressWarnings("unused")
     private final String applicationTableName;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/data/ReportData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/data/ReportData.java
index d89d26e..eafb5fa 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/data/ReportData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/data/ReportData.java
@@ -22,7 +22,7 @@ import java.util.ArrayList;
 import java.util.Collection;
 import java.util.List;
 
-final public class ReportData {
+public final class ReportData {
 
     @SuppressWarnings("unused")
     private final Long id;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/data/ReportParameterData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/data/ReportParameterData.java
index 03fb12e..f07f3b4 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/data/ReportParameterData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/data/ReportParameterData.java
@@ -19,7 +19,7 @@
 package org.apache.fineract.infrastructure.dataqueries.data;
 
 /* used to show list of parameters used by a report and also for getting a list of parameters available (the reportParameterName is left null */
-final public class ReportParameterData {
+public final class ReportParameterData {
 
     @SuppressWarnings("unused")
     private final Long id;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/data/ReportParameterJoinData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/data/ReportParameterJoinData.java
index 08caaa8..144da4e 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/data/ReportParameterJoinData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/data/ReportParameterJoinData.java
@@ -18,7 +18,7 @@
  */
 package org.apache.fineract.infrastructure.dataqueries.data;
 
-final public class ReportParameterJoinData {
+public final class ReportParameterJoinData {
 
     private final Long reportId;
     private final String reportName;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/data/ResultsetRowData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/data/ResultsetRowData.java
index 452934f..d60252a 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/data/ResultsetRowData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/dataqueries/data/ResultsetRowData.java
@@ -20,7 +20,7 @@ package org.apache.fineract.infrastructure.dataqueries.data;
 
 import java.util.List;
 
-public class ResultsetRowData {
+public final class ResultsetRowData {
 
     private final List<String> row;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/entityaccess/data/FineractEntityToEntityMappingData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/entityaccess/data/FineractEntityToEntityMappingData.java
index 85d2712..56d7f1c 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/entityaccess/data/FineractEntityToEntityMappingData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/entityaccess/data/FineractEntityToEntityMappingData.java
@@ -21,7 +21,7 @@ package org.apache.fineract.infrastructure.entityaccess.data;
 import java.io.Serializable;
 import java.util.Date;
 
-public class FineractEntityToEntityMappingData implements Serializable {
+public final class FineractEntityToEntityMappingData implements Serializable {
 
     @SuppressWarnings("unused")
     private Long mapId;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/entityaccess/domain/FineractEntityAccessType.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/entityaccess/domain/FineractEntityAccessType.java
index 90c531d..55274f0 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/entityaccess/domain/FineractEntityAccessType.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/entityaccess/domain/FineractEntityAccessType.java
@@ -18,7 +18,7 @@
  */
 package org.apache.fineract.infrastructure.entityaccess.domain;
 
-public class FineractEntityAccessType {
+public final class FineractEntityAccessType {
 
     private String str;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/entityaccess/domain/FineractEntityRelationRepositoryWrapper.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/entityaccess/domain/FineractEntityRelationRepositoryWrapper.java
index b224ec5..5712310 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/entityaccess/domain/FineractEntityRelationRepositoryWrapper.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/entityaccess/domain/FineractEntityRelationRepositoryWrapper.java
@@ -23,7 +23,7 @@ import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Service;
 
 @Service
-public class FineractEntityRelationRepositoryWrapper {
+public final class FineractEntityRelationRepositoryWrapper {
 
     private final FineractEntityRelationRepository fineractEntityRelationRepository;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/entityaccess/domain/FineractEntityType.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/entityaccess/domain/FineractEntityType.java
index 5797ee0..0193a5e 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/entityaccess/domain/FineractEntityType.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/entityaccess/domain/FineractEntityType.java
@@ -18,7 +18,7 @@
  */
 package org.apache.fineract.infrastructure.entityaccess.domain;
 
-public class FineractEntityType {
+public final class FineractEntityType {
 
     private String type;
     private String description;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/gcm/domain/DeviceRegistration.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/gcm/domain/DeviceRegistration.java
index 1b65659..1eb9e51 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/gcm/domain/DeviceRegistration.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/gcm/domain/DeviceRegistration.java
@@ -32,7 +32,7 @@ import org.apache.fineract.portfolio.client.domain.Client;
 
 @Entity
 @Table(name = "client_device_registration")
-public class DeviceRegistration extends AbstractPersistableCustom {
+public final class DeviceRegistration extends AbstractPersistableCustom {
 
     @OneToOne
     @JoinColumn(name = "client_id", nullable = false, unique = true)
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/gcm/domain/DeviceRegistrationData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/gcm/domain/DeviceRegistrationData.java
index a2334fe..9de71ba 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/gcm/domain/DeviceRegistrationData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/gcm/domain/DeviceRegistrationData.java
@@ -21,7 +21,7 @@ package org.apache.fineract.infrastructure.gcm.domain;
 import java.util.Date;
 import org.apache.fineract.portfolio.client.data.ClientData;
 
-public class DeviceRegistrationData {
+public final class DeviceRegistrationData {
 
     public Long id;
     public ClientData clientData;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/hooks/data/Event.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/hooks/data/Event.java
index 5e28c98..bc0a09f 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/hooks/data/Event.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/hooks/data/Event.java
@@ -20,7 +20,7 @@ package org.apache.fineract.infrastructure.hooks.data;
 
 import java.io.Serializable;
 
-public class Event implements Serializable {
+public final class Event implements Serializable {
 
     private final String actionName;
     private final String entityName;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/hooks/data/Field.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/hooks/data/Field.java
index 40ceb36..ac4603f 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/hooks/data/Field.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/hooks/data/Field.java
@@ -21,7 +21,7 @@ package org.apache.fineract.infrastructure.hooks.data;
 import java.io.Serializable;
 
 @SuppressWarnings("unused")
-public class Field implements Serializable {
+public final class Field implements Serializable {
 
     private final String fieldName;
     private final String fieldValue;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/hooks/data/HookData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/hooks/data/HookData.java
index ff0efff..e06e14e 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/hooks/data/HookData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/hooks/data/HookData.java
@@ -22,7 +22,7 @@ import java.io.Serializable;
 import java.util.List;
 import org.joda.time.LocalDate;
 
-public class HookData implements Serializable {
+public final class HookData implements Serializable {
 
     private final Long id;
     private final String name;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/hooks/data/HookTemplateData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/hooks/data/HookTemplateData.java
index ff44ceb..d8ac8e4 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/hooks/data/HookTemplateData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/hooks/data/HookTemplateData.java
@@ -22,7 +22,7 @@ import java.io.Serializable;
 import java.util.List;
 
 @SuppressWarnings("unused")
-public class HookTemplateData implements Serializable {
+public final class HookTemplateData implements Serializable {
 
     private final Long id;
     private final String name;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/reportmailingjob/data/ReportMailingJobConfigurationData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/reportmailingjob/data/ReportMailingJobConfigurationData.java
index ecb3d26..16e0144 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/reportmailingjob/data/ReportMailingJobConfigurationData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/reportmailingjob/data/ReportMailingJobConfigurationData.java
@@ -21,7 +21,7 @@ package org.apache.fineract.infrastructure.reportmailingjob.data;
 /**
  * Immutable data object representing report mailing job configuration data.
  **/
-public class ReportMailingJobConfigurationData {
+public final class ReportMailingJobConfigurationData {
 
     private final int id;
     private final String name;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/reportmailingjob/data/ReportMailingJobData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/reportmailingjob/data/ReportMailingJobData.java
index 2b0a7df..40b58b9 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/reportmailingjob/data/ReportMailingJobData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/reportmailingjob/data/ReportMailingJobData.java
@@ -26,7 +26,7 @@ import org.joda.time.DateTime;
 /**
  * Immutable data object representing report mailing job data.
  **/
-public class ReportMailingJobData {
+public final class ReportMailingJobData {
 
     private final Long id;
     private final String name;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/reportmailingjob/data/ReportMailingJobRunHistoryData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/reportmailingjob/data/ReportMailingJobRunHistoryData.java
index b78e107..d861599 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/reportmailingjob/data/ReportMailingJobRunHistoryData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/reportmailingjob/data/ReportMailingJobRunHistoryData.java
@@ -23,7 +23,7 @@ import org.joda.time.DateTime;
 /**
  * Immutable data object representing report mailing job run history data.
  **/
-public class ReportMailingJobRunHistoryData {
+public final class ReportMailingJobRunHistoryData {
 
     private final Long id;
     private final Long reportMailingJobId;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/security/data/PlatformRequestLog.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/security/data/PlatformRequestLog.java
index b2ccc06..923f034 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/security/data/PlatformRequestLog.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/security/data/PlatformRequestLog.java
@@ -28,7 +28,7 @@ import org.apache.commons.lang3.time.StopWatch;
  * Immutable data object representing platform API request used for
  * logging/debugging.
  */
-public class PlatformRequestLog {
+public final class PlatformRequestLog {
 
     @SuppressWarnings("unused")
     private final long startTime;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/sms/data/SmsData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/sms/data/SmsData.java
index ddc0ad4..dd3b0f6 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/sms/data/SmsData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/sms/data/SmsData.java
@@ -23,7 +23,7 @@ import org.apache.fineract.infrastructure.core.data.EnumOptionData;
 /**
  * Immutable data object representing a SMS message.
  */
-public class SmsData {
+public final class SmsData {
 
     private final Long id;
     private final Long groupId;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/survey/data/SurveyDataTableData.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/survey/data/SurveyDataTableData.java
index ce83b48..d20504f 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/survey/data/SurveyDataTableData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/survey/data/SurveyDataTableData.java
@@ -20,7 +20,7 @@ package org.apache.fineract.infrastructure.survey.data;
 
 import org.apache.fineract.infrastructure.dataqueries.data.DatatableData;
 
-public class SurveyDataTableData {
+public final class SurveyDataTableData {
 
     @SuppressWarnings("unused")
     private final DatatableData datatableData;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/interoperation/data/InteropQuoteResponseData.java b/fineract-provider/src/main/java/org/apache/fineract/interoperation/data/InteropQuoteResponseData.java
index 398ef5e..14148e0 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/interoperation/data/InteropQuoteResponseData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/interoperation/data/InteropQuoteResponseData.java
@@ -24,7 +24,7 @@ import javax.validation.constraints.NotNull;
 import org.apache.fineract.interoperation.domain.InteropActionState;
 import org.joda.time.LocalDateTime;
 
-public class InteropQuoteResponseData extends InteropResponseData {
+public final class InteropQuoteResponseData extends InteropResponseData {
 
     @NotNull
     private final String quoteCode;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/interoperation/data/InteropTransactionRequestResponseData.java b/fineract-provider/src/main/java/org/apache/fineract/interoperation/data/InteropTransactionRequestResponseData.java
index df61695..7802f5f 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/interoperation/data/InteropTransactionRequestResponseData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/interoperation/data/InteropTransactionRequestResponseData.java
@@ -24,7 +24,7 @@ import javax.validation.constraints.NotNull;
 import org.apache.fineract.interoperation.domain.InteropActionState;
 import org.joda.time.LocalDateTime;
 
-public class InteropTransactionRequestResponseData extends InteropResponseData {
+public final class InteropTransactionRequestResponseData extends InteropResponseData {
 
     @NotNull
     private final String requestCode;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/interoperation/data/InteropTransferResponseData.java b/fineract-provider/src/main/java/org/apache/fineract/interoperation/data/InteropTransferResponseData.java
index 42f579f..536b308 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/interoperation/data/InteropTransferResponseData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/interoperation/data/InteropTransferResponseData.java
@@ -26,7 +26,7 @@ import javax.validation.constraints.NotNull;
 import org.apache.fineract.interoperation.domain.InteropActionState;
 import org.joda.time.LocalDateTime;
 
-public class InteropTransferResponseData extends InteropResponseData {
+public final class InteropTransferResponseData extends InteropResponseData {
 
     @NotNull
     private final String transferCode;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/organisation/office/data/OfficeTransactionData.java b/fineract-provider/src/main/java/org/apache/fineract/organisation/office/data/OfficeTransactionData.java
index 9e603e0..3625a6f 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/organisation/office/data/OfficeTransactionData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/organisation/office/data/OfficeTransactionData.java
@@ -26,7 +26,7 @@ import org.joda.time.LocalDate;
 /**
  * Immutable data object for office transactions.
  */
-public class OfficeTransactionData {
+public final class OfficeTransactionData {
 
     @SuppressWarnings("unused")
     private final Long id;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/organisation/provisioning/data/ProvisioningCriteriaData.java b/fineract-provider/src/main/java/org/apache/fineract/organisation/provisioning/data/ProvisioningCriteriaData.java
index ccf86ed..650983d 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/organisation/provisioning/data/ProvisioningCriteriaData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/organisation/provisioning/data/ProvisioningCriteriaData.java
@@ -25,7 +25,7 @@ import org.apache.fineract.accounting.glaccount.data.GLAccountData;
 import org.apache.fineract.portfolio.loanproduct.data.LoanProductData;
 
 @SuppressWarnings("unused")
-public class ProvisioningCriteriaData implements Comparable<ProvisioningCriteriaData>, Serializable {
+public final class ProvisioningCriteriaData implements Comparable<ProvisioningCriteriaData>, Serializable {
 
     private final Long criteriaId;
     private final String criteriaName;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/organisation/staff/data/BulkTransferLoanOfficerData.java b/fineract-provider/src/main/java/org/apache/fineract/organisation/staff/data/BulkTransferLoanOfficerData.java
index b468275..3f065ff 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/organisation/staff/data/BulkTransferLoanOfficerData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/organisation/staff/data/BulkTransferLoanOfficerData.java
@@ -25,7 +25,7 @@ import org.joda.time.LocalDate;
 /**
  * Immutable data object returned for loan-officer bulk transfer screens.
  */
-public class BulkTransferLoanOfficerData {
+public final class BulkTransferLoanOfficerData {
 
     @SuppressWarnings("unused")
     private final Long officeId;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/organisation/staff/data/StaffData.java b/fineract-provider/src/main/java/org/apache/fineract/organisation/staff/data/StaffData.java
index 017efda..defdb61 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/organisation/staff/data/StaffData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/organisation/staff/data/StaffData.java
@@ -26,7 +26,7 @@ import org.joda.time.LocalDate;
 /**
  * Immutable data object representing staff data.
  */
-public class StaffData implements Serializable {
+public final class StaffData implements Serializable {
 
     private final Long id;
     private final String externalId;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/organisation/teller/domain/CashierTxnType.java b/fineract-provider/src/main/java/org/apache/fineract/organisation/teller/domain/CashierTxnType.java
index dfaaac3..e177b52 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/organisation/teller/domain/CashierTxnType.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/organisation/teller/domain/CashierTxnType.java
@@ -20,7 +20,7 @@ package org.apache.fineract.organisation.teller.domain;
 
 import java.io.Serializable;
 
-public class CashierTxnType implements Serializable {
+public final class CashierTxnType implements Serializable {
 
     private Integer id;
     private String value;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/account/data/AccountTransferData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/account/data/AccountTransferData.java
index 06955e3..ec8b6e3 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/account/data/AccountTransferData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/account/data/AccountTransferData.java
@@ -31,7 +31,7 @@ import org.joda.time.LocalDate;
  * Immutable data object representing a savings account.
  */
 @SuppressWarnings("unused")
-public class AccountTransferData implements Serializable {
+public final class AccountTransferData implements Serializable {
 
     private final Long id;
     private final Boolean reversed;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/account/data/StandingInstructionData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/account/data/StandingInstructionData.java
index 778432c..d48d687 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/account/data/StandingInstructionData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/account/data/StandingInstructionData.java
@@ -38,7 +38,7 @@ import org.joda.time.MonthDay;
  * Immutable data object representing a savings account.
  */
 @SuppressWarnings("unused")
-public class StandingInstructionData {
+public final class StandingInstructionData {
 
     private final Long id;
     private final Long accountDetailId;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/address/data/ClientAddressData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/address/data/ClientAddressData.java
index 2e32169..30eb5b4 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/address/data/ClientAddressData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/address/data/ClientAddressData.java
@@ -18,7 +18,7 @@
  */
 package org.apache.fineract.portfolio.address.data;
 
-public class ClientAddressData {
+public final class ClientAddressData {
 
     private final long clientAddressId;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/address/data/FieldConfigurationData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/address/data/FieldConfigurationData.java
index 9f60a4b..8964818 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/address/data/FieldConfigurationData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/address/data/FieldConfigurationData.java
@@ -18,7 +18,7 @@
  */
 package org.apache.fineract.portfolio.address.data;
 
-public class FieldConfigurationData {
+public final class FieldConfigurationData {
 
     private final long fieldConfigurationId;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/calendar/data/CalendarData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/calendar/data/CalendarData.java
index 33bc74c..dc70ba6 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/calendar/data/CalendarData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/calendar/data/CalendarData.java
@@ -36,7 +36,7 @@ import org.joda.time.LocalTime;
 /**
  * Immutable data object representing a Calendar.
  */
-public class CalendarData implements Serializable {
+public final class CalendarData implements Serializable {
 
     private final Long id;
     private final Long calendarInstanceId;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/charge/data/ChargeData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/charge/data/ChargeData.java
index 2b23364..1025c72 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/charge/data/ChargeData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/charge/data/ChargeData.java
@@ -37,7 +37,7 @@ import org.joda.time.MonthDay;
 /**
  * Immutable data object for charge data.
  */
-public class ChargeData implements Comparable<ChargeData>, Serializable {
+public final class ChargeData implements Comparable<ChargeData>, Serializable {
 
     private final Long id;
     private final String name;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/data/ClientChargeData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/data/ClientChargeData.java
index 71a879d..472ec8a 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/data/ClientChargeData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/data/ClientChargeData.java
@@ -26,7 +26,7 @@ import org.apache.fineract.portfolio.charge.data.ChargeData;
 import org.joda.time.LocalDate;
 
 @SuppressWarnings("unused")
-public class ClientChargeData {
+public final class ClientChargeData {
 
     private final Long id;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/data/ClientData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/data/ClientData.java
index 16dab3f..55ce6b9 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/data/ClientData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/data/ClientData.java
@@ -40,7 +40,7 @@ import org.joda.time.LocalDate;
  * Immutable data object representing client data.
  */
 @SuppressWarnings("unused")
-final public class ClientData implements Comparable<ClientData>, Serializable {
+public final class ClientData implements Comparable<ClientData>, Serializable {
 
     private final Long id;
     private final String accountNo;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/data/ClientFamilyMembersData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/data/ClientFamilyMembersData.java
index 6e1f750..eab20ae 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/data/ClientFamilyMembersData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/data/ClientFamilyMembersData.java
@@ -24,7 +24,7 @@ import java.util.Collection;
 import org.apache.fineract.infrastructure.codes.data.CodeValueData;
 import org.joda.time.LocalDate;
 
-public class ClientFamilyMembersData implements Serializable {
+public final class ClientFamilyMembersData implements Serializable {
 
     private final Long id;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/data/ClientTransactionData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/data/ClientTransactionData.java
index be7788d..5aea56a 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/data/ClientTransactionData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/data/ClientTransactionData.java
@@ -27,7 +27,7 @@ import org.apache.fineract.portfolio.paymenttype.data.PaymentTypeData;
 import org.joda.time.LocalDate;
 
 @SuppressWarnings("unused")
-public class ClientTransactionData {
+public final class ClientTransactionData {
 
     private final Long id;
     private final Long officeId;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/serialization/ClientFamilyMemberCommandFromApiJsonDeserializer.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/serialization/ClientFamilyMemberCommandFromApiJsonDeserializer.java
index 2e624cc..16004b5 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/serialization/ClientFamilyMemberCommandFromApiJsonDeserializer.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/client/serialization/ClientFamilyMemberCommandFromApiJsonDeserializer.java
@@ -41,7 +41,7 @@ import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Component;
 
 @Component
-public class ClientFamilyMemberCommandFromApiJsonDeserializer {
+public final class ClientFamilyMemberCommandFromApiJsonDeserializer {
 
     private final FromJsonHelper fromApiJsonHelper;
     private final Set<String> supportedParameters = new HashSet<>(Arrays.asList("id", "clientId", "firstName", "middleName", "lastName",
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/collateral/data/CollateralData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/collateral/data/CollateralData.java
index 28cc06b..3a4d85a 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/collateral/data/CollateralData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/collateral/data/CollateralData.java
@@ -26,7 +26,7 @@ import org.apache.fineract.organisation.monetary.data.CurrencyData;
 /**
  * Immutable data object for Collateral data.
  */
-public class CollateralData {
+public final class CollateralData {
 
     private final Long id;
     private final CodeValueData type;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/IndividualClientData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/IndividualClientData.java
index 5eabd1c..59803c8 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/IndividualClientData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/IndividualClientData.java
@@ -24,7 +24,7 @@ import java.util.Collection;
  * Immutable data object for clients with loans due for disbursement or
  * collection.
  */
-public class IndividualClientData {
+public final class IndividualClientData {
 
     private final Long clientId;
     private final String clientName;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/IndividualCollectionSheetData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/IndividualCollectionSheetData.java
index fca2417..63f1bc5 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/IndividualCollectionSheetData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/IndividualCollectionSheetData.java
@@ -25,7 +25,7 @@ import org.joda.time.LocalDate;
 /**
  * Immutable data object for collection sheet.
  */
-public class IndividualCollectionSheetData {
+public final class IndividualCollectionSheetData {
 
     @SuppressWarnings("unused")
     private final LocalDate dueDate;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/JLGClientData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/JLGClientData.java
index 733c2ac..ad40290 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/JLGClientData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/JLGClientData.java
@@ -25,7 +25,7 @@ import org.apache.fineract.infrastructure.core.data.EnumOptionData;
  * Immutable data object for clients with loans due for disbursement or
  * collection.
  */
-public class JLGClientData {
+public final class JLGClientData {
 
     private final Long clientId;
     private final String clientName;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/JLGCollectionSheetData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/JLGCollectionSheetData.java
index 89bbfc8..7698c45 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/JLGCollectionSheetData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/JLGCollectionSheetData.java
@@ -29,7 +29,7 @@ import org.joda.time.LocalDate;
 /**
  * Immutable data object for join liability group's collection sheet.
  */
-public class JLGCollectionSheetData {
+public final class JLGCollectionSheetData {
 
     private final LocalDate dueDate;
     private final Collection<LoanProductData> loanProducts;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/JLGGroupData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/JLGGroupData.java
index 669641e..05d4a3e 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/JLGGroupData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/JLGGroupData.java
@@ -24,7 +24,7 @@ import java.util.Collection;
  * Immutable data object for groups with clients due for disbursement or
  * collection.
  */
-public class JLGGroupData {
+public final class JLGGroupData {
 
     private final Long groupId;
     private final String groupName;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/SavingsDueData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/SavingsDueData.java
index 88cd981..b15684a 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/SavingsDueData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/collectionsheet/data/SavingsDueData.java
@@ -25,7 +25,7 @@ import org.apache.fineract.organisation.monetary.data.CurrencyData;
  * Immutable data object for representing loan with dues (example: loan is due
  * for disbursement, repayments).
  */
-public class SavingsDueData {
+public final class SavingsDueData {
 
     @SuppressWarnings("unused")
     private final Long savingsId;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/fund/data/FundData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/fund/data/FundData.java
index 2ff5ed4..989a03d 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/fund/data/FundData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/fund/data/FundData.java
@@ -23,7 +23,7 @@ import java.io.Serializable;
 /**
  * Immutable data object to represent fund data.
  */
-public class FundData implements Serializable {
+public final class FundData implements Serializable {
 
     @SuppressWarnings("unused")
     private final Long id;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/group/data/CenterData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/group/data/CenterData.java
index 8353b7c..6666d5a 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/group/data/CenterData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/group/data/CenterData.java
@@ -34,7 +34,7 @@ import org.joda.time.LocalDate;
 /**
  * Immutable data object representing groups.
  */
-public class CenterData implements Serializable {
+public final class CenterData implements Serializable {
 
     private final Long id;
     private String accountNo;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/group/data/StaffCenterData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/group/data/StaffCenterData.java
index 8a84371..f2bcca9 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/group/data/StaffCenterData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/group/data/StaffCenterData.java
@@ -20,7 +20,7 @@ package org.apache.fineract.portfolio.group.data;
 
 import java.util.Collection;
 
-public class StaffCenterData {
+public final class StaffCenterData {
 
     private final Long staffId;
     @SuppressWarnings("unused")
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/interestratechart/data/InterestIncentiveData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/interestratechart/data/InterestIncentiveData.java
index 68cae9a..412a39f 100755
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/interestratechart/data/InterestIncentiveData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/interestratechart/data/InterestIncentiveData.java
@@ -21,7 +21,7 @@ package org.apache.fineract.portfolio.interestratechart.data;
 import java.math.BigDecimal;
 import org.apache.fineract.infrastructure.core.data.EnumOptionData;
 
-public class InterestIncentiveData {
+public final class InterestIncentiveData {
 
     @SuppressWarnings("unused")
     private final Long id;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/interestratechart/data/InterestRateChartData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/interestratechart/data/InterestRateChartData.java
index b0a2910..0363166 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/interestratechart/data/InterestRateChartData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/interestratechart/data/InterestRateChartData.java
@@ -27,7 +27,7 @@ import org.joda.time.LocalDate;
 /**
  * Immutable data object representing a InterestRateChart.
  */
-public class InterestRateChartData {
+public final class InterestRateChartData {
 
     private final Long id;
     private final String name;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/interestratechart/data/InterestRateChartSlabData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/interestratechart/data/InterestRateChartSlabData.java
index 950a64d..cf07e50 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/interestratechart/data/InterestRateChartSlabData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/interestratechart/data/InterestRateChartSlabData.java
@@ -29,7 +29,7 @@ import org.apache.fineract.organisation.monetary.data.CurrencyData;
 /**
  * Immutable data object representing a InterestRateChartSlab.
  */
-public class InterestRateChartSlabData {
+public final class InterestRateChartSlabData {
 
     private final Long id;
     private final String description;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/data/GlimRepaymentTemplate.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/data/GlimRepaymentTemplate.java
index abec34c..7ff7cec 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/data/GlimRepaymentTemplate.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/data/GlimRepaymentTemplate.java
@@ -21,7 +21,7 @@ package org.apache.fineract.portfolio.loanaccount.data;
 
 import java.math.BigDecimal;
 
-public class GlimRepaymentTemplate {
+public final class GlimRepaymentTemplate {
 
     private final BigDecimal glimId;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/data/GroupLoanIndividualMonitoringAccountData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/data/GroupLoanIndividualMonitoringAccountData.java
index 5eabd0a..0231009 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/data/GroupLoanIndividualMonitoringAccountData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/data/GroupLoanIndividualMonitoringAccountData.java
@@ -21,7 +21,7 @@ package org.apache.fineract.portfolio.loanaccount.data;
 
 import java.math.BigDecimal;
 
-public class GroupLoanIndividualMonitoringAccountData {
+public final class GroupLoanIndividualMonitoringAccountData {
 
     private final BigDecimal glimId;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/data/LoanAccountData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/data/LoanAccountData.java
index ab4e4d7..c871450 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/data/LoanAccountData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/data/LoanAccountData.java
@@ -58,7 +58,7 @@ import org.springframework.util.CollectionUtils;
  * Immutable data object representing loan account data.
  */
 @SuppressWarnings("unused")
-public class LoanAccountData {
+public final class LoanAccountData {
 
     // basic loan details
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/GroupLoanIndividualMonitoringAccount.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/GroupLoanIndividualMonitoringAccount.java
index 22d50e0..d33e729 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/GroupLoanIndividualMonitoringAccount.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/GroupLoanIndividualMonitoringAccount.java
@@ -33,7 +33,7 @@ import org.apache.fineract.portfolio.group.domain.Group;
 
 @Entity
 @Table(name = "glim_accounts", uniqueConstraints = { @UniqueConstraint(columnNames = { "account_number" }, name = "FK_glim_id") })
-public class GroupLoanIndividualMonitoringAccount extends AbstractPersistableCustom {
+public final class GroupLoanIndividualMonitoringAccount extends AbstractPersistableCustom {
 
     @ManyToOne
     @JoinColumn(name = "group_id", nullable = false)
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/guarantor/data/GuarantorFundingData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/guarantor/data/GuarantorFundingData.java
index 061f60f..6784007 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/guarantor/data/GuarantorFundingData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/guarantor/data/GuarantorFundingData.java
@@ -23,7 +23,7 @@ import java.util.Collection;
 import org.apache.fineract.infrastructure.core.data.EnumOptionData;
 import org.apache.fineract.portfolio.account.data.PortfolioAccountData;
 
-public class GuarantorFundingData {
+public final class GuarantorFundingData {
 
     @SuppressWarnings("unused")
     private final Long id;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/guarantor/data/GuarantorTransactionData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/guarantor/data/GuarantorTransactionData.java
index 6187096..f8cd576 100755
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/guarantor/data/GuarantorTransactionData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/guarantor/data/GuarantorTransactionData.java
@@ -21,7 +21,7 @@ package org.apache.fineract.portfolio.loanaccount.guarantor.data;
 import org.apache.fineract.portfolio.loanaccount.data.LoanTransactionData;
 import org.apache.fineract.portfolio.savings.data.DepositAccountOnHoldTransactionData;
 
-public class GuarantorTransactionData {
+public final class GuarantorTransactionData {
 
     @SuppressWarnings("unused")
     private final Long id;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/guarantor/data/ObligeeData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/guarantor/data/ObligeeData.java
index 992b6a9..f2e2f2b 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/guarantor/data/ObligeeData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/guarantor/data/ObligeeData.java
@@ -22,7 +22,7 @@ package org.apache.fineract.portfolio.loanaccount.guarantor.data;
 import java.math.BigDecimal;
 
 @SuppressWarnings("unused")
-public class ObligeeData {
+public final class ObligeeData {
 
     private final String firstName;
     private final String lastName;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/loanschedule/data/LoanScheduleDTO.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/loanschedule/data/LoanScheduleDTO.java
index a335471..c3fd4ef 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/loanschedule/data/LoanScheduleDTO.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/loanschedule/data/LoanScheduleDTO.java
@@ -25,7 +25,7 @@ import org.apache.fineract.portfolio.loanaccount.loanschedule.domain.LoanSchedul
 /**
  * Transfer object to return the schedule after generation of schedule
  */
-public class LoanScheduleDTO {
+public final class LoanScheduleDTO {
 
     private final List<LoanRepaymentScheduleInstallment> installments;
     private final LoanScheduleModel loanScheduleModel;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/loanschedule/data/LoanScheduleParams.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/loanschedule/data/LoanScheduleParams.java
index 6954392..92c3755 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/loanschedule/data/LoanScheduleParams.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/loanschedule/data/LoanScheduleParams.java
@@ -31,7 +31,7 @@ import org.apache.fineract.portfolio.loanaccount.domain.transactionprocessor.Loa
 import org.apache.fineract.portfolio.loanaccount.loanschedule.domain.RecalculationDetail;
 import org.joda.time.LocalDate;
 
-public class LoanScheduleParams {
+public final class LoanScheduleParams {
 
     // Actual period Number as per the schedule
     private int periodNumber;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/loanschedule/data/LoanSchedulePeriodData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/loanschedule/data/LoanSchedulePeriodData.java
index 8b13322..5463f8d 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/loanschedule/data/LoanSchedulePeriodData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/loanschedule/data/LoanSchedulePeriodData.java
@@ -26,7 +26,7 @@ import org.joda.time.LocalDate;
  * Immutable data object that represents a period of a loan schedule.
  *
  */
-public class LoanSchedulePeriodData {
+public final class LoanSchedulePeriodData {
 
     private final Integer period;
     private final LocalDate fromDate;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/rescheduleloan/data/LoanRescheduleRequestData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/rescheduleloan/data/LoanRescheduleRequestData.java
index 7cd0d99..18f8680 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/rescheduleloan/data/LoanRescheduleRequestData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/rescheduleloan/data/LoanRescheduleRequestData.java
@@ -26,7 +26,7 @@ import org.joda.time.LocalDate;
 /**
  * Immutable data object representing loan reschedule request data.
  **/
-public class LoanRescheduleRequestData {
+public final class LoanRescheduleRequestData {
 
     private final Long id;
     private final Long loanId;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanproduct/data/LoanProductGuaranteeData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanproduct/data/LoanProductGuaranteeData.java
index cbb7eeb..1c3f608 100755
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanproduct/data/LoanProductGuaranteeData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanproduct/data/LoanProductGuaranteeData.java
@@ -21,7 +21,7 @@ package org.apache.fineract.portfolio.loanproduct.data;
 import java.io.Serializable;
 import java.math.BigDecimal;
 
-public class LoanProductGuaranteeData implements Serializable {
+public final class LoanProductGuaranteeData implements Serializable {
 
     @SuppressWarnings("unused")
     private final Long id;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/meeting/attendance/data/ClientAttendanceData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/meeting/attendance/data/ClientAttendanceData.java
index db2a6e8..4304b8b 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/meeting/attendance/data/ClientAttendanceData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/meeting/attendance/data/ClientAttendanceData.java
@@ -23,7 +23,7 @@ import org.apache.fineract.infrastructure.core.data.EnumOptionData;
 /**
  * Immutable data object representing a ClientAttendance.
  */
-public class ClientAttendanceData {
+public final class ClientAttendanceData {
 
     @SuppressWarnings("unused")
     private final Long id;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/meeting/data/MeetingData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/meeting/data/MeetingData.java
index 333cd8e..7606f4d 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/meeting/data/MeetingData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/meeting/data/MeetingData.java
@@ -29,7 +29,7 @@ import org.joda.time.LocalDate;
 /**
  * Immutable data object representing a Meeting.
  */
-public class MeetingData {
+public final class MeetingData {
 
     private final Long id;
     private final LocalDate meetingDate;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/rate/data/RateData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/rate/data/RateData.java
index 589a0df..8e421c4 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/rate/data/RateData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/rate/data/RateData.java
@@ -26,7 +26,7 @@ import org.apache.fineract.infrastructure.core.data.EnumOptionData;
 /**
  * Bowpi GT Created by Jose on 19/07/2017.
  */
-public class RateData implements Serializable {
+public final class RateData implements Serializable {
 
     private Long id;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/ClosingOfSavingsAccounts.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/ClosingOfSavingsAccounts.java
index 40e7cbf..8a60f69 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/ClosingOfSavingsAccounts.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/ClosingOfSavingsAccounts.java
@@ -20,7 +20,7 @@ package org.apache.fineract.portfolio.savings.data;
 
 import org.joda.time.LocalDate;
 
-public class ClosingOfSavingsAccounts {
+public final class ClosingOfSavingsAccounts {
 
     private final transient Integer rowIndex;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/DepositAccountInterestIncentiveData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/DepositAccountInterestIncentiveData.java
index 2d1557a..03c6e0d 100755
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/DepositAccountInterestIncentiveData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/DepositAccountInterestIncentiveData.java
@@ -22,7 +22,7 @@ import java.math.BigDecimal;
 import org.apache.fineract.infrastructure.core.data.EnumOptionData;
 import org.apache.fineract.portfolio.interestratechart.data.InterestIncentiveData;
 
-public class DepositAccountInterestIncentiveData {
+public final class DepositAccountInterestIncentiveData {
 
     @SuppressWarnings("unused")
     private final Long id;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/DepositAccountInterestRateChartData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/DepositAccountInterestRateChartData.java
index 09b2ebb..468af49 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/DepositAccountInterestRateChartData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/DepositAccountInterestRateChartData.java
@@ -29,7 +29,7 @@ import org.joda.time.LocalDate;
 /**
  * Immutable data object representing a deposit account interest rate chart.
  */
-public class DepositAccountInterestRateChartData {
+public final class DepositAccountInterestRateChartData {
 
     private final Long id;
     private final String name;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/DepositAccountInterestRateChartSlabData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/DepositAccountInterestRateChartSlabData.java
index f2fea6b..42ab9cb 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/DepositAccountInterestRateChartSlabData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/DepositAccountInterestRateChartSlabData.java
@@ -31,7 +31,7 @@ import org.apache.fineract.portfolio.interestratechart.data.InterestRateChartSla
 /**
  * Immutable data object representing deposit accounts Interest rate Slabs.
  */
-public class DepositAccountInterestRateChartSlabData {
+public final class DepositAccountInterestRateChartSlabData {
 
     private final Long id;
     private final String description;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/DepositAccountOnHoldTransactionData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/DepositAccountOnHoldTransactionData.java
index 0520814..d401bb9 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/DepositAccountOnHoldTransactionData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/DepositAccountOnHoldTransactionData.java
@@ -22,7 +22,7 @@ import java.math.BigDecimal;
 import org.apache.fineract.infrastructure.core.data.EnumOptionData;
 import org.joda.time.LocalDate;
 
-public class DepositAccountOnHoldTransactionData {
+public final class DepositAccountOnHoldTransactionData {
 
     @SuppressWarnings("unused")
     private final Long id;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/FixedDepositAccountData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/FixedDepositAccountData.java
index 38a737c..9591517 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/FixedDepositAccountData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/FixedDepositAccountData.java
@@ -36,7 +36,7 @@ import org.joda.time.LocalDate;
 /**
  * Immutable data object representing a Fixed Deposit account.
  */
-public class FixedDepositAccountData extends DepositAccountData {
+public final class FixedDepositAccountData extends DepositAccountData {
 
     private boolean preClosurePenalApplicable;
     private BigDecimal preClosurePenalInterest;
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 6d34a84..b59532b 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
@@ -35,7 +35,7 @@ import org.apache.fineract.portfolio.tax.data.TaxGroupData;
 /**
  * Immutable data object representing a Fixed Deposit product.
  */
-public class FixedDepositProductData extends DepositProductData {
+public final class FixedDepositProductData extends DepositProductData {
 
     // additional fields
     private boolean preClosurePenalApplicable;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/GroupSavingsIndividualMonitoringAccountData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/GroupSavingsIndividualMonitoringAccountData.java
index 4d704f0..c7d861c 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/GroupSavingsIndividualMonitoringAccountData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/GroupSavingsIndividualMonitoringAccountData.java
@@ -21,7 +21,7 @@ package org.apache.fineract.portfolio.savings.data;
 
 import java.math.BigDecimal;
 
-public class GroupSavingsIndividualMonitoringAccountData {
+public final class GroupSavingsIndividualMonitoringAccountData {
 
     private final BigDecimal gsimId;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/RecurringDepositAccountData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/RecurringDepositAccountData.java
index bc73f63..c53ef24 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/RecurringDepositAccountData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/RecurringDepositAccountData.java
@@ -35,7 +35,7 @@ import org.joda.time.LocalDate;
 /**
  * Immutable data object representing a Recurring Deposit account.
  */
-public class RecurringDepositAccountData extends DepositAccountData {
+public final class RecurringDepositAccountData extends DepositAccountData {
 
     // additional fields
     private final boolean preClosurePenalApplicable;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/RecurringDepositProductData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/RecurringDepositProductData.java
index b550c3f..1be8d15 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/RecurringDepositProductData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/RecurringDepositProductData.java
@@ -35,7 +35,7 @@ import org.apache.fineract.portfolio.tax.data.TaxGroupData;
 /**
  * Immutable data object represent a Recurring Deposit product.
  */
-public class RecurringDepositProductData extends DepositProductData {
+public final class RecurringDepositProductData extends DepositProductData {
 
     private boolean preClosurePenalApplicable;
     private BigDecimal preClosurePenalInterest;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsAccountAnnualFeeData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsAccountAnnualFeeData.java
index a83c757..33b8836 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsAccountAnnualFeeData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsAccountAnnualFeeData.java
@@ -20,7 +20,7 @@ package org.apache.fineract.portfolio.savings.data;
 
 import org.joda.time.LocalDate;
 
-public class SavingsAccountAnnualFeeData {
+public final class SavingsAccountAnnualFeeData {
 
     private final Long id;
     private final Long accountId;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsAccountData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsAccountData.java
index 77c6222..b2c2806 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsAccountData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsAccountData.java
@@ -38,7 +38,7 @@ import org.joda.time.LocalDate;
 /**
  * Immutable data object representing a savings account.
  */
-public class SavingsAccountData implements Serializable {
+public final class SavingsAccountData implements Serializable {
 
     private final Long id;
     private final String accountNo;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsAccountTransactionData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsAccountTransactionData.java
index a8681ce..da1ffd8 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsAccountTransactionData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsAccountTransactionData.java
@@ -35,7 +35,7 @@ import org.joda.time.LocalDate;
  * Immutable data object representing a savings account transaction.
  */
 @SuppressWarnings("unused")
-public class SavingsAccountTransactionData implements Serializable {
+public final class SavingsAccountTransactionData implements Serializable {
 
     private final Long id;
     private final SavingsAccountTransactionEnumData transactionType;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsActivation.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsActivation.java
index 48dda82..fdfe567 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsActivation.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsActivation.java
@@ -20,7 +20,7 @@ package org.apache.fineract.portfolio.savings.data;
 
 import org.joda.time.LocalDate;
 
-public class SavingsActivation {
+public final class SavingsActivation {
 
     private final transient Integer rowIndex;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsApproval.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsApproval.java
index 932170e..e949526 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsApproval.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsApproval.java
@@ -20,7 +20,7 @@ package org.apache.fineract.portfolio.savings.data;
 
 import org.joda.time.LocalDate;
 
-public class SavingsApproval {
+public final class SavingsApproval {
 
     private final transient Integer rowIndex;
 
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsProductData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsProductData.java
index 8adc4c9..5937f33 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsProductData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/data/SavingsProductData.java
@@ -36,7 +36,7 @@ import org.apache.fineract.portfolio.tax.data.TaxGroupData;
 /**
  * Immutable data object represent a savings product.
  */
-public class SavingsProductData implements Serializable {
+public final class SavingsProductData implements Serializable {
 
     private final Long id;
     private final String name;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/GroupSavingsIndividualMonitoring.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/GroupSavingsIndividualMonitoring.java
index 8001edd..202a63f 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/GroupSavingsIndividualMonitoring.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/GroupSavingsIndividualMonitoring.java
@@ -33,7 +33,7 @@ import org.apache.fineract.portfolio.group.domain.Group;
 
 @Entity
 @Table(name = "gsim_accounts", uniqueConstraints = { @UniqueConstraint(columnNames = { "account_number" }, name = "gsim_id") })
-public class GroupSavingsIndividualMonitoring extends AbstractPersistableCustom {
+public final class GroupSavingsIndividualMonitoring extends AbstractPersistableCustom {
 
     @ManyToOne
     @JoinColumn(name = "group_id", nullable = false)
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/AnnualCompoundingPeriod.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/AnnualCompoundingPeriod.java
index 3fd24db..a3426b8 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/AnnualCompoundingPeriod.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/AnnualCompoundingPeriod.java
@@ -28,7 +28,7 @@ import org.apache.fineract.portfolio.savings.SavingsCompoundingInterestPeriodTyp
 import org.apache.fineract.portfolio.savings.SavingsInterestCalculationType;
 import org.joda.time.LocalDate;
 
-public class AnnualCompoundingPeriod implements CompoundingPeriod {
+public final class AnnualCompoundingPeriod implements CompoundingPeriod {
 
     @SuppressWarnings("unused")
     private final LocalDateInterval periodInterval;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/BiAnnualCompoundingPeriod.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/BiAnnualCompoundingPeriod.java
index 49e15df..09f2129 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/BiAnnualCompoundingPeriod.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/BiAnnualCompoundingPeriod.java
@@ -28,7 +28,7 @@ import org.apache.fineract.portfolio.savings.SavingsCompoundingInterestPeriodTyp
 import org.apache.fineract.portfolio.savings.SavingsInterestCalculationType;
 import org.joda.time.LocalDate;
 
-public class BiAnnualCompoundingPeriod implements CompoundingPeriod {
+public final class BiAnnualCompoundingPeriod implements CompoundingPeriod {
 
     @SuppressWarnings("unused")
     private final LocalDateInterval periodInterval;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/DailyCompoundingPeriod.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/DailyCompoundingPeriod.java
index 515ae11..507d126 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/DailyCompoundingPeriod.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/DailyCompoundingPeriod.java
@@ -27,7 +27,7 @@ import org.apache.fineract.portfolio.savings.SavingsCompoundingInterestPeriodTyp
 import org.apache.fineract.portfolio.savings.SavingsInterestCalculationType;
 import org.joda.time.LocalDate;
 
-public class DailyCompoundingPeriod implements CompoundingPeriod {
+public final class DailyCompoundingPeriod implements CompoundingPeriod {
 
     private final LocalDateInterval periodInterval;
     private final List<EndOfDayBalance> endOfDayBalances;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/MonthlyCompoundingPeriod.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/MonthlyCompoundingPeriod.java
index 4307165..6475ca1 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/MonthlyCompoundingPeriod.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/MonthlyCompoundingPeriod.java
@@ -28,7 +28,7 @@ import org.apache.fineract.portfolio.savings.SavingsCompoundingInterestPeriodTyp
 import org.apache.fineract.portfolio.savings.SavingsInterestCalculationType;
 import org.joda.time.LocalDate;
 
-public class MonthlyCompoundingPeriod implements CompoundingPeriod {
+public final class MonthlyCompoundingPeriod implements CompoundingPeriod {
 
     @SuppressWarnings("unused")
     private final LocalDateInterval periodInterval;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/PostingPeriod.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/PostingPeriod.java
index 320d977..f0f057e 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/PostingPeriod.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/PostingPeriod.java
@@ -33,7 +33,7 @@ import org.apache.fineract.portfolio.savings.domain.SavingsAccountTransaction;
 import org.joda.time.DateTime;
 import org.joda.time.LocalDate;
 
-public class PostingPeriod {
+public final class PostingPeriod {
 
     private final LocalDateInterval periodInterval;
     private final MonetaryCurrency currency;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/QuarterlyCompoundingPeriod.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/QuarterlyCompoundingPeriod.java
index bf54af2..1c316e0 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/QuarterlyCompoundingPeriod.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/interest/QuarterlyCompoundingPeriod.java
@@ -28,7 +28,7 @@ import org.apache.fineract.portfolio.savings.SavingsCompoundingInterestPeriodTyp
 import org.apache.fineract.portfolio.savings.SavingsInterestCalculationType;
 import org.joda.time.LocalDate;
 
-public class QuarterlyCompoundingPeriod implements CompoundingPeriod {
+public final class QuarterlyCompoundingPeriod implements CompoundingPeriod {
 
     private final LocalDateInterval periodInterval;
     private final List<EndOfDayBalance> endOfDayBalances;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/search/data/AdHocSearchQueryData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/search/data/AdHocSearchQueryData.java
index bae99cb..5c3c1fa 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/search/data/AdHocSearchQueryData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/search/data/AdHocSearchQueryData.java
@@ -23,7 +23,7 @@ import java.util.Collection;
 import org.apache.fineract.organisation.office.data.OfficeData;
 import org.apache.fineract.portfolio.loanproduct.data.LoanProductData;
 
-public class AdHocSearchQueryData {
+public final class AdHocSearchQueryData {
 
     @SuppressWarnings("unused")
     private final String officeName;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/self/pockets/data/PocketAccountMappingData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/self/pockets/data/PocketAccountMappingData.java
index 84aac98..99f040c 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/self/pockets/data/PocketAccountMappingData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/self/pockets/data/PocketAccountMappingData.java
@@ -23,7 +23,7 @@ import java.util.Collection;
 import org.apache.fineract.portfolio.self.pockets.domain.PocketAccountMapping;
 
 @SuppressWarnings("unused")
-public class PocketAccountMappingData {
+public final class PocketAccountMappingData {
 
     private final Collection<PocketAccountMapping> loanAccounts;
     private final Collection<PocketAccountMapping> savingsAccounts;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/shareproducts/data/ShareProductData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/shareproducts/data/ShareProductData.java
index 1db5007..0a4038c 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/shareproducts/data/ShareProductData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/shareproducts/data/ShareProductData.java
@@ -31,7 +31,7 @@ import org.apache.fineract.organisation.monetary.data.CurrencyData;
 import org.apache.fineract.portfolio.charge.data.ChargeData;
 import org.apache.fineract.portfolio.products.data.ProductData;
 
-public class ShareProductData implements ProductData {
+public final class ShareProductData implements ProductData {
 
     private final Long id;
     private final String name;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/tax/data/TaxComponentData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/tax/data/TaxComponentData.java
index 9ef50e9..0fa29be 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/tax/data/TaxComponentData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/tax/data/TaxComponentData.java
@@ -27,7 +27,7 @@ import org.apache.fineract.accounting.glaccount.data.GLAccountData;
 import org.apache.fineract.infrastructure.core.data.EnumOptionData;
 import org.joda.time.LocalDate;
 
-public class TaxComponentData implements Serializable {
+public final class TaxComponentData implements Serializable {
 
     @SuppressWarnings("unused")
     private final Long id;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/tax/data/TaxGroupData.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/tax/data/TaxGroupData.java
index 1d618a4..d42f713 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/tax/data/TaxGroupData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/tax/data/TaxGroupData.java
@@ -21,7 +21,7 @@ package org.apache.fineract.portfolio.tax.data;
 import java.io.Serializable;
 import java.util.Collection;
 
-public class TaxGroupData implements Serializable {
+public final class TaxGroupData implements Serializable {
 
     private final Long id;
     private final String name;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/spm/util/LookupTableMapper.java b/fineract-provider/src/main/java/org/apache/fineract/spm/util/LookupTableMapper.java
index af33132..5461da0 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/spm/util/LookupTableMapper.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/spm/util/LookupTableMapper.java
@@ -27,7 +27,7 @@ import org.apache.fineract.spm.data.LookupTableEntry;
 import org.apache.fineract.spm.domain.LookupTable;
 import org.apache.fineract.spm.domain.Survey;
 
-public class LookupTableMapper {
+public final class LookupTableMapper {
 
     private LookupTableMapper() {
         super();
diff --git a/fineract-provider/src/main/java/org/apache/fineract/spm/util/ScorecardMapper.java b/fineract-provider/src/main/java/org/apache/fineract/spm/util/ScorecardMapper.java
index dbf1a4b..60f13d2 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/spm/util/ScorecardMapper.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/spm/util/ScorecardMapper.java
@@ -31,7 +31,7 @@ import org.apache.fineract.spm.domain.Survey;
 import org.apache.fineract.spm.exception.SurveyResponseNotAvailableException;
 import org.apache.fineract.useradministration.domain.AppUser;
 
-public class ScorecardMapper {
+public final class ScorecardMapper {
 
     private ScorecardMapper() {
         super();
diff --git a/fineract-provider/src/main/java/org/apache/fineract/spm/util/SurveyMapper.java b/fineract-provider/src/main/java/org/apache/fineract/spm/util/SurveyMapper.java
index c7dd8ce..d12055f 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/spm/util/SurveyMapper.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/spm/util/SurveyMapper.java
@@ -29,7 +29,7 @@ import org.apache.fineract.spm.domain.Question;
 import org.apache.fineract.spm.domain.Response;
 import org.apache.fineract.spm.domain.Survey;
 
-public class SurveyMapper {
+public final class SurveyMapper {
 
     private SurveyMapper() {
         super();
diff --git a/fineract-provider/src/main/java/org/apache/fineract/template/data/TemplateData.java b/fineract-provider/src/main/java/org/apache/fineract/template/data/TemplateData.java
index 4ce312e..b083069 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/template/data/TemplateData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/template/data/TemplateData.java
@@ -26,7 +26,7 @@ import org.apache.fineract.template.domain.Template;
 import org.apache.fineract.template.domain.TemplateEntity;
 import org.apache.fineract.template.domain.TemplateType;
 
-public class TemplateData {
+public final class TemplateData {
 
     @SuppressWarnings("unused")
     private final List<Map<String, Object>> entities;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/useradministration/data/AppUserData.java b/fineract-provider/src/main/java/org/apache/fineract/useradministration/data/AppUserData.java
index 6fae85d..6538af4 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/useradministration/data/AppUserData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/useradministration/data/AppUserData.java
@@ -28,7 +28,7 @@ import org.apache.fineract.portfolio.client.data.ClientData;
 /**
  * Immutable data object for application user data.
  */
-public class AppUserData {
+public final class AppUserData {
 
     private final Long id;
     private final String username;
diff --git a/fineract-provider/src/main/java/org/apache/fineract/useradministration/data/PermissionData.java b/fineract-provider/src/main/java/org/apache/fineract/useradministration/data/PermissionData.java
index 8a0d38b..d1cbc34 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/useradministration/data/PermissionData.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/useradministration/data/PermissionData.java
@@ -21,7 +21,7 @@ package org.apache.fineract.useradministration.data;
 /**
  * Immutable representation of permissions
  */
-public class PermissionData {
+public final class PermissionData {
 
     @SuppressWarnings("unused")
     private final String grouping;


Mime
View raw message