josehernandezfintecheandomx commented on code in PR #2783:
URL: https://github.com/apache/fineract/pull/2783#discussion_r1044527454
##########
fineract-provider/src/main/java/org/apache/fineract/portfolio/client/service/ClientReadPlatformServiceImpl.java:
##########
@@ -370,77 +305,141 @@ public Collection<ClientData>
retrieveActiveClientMembersOfGroup(final Long grou
hierarchySearchString, groupId,
ClientStatus.ACTIVE.getValue());
}
- private static final class ClientMembersOfGroupMapper implements
RowMapper<ClientData> {
+ private String buildSqlStringFromClientCriteria(String schemaSql, final
SearchParameters searchParameters, List<Object> paramList) {
+
+ String sqlSearch = searchParameters.getSqlSearch();
+ final Long officeId = searchParameters.getOfficeId();
+ final String externalId = searchParameters.getExternalId();
+ final String displayName = searchParameters.getName();
+ final String firstname = searchParameters.getFirstname();
+ final String lastname = searchParameters.getLastname();
+ final String status = searchParameters.getStatus();
+
+ String extraCriteria = "";
+ if (sqlSearch != null) {
+ sqlSearch = sqlSearch.replaceAll(" display_name ", "
c.display_name ");
+ sqlSearch = sqlSearch.replaceAll("display_name ", "c.display_name
");
+ extraCriteria = " and (" + sqlSearch + ")";
+ this.columnValidator.validateSqlInjection(schemaSql, sqlSearch);
+ }
+
+ if (officeId != null) {
+ extraCriteria += " and c.office_id = ? ";
+ paramList.add(officeId);
+ }
+
+ if (externalId != null) {
+ paramList.add(externalId);
+ extraCriteria += " and c.external_id like ? ";
+ }
+
+ if (displayName != null) {
+ // extraCriteria += " and concatcoalesce(c.firstname, ''),
+ // if(c.firstname > '',' ', '') , coalesce(c.lastname, '')) like "
+ paramList.add("%" + displayName + "%");
+ extraCriteria += " and c.display_name like ? ";
+ }
+
+ if (status != null) {
+ ClientStatus clientStatus = ClientStatus.fromString(status);
+ extraCriteria += " and c.status_enum = " +
clientStatus.getValue().toString() + " ";
+ }
+
+ if (firstname != null) {
+ paramList.add(firstname);
+ extraCriteria += " and c.firstname like ? ";
+ }
+
+ if (lastname != null) {
+ paramList.add(lastname);
+ extraCriteria += " and c.lastname like ? ";
+ }
+
+ if (searchParameters.isScopedByOfficeHierarchy()) {
+ paramList.add(searchParameters.getHierarchy() + "%");
+ extraCriteria += " and o.hierarchy like ? ";
+ }
+
+ if (searchParameters.isOrphansOnly()) {
+ extraCriteria += " and c.id NOT IN (select client_id from
m_group_client) ";
+ }
+
+ if (StringUtils.isNotBlank(extraCriteria)) {
+ extraCriteria = extraCriteria.substring(4);
+ }
+ return extraCriteria;
+ }
+
+ private static final class ClientToDataMapper implements
RowMapper<ClientData> {
private final String schema;
- ClientMembersOfGroupMapper() {
- final StringBuilder sqlBuilder = new StringBuilder(200);
+ ClientToDataMapper() {
+ final StringBuilder builder = new StringBuilder(400);
- sqlBuilder.append(
+ builder.append(
"c.id as id, c.account_no as accountNo, c.external_id as
externalId, c.status_enum as statusEnum,c.sub_status as subStatus, ");
- sqlBuilder.append(
+ builder.append(
"cvSubStatus.code_value as
subStatusValue,cvSubStatus.code_description as subStatusDesc,c.office_id as
officeId, o.name as officeName, ");
- sqlBuilder.append("c.transfer_to_office_id as transferToOfficeId,
transferToOffice.name as transferToOfficeName, ");
- sqlBuilder.append("c.firstname as firstname, c.middlename as
middlename, c.lastname as lastname, ");
- sqlBuilder.append("c.fullname as fullname, c.display_name as
displayName, ");
- sqlBuilder.append("c.mobile_no as mobileNo, ");
- sqlBuilder.append("c.is_staff as isStaff, ");
- sqlBuilder.append("c.email_address as emailAddress, ");
- sqlBuilder.append("c.date_of_birth as dateOfBirth, ");
- sqlBuilder.append("c.gender_cv_id as genderId, ");
- sqlBuilder.append("cv.code_value as genderValue, ");
- sqlBuilder.append("c.client_type_cv_id as clienttypeId, ");
- sqlBuilder.append("cvclienttype.code_value as clienttypeValue, ");
- sqlBuilder.append("c.client_classification_cv_id as
classificationId, ");
- sqlBuilder.append("cvclassification.code_value as
classificationValue, ");
- sqlBuilder.append("c.legal_form_enum as legalFormEnum, ");
- sqlBuilder.append("c.activation_date as activationDate, c.image_id
as imageId, ");
- sqlBuilder.append("c.staff_id as staffId, s.display_name as
staffName,");
- sqlBuilder.append("c.default_savings_product as savingsProductId,
sp.name as savingsProductName, ");
- sqlBuilder.append("c.default_savings_account as savingsAccountId,
");
-
- sqlBuilder.append("c.submittedon_date as submittedOnDate, ");
- sqlBuilder.append("sbu.username as submittedByUsername, ");
- sqlBuilder.append("sbu.firstname as submittedByFirstname, ");
- sqlBuilder.append("sbu.lastname as submittedByLastname, ");
+ builder.append("c.transfer_to_office_id as transferToOfficeId,
transferToOffice.name as transferToOfficeName, ");
+ builder.append("c.firstname as firstname, c.middlename as
middlename, c.lastname as lastname, ");
+ builder.append("c.fullname as fullname, c.display_name as
displayName, ");
+ builder.append("c.mobile_no as mobileNo, ");
+ builder.append("c.is_staff as isStaff, ");
+ builder.append("c.email_address as emailAddress, ");
+ builder.append("c.date_of_birth as dateOfBirth, ");
+ builder.append("c.gender_cv_id as genderId, ");
+ builder.append("cv.code_value as genderValue, ");
+ builder.append("c.client_type_cv_id as clienttypeId, ");
+ builder.append("cvclienttype.code_value as clienttypeValue, ");
+ builder.append("c.client_classification_cv_id as classificationId,
");
+ builder.append("cvclassification.code_value as
classificationValue, ");
+ builder.append("c.legal_form_enum as legalFormEnum, ");
- sqlBuilder.append("c.closedon_date as closedOnDate, ");
- sqlBuilder.append("clu.username as closedByUsername, ");
- sqlBuilder.append("clu.firstname as closedByFirstname, ");
- sqlBuilder.append("clu.lastname as closedByLastname, ");
+ builder.append("c.submittedon_date as submittedOnDate, ");
+ builder.append("sbu.username as submittedByUsername, ");
+ builder.append("sbu.firstname as submittedByFirstname, ");
+ builder.append("sbu.lastname as submittedByLastname, ");
- sqlBuilder.append("acu.username as activatedByUsername, ");
- sqlBuilder.append("acu.firstname as activatedByFirstname, ");
- sqlBuilder.append("acu.lastname as activatedByLastname, ");
+ builder.append("c.closedon_date as closedOnDate, ");
+ builder.append("clu.username as closedByUsername, ");
+ builder.append("clu.firstname as closedByFirstname, ");
+ builder.append("clu.lastname as closedByLastname, ");
- sqlBuilder.append("cnp.constitution_cv_id as constitutionId, ");
- sqlBuilder.append("cvConstitution.code_value as constitutionValue,
");
- sqlBuilder.append("cnp.incorp_no as incorpNo, ");
- sqlBuilder.append("cnp.incorp_validity_till as incorpValidityTill,
");
- sqlBuilder.append("cnp.main_business_line_cv_id as
mainBusinessLineId, ");
- sqlBuilder.append("cvMainBusinessLine.code_value as
mainBusinessLineValue, ");
- sqlBuilder.append("cnp.remarks as remarks ");
+ // builder.append("c.submittedon as submittedOnDate, ");
Review Comment:
This commented line is part of the original code, I mean I did not commented
out this line.
<img width="919" alt="Screenshot 2022-12-09 at 8 49 00"
src="https://user-images.githubusercontent.com/44206706/206728300-c128ee07-b2c2-4767-acb7-9a9d674b14a4.png";>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@fineract.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
![https://user-images.githubusercontent.com/44206706/206728300-c128ee07-b2c2-4767-acb7-9a9d674b14a4.png"](https://user-images.githubusercontent.com/44206706/206728300-c128ee07-b2c2-4767-acb7-9a9d674b14a4.png"){kind=link}