From 7031ed40b4503d6bc744f9c9841bedf486c5bf5a Mon Sep 17 00:00:00 2001
From: Michal Dudzinski <mdudzinski@soldevelo.com>
Date: Sat, 17 Nov 2012 15:55:12 +0100
Subject: [PATCH] MIFOSX-73: track loan officer reassignment history only when
 loan is approved.

---
 .../main/java/org/mifosng/platform/loan/domain/Loan.java    | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/mifosng-provider/src/main/java/org/mifosng/platform/loan/domain/Loan.java b/mifosng-provider/src/main/java/org/mifosng/platform/loan/domain/Loan.java
index 6a89777f7..2b807680d 100644
--- a/mifosng-provider/src/main/java/org/mifosng/platform/loan/domain/Loan.java
+++ b/mifosng-provider/src/main/java/org/mifosng/platform/loan/domain/Loan.java
@@ -1612,8 +1612,10 @@ public class Loan extends AbstractAuditableCustom<AppUser, Long> {
             }
 
         	this.loanofficer = newLoanOfficer;
-        	final LoanOfficerAssignmentHistory loanOfficerAssignmentHistory = LoanOfficerAssignmentHistory.createNew(this, this.loanofficer, assignmentDate);
-            this.loanOfficerHistory.add(loanOfficerAssignmentHistory);
+            if (this.isNotSubmittedAndPendingApproval()){
+                final LoanOfficerAssignmentHistory loanOfficerAssignmentHistory = LoanOfficerAssignmentHistory.createNew(this, this.loanofficer, assignmentDate);
+                this.loanOfficerHistory.add(loanOfficerAssignmentHistory);
+            }
         }
 	}