Wrong memo order in splits?

Aug 1, 2014 at 9:53 AM
Edited Aug 1, 2014 at 9:56 AM
Exportet bank transactions with splittet amounts have wrong orders in the Memo fields. Here an example of the exportet qif file:

!Type:Bank
D6/15/95
T-10.0
LSupplies
MTransactionMemo
PNameofPayee
SSupplies
$-7.0
EFirst memo in split
SGarden
$-3.0
ESecond Memo in split
^

The correct order of the splittet Memo fields should be as follows in the exportet qif file:

!Type:Bank
D6/15/95
T-10.0
LSupplies
MTransactionMemo
PNameofPayee
SSupplies
EFirst memo in split
$-7.0
SGarden
ESecond Memo in split
$-3.0
^

Shouldn't be the code in the BankLogic class (file BankLogic.cs) changed to:
foreach (int i in item.SplitCategories.Keys)
{
      writer.WriteLine(NonInvestmentAccountFields.SplitCategory + item.SplitCategories[i]);
      
      // Output of split Memo field before the split amount
      if (item.SplitMemos.ContainsKey(i))
      {
          writer.WriteLine(NonInvestmentAccountFields.SplitMemo + item.SplitMemos[i]);
      }
          writer.WriteLine(NonInvestmentAccountFields.SplitAmount + item.SplitAmounts[i]);
}
Coordinator
Aug 15, 2014 at 3:31 AM
Please file this as a proposed issue. In the meanwhile, I'll get started on a fix.
Coordinator
Aug 15, 2014 at 8:11 AM
This should be resolved now. Thank you for pointing this out!