Fixed: Cannot edit transactions when in single-entry mode

Fixed: Transaction type button sometimes hidden in single-entry mode
Fixed: Creating transactions from templates always saves the template amount, even if edited

Author: codinguser

app/src/org/gnucash/android/db/SplitsDbAdapter.java

@@ -286,6 +286,9 @@ public String getUID(long id){
      * @return Cursor to splits
      */
     public Cursor fetchSplitsForTransaction(String transactionUID){
+        if (transactionUID == null)
+            throw new IllegalArgumentException("Transaction UID cannot be null");
+
         Log.v(TAG, "Fetching all splits for transaction UID " + transactionUID);
         return mDb.query(SplitEntry.TABLE_NAME,
                 null, SplitEntry.COLUMN_TRANSACTION_UID + " = ?",
@@ -390,6 +393,17 @@ public boolean deleteRecord(long rowId) {
         return result;
     }
 
+    /**
+     * Deletes a split from the database.
+     * This is a convenience method which essentially calls {@link #deleteRecord(long)}
+     * @param uid String unique ID of split
+     * @return <code>true</code> if the split was deleted, <code>false</code> otherwise
+     */
+    public boolean deleteSplit(String uid){
+        long id = getID(uid);
+        return deleteRecord(id);
+    }
+
     /**
      * Returns the database record ID for the specified transaction UID
      * @param transactionUID Unique idendtifier of the transaction

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

@@ -209,7 +209,7 @@ public void onActivityCreated(Bundle savedInstanceState) {
 		actionBar.setDisplayShowTitleEnabled(false);
 
 		SharedPreferences sharedPrefs = PreferenceManager.getDefaultSharedPreferences(getActivity());
-		mUseDoubleEntry = sharedPrefs.getBoolean(getString(R.string.key_use_double_entry), true);
+		mUseDoubleEntry = sharedPrefs.getBoolean(getString(R.string.key_use_double_entry), false);
 		if (!mUseDoubleEntry){
 			getView().findViewById(R.id.layout_double_entry).setVisibility(View.GONE);
             mOpenSplitsButton.setVisibility(View.GONE);
@@ -306,7 +306,9 @@ public void onItemClick(AdapterView<?> adapterView, View view, int position, lon
                         mSplitsList.clear();
                         setAmountEditViewVisible(View.VISIBLE);
                     } else {
-                        setAmountEditViewVisible(View.GONE);
+                        if (mUseDoubleEntry) { //don't hide the view in single entry mode
+                            setAmountEditViewVisible(View.GONE);
+                        }
                     }
                 }
                 mTransaction = null; //we are creating a new transaction after all
@@ -346,7 +348,7 @@ private void initializeViewsWithTransaction(){
         } else {
             for (Split split : mTransaction.getSplits()) {
                 //two splits, one belongs to this account and the other to another account
-                if (!split.getAccountUID().equals(accountUID)) {
+                if (mUseDoubleEntry && !split.getAccountUID().equals(accountUID)) {
                     setSelectedTransferAccount(mAccountsDbAdapter.getAccountID(split.getAccountUID()));
                 }
             }
@@ -593,15 +595,30 @@ private void saveNewTransaction() {
 
 		if (mTransaction != null){
             if (!mUseDoubleEntry){
+                //first remove old splits for this transaction, since there is only one split
+                SplitsDbAdapter splitsDbAdapter = new SplitsDbAdapter(getActivity());
+                for (Split split : mTransaction.getSplits()) {
+                    splitsDbAdapter.deleteSplit(split.getUID());
+                }
+                splitsDbAdapter.close();
+
                 Split split = new Split(amount, accountUID);
                 split.setType(mTransactionTypeButton.getTransactionType());
                 mTransaction.getSplits().clear();
                 mTransaction.addSplit(split);
-            } else
+            } else {
                 mTransaction.setSplits(mSplitsList);
+            }
 			mTransaction.setDescription(description);
 		} else {
 			mTransaction = new Transaction(description);
+            if (!mUseDoubleEntry){
+                Split split = new Split(amount, accountUID);
+                split.setType(mTransactionTypeButton.getTransactionType());
+                mSplitsList.clear();
+                mSplitsList.add(split);
+            }
+
             if (mSplitsList.isEmpty()) { //amount entered in the simple interface (not using splits Editor)
                 Split split = new Split(amount, accountUID);
                 split.setType(mTransactionTypeButton.getTransactionType());
@@ -707,8 +724,7 @@ public void setSplitList(List<Split> splitList, List<String> removedSplitUIDs){
         //once we set the split list, do not allow direct editing of the total
         if (mSplitsList.size() > 1){
             mAmountEditText.setEnabled(false);
-            mTransactionTypeButton.setVisibility(View.GONE);
-            getView().findViewById(R.id.layout_double_entry).setVisibility(View.GONE);
+            setAmountEditViewVisible(View.GONE);
         }
 
         SplitsDbAdapter splitsDbAdapter = new SplitsDbAdapter(getActivity());