fixed MIFOSX-1655 issue with updating a loan with a charge

This commit is contained in:
andrew-dzak
2014-11-16 20:54:55 +01:00
parent 61a096f33c
commit e1313fd254
4 changed files with 20 additions and 3 deletions
@@ -1,3 +1,8 @@
/**
* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this file,
* You can obtain one at http://mozilla.org/MPL/2.0/.
*/
package org.mifosplatform.portfolio.common.domain; package org.mifosplatform.portfolio.common.domain;
import org.joda.time.DateTimeConstants; import org.joda.time.DateTimeConstants;
@@ -4,7 +4,11 @@
* You can obtain one at http://mozilla.org/MPL/2.0/. * You can obtain one at http://mozilla.org/MPL/2.0/.
*/ */
package org.mifosplatform.portfolio.common.domain; package org.mifosplatform.portfolio.common.domain;
/**
* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this file,
* You can obtain one at http://mozilla.org/MPL/2.0/.
*/
import java.util.ArrayList; import java.util.ArrayList;
import java.util.List; import java.util.List;
@@ -1,3 +1,8 @@
/**
* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this file,
* You can obtain one at http://mozilla.org/MPL/2.0/.
*/
package org.mifosplatform.portfolio.common.domain; package org.mifosplatform.portfolio.common.domain;
public enum NthDayType { public enum NthDayType {
@@ -389,10 +389,13 @@ public class LoanApplicationWritePlatformServiceJpaRepositoryImpl implements Loa
* If there are any charges already present, which are now not * If there are any charges already present, which are now not
* passed in as a part of the request, deem the charges as modified * passed in as a part of the request, deem the charges as modified
**/ **/
if (!possiblyModifedLoanCharges.containsAll(existingCharges)) { if(!possiblyModifedLoanCharges.isEmpty()){
isChargeModified = true; if (!possiblyModifedLoanCharges.containsAll(existingCharges)) {
isChargeModified = true;
}
} }
/** /**
* If any new charges are added or values of existing charges are * If any new charges are added or values of existing charges are
* modified * modified