[PATCH 07/13] Remove useless variable (#5).

Vincent Ordy vincent at ordy.net
Thu Jun 17 18:26:14 CEST 2010


Because of the recent move of the files in the repository I can't do a svn blame to find out why the fixDuplicateAccounts() call has been disabled, maybe there is a TODO to add here.
---
 kmymoney/views/kmymoneyview.cpp |   12 +-----------
 1 files changed, 1 insertions(+), 11 deletions(-)

diff --git a/kmymoney/views/kmymoneyview.cpp b/kmymoney/views/kmymoneyview.cpp
index 394776f..de2109f 100644
--- a/kmymoney/views/kmymoneyview.cpp
+++ b/kmymoney/views/kmymoneyview.cpp
@@ -2113,14 +2113,11 @@ void KMyMoneyView::fixTransactions_0(void)
     }
 #endif
 
-    bool hasDuplicateAccounts = false;
     // Check for correct assignment of ActionInterest in all splits
     // and check if there are any duplicates in this transactions
     for (it_s = splits.begin(); it_s != splits.end(); ++it_s) {
       MyMoneyAccount splitAccount = file->account((*it_s).accountId());
-      if (accounts.contains((*it_s).accountId())) {
-        hasDuplicateAccounts = true;
-      } else {
+      if (!accounts.contains((*it_s).accountId())) {
         accounts << (*it_s).accountId();
       }
       // if this split references an interest account, the action
@@ -2174,13 +2171,6 @@ void KMyMoneyView::fixTransactions_0(void)
       }
     }
 
-    /*
-        // if there are at least two splits referencing the same account,
-        // we need to combine them into one and get rid of the others
-        if(hasDuplicateAccounts) {
-          fixDuplicateAccounts(*it_t);
-        }
-    */
     ++cnt;
     if (!(cnt % 10))
       kmymoney->slotStatusProgressBar(cnt);
-- 
1.7.1


--------------060703030708080806090607
Content-Type: text/x-diff;
 name="0008-Remove-useless-variable-6.patch"
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment;
 filename="0008-Remove-useless-variable-6.patch"



More information about the KMyMoney-devel mailing list