fix openMF#778:transfer within same account

AkshGautam · Mar 22, 2018 · b479be4 · b479be4
1 parent 70830ec
commit b479be4
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/app/src/main/java/org/mifos/mobilebanking/ui/fragments/SavingsMakeTransferFragment.java b/app/src/main/java/org/mifos/mobilebanking/ui/fragments/SavingsMakeTransferFragment.java
@@ -107,7 +107,7 @@ public class SavingsMakeTransferFragment extends BaseFragment implements
     private String transferDate;
     private AccountOption toAccountOption, fromAccountOption;
     private AccountOptionsTemplate accountOptionsTemplate;
-    private String transferType;
+    private String transferType, payTo, payFrom;
     private long accountId;
 
     /**
@@ -295,9 +295,11 @@ public void onItemSelected(AdapterView<?> parent, View view, int position, long
         switch (parent.getId()) {
             case R.id.sp_pay_to:
                 toAccountOption = accountOptionsTemplate.getToAccountOptions().get(position);
+                payTo = toAccountOption.getAccountNo();
                 break;
             case R.id.sp_pay_from:
                 fromAccountOption = accountOptionsTemplate.getFromAccountOptions().get(position);
+                payFrom = fromAccountOption.getAccountNo();
                 break;
         }
 
@@ -353,7 +355,7 @@ public void payToSelected() {
      */
     @OnClick(R.id.btn_pay_from)
     public void payFromSelected() {
-        if (spPayTo.getSelectedItem().toString().equals(spPayFrom.getSelectedItem().toString())) {
+        if (payTo.equals(payFrom) && payTo!=null) {
             showToaster(getString(R.string.error_same_account_transfer));
             return;
         }