From 5647ffb966a9a636833663784be6630613359349 Mon Sep 17 00:00:00 2001
From: Emmanuel Nnaa <emmanuelnnaa@musoni.eu>
Date: Tue, 2 Aug 2016 15:46:47 +0200
Subject: [PATCH] commit for FINERACT-70 (Rename deleted standing instruction
 by appending '_deleted_' string and the id)

---
 .../domain/AccountTransferStandingInstruction.java        | 8 ++++++++
 .../StandingInstructionWritePlatformServiceImpl.java      | 4 +++-
 2 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/account/domain/AccountTransferStandingInstruction.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/account/domain/AccountTransferStandingInstruction.java
index a9b28580a..4249338ef 100755
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/account/domain/AccountTransferStandingInstruction.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/account/domain/AccountTransferStandingInstruction.java
@@ -291,4 +291,12 @@ public class AccountTransferStandingInstruction extends AbstractPersistable<Long
     public void updateStatus(Integer status){
         this.status = status;
     }
+    
+    /** 
+     * delete the standing instruction by setting the status to 3 and appending "_deleted_" and the id to the name
+     **/
+     public void delete() {
+         this.status = StandingInstructionStatus.DELETED.getValue();
+         this.name = this.name + "_deleted_" + this.getId();
+     }
 }
diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/account/service/StandingInstructionWritePlatformServiceImpl.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/account/service/StandingInstructionWritePlatformServiceImpl.java
index 9d6a48b7d..6cb3c4a85 100755
--- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/account/service/StandingInstructionWritePlatformServiceImpl.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/account/service/StandingInstructionWritePlatformServiceImpl.java
@@ -178,7 +178,9 @@ public class StandingInstructionWritePlatformServiceImpl implements StandingInst
     @Override
     public CommandProcessingResult delete(final Long id) {
         AccountTransferStandingInstruction standingInstructionsForUpdate = this.standingInstructionRepository.findOne(id);
-        standingInstructionsForUpdate.updateStatus(StandingInstructionStatus.DELETED.getValue());
+        // update the "deleted" and "name" properties of the standing instruction
+        standingInstructionsForUpdate.delete();
+        
         final Map<String, Object> actualChanges = new HashMap<>();
         actualChanges.put(statusParamName, StandingInstructionStatus.DELETED.getValue());
         return new CommandProcessingResultBuilder() //