Fixed: opening split dialog but without adding splits causes transfer account to be ignored when saving a transaction

Author: codinguser

app/src/main/java/org/gnucash/android/ui/transaction/TransactionFormFragment.java

@@ -689,6 +689,17 @@ private void saveNewTransaction() {
 		Currency currency = Currency.getInstance(mTransactionsDbAdapter.getAccountCurrencyCode(mAccountUID));
 		Money amount 	= new Money(amountBigd, currency).absolute();
 
+        if (mSplitsList.size() == 1){ //means split editor was opened but no split was added
+            String transferAcctUID;
+            if (mUseDoubleEntry) {
+                long transferAcctId = mDoubleAccountSpinner.getSelectedItemId();
+                transferAcctUID = mAccountsDbAdapter.getUID(transferAcctId);
+            } else {
+                transferAcctUID = mAccountsDbAdapter.getOrCreateImbalanceAccountUID(currency);
+            }
+            mSplitsList.add(mSplitsList.get(0).createPair(transferAcctUID));
+        }
+
         //capture any edits which were done directly (not using split editor)
         if (mSplitsList.size() == 2 && mSplitsList.get(0).isPairOf(mSplitsList.get(1))) {
             //if it is a simple transfer where the editor was not used, then respect the button

app/src/main/java/org/gnucash/android/ui/transaction/dialog/SplitEditorDialogFragment.java

@@ -116,7 +116,7 @@ public void onActivityCreated(Bundle savedInstanceState) {
 
         getDialog().setTitle(R.string.title_transaction_splits);
 
-        mSplitItemViewList = new ArrayList<View>();
+        mSplitItemViewList = new ArrayList<>();
         mSplitsDbAdapter = SplitsDbAdapter.getInstance();
 
         //we are editing splits for a new transaction.