<p dir="ltr">I have not yet.  I moved last week and am still unpacking.  I will try to take a peek this weekend.<br>
</p>
<div class="gmail_quote">On Dec 5, 2013 3:54 AM, "Allan" <<a href="mailto:agander93@gmail.com">agander93@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Chris<br>
<br>
Just wondering if you've been able to do any tests for my query, below.<br>
<br>
<snip><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Another bug found  when importing from OFX direct connect.  The reader trying to create splits cannot find the brokerage account for a split using statementTransactionUnderImpor<u></u>t so all the imported transactions are broken.  This is likely because the transaction doesn't know the account name, but m_account does.  A patch:<br>

<br>
*** mymoneystatementreader.cpp.<u></u>orig     2013-11-24 17:27:39.959812598 -0800<br>
--- mymoneystatementreader.cpp  2013-11-24 20:01:40.339204121 -0800<br>
***************<br>
*** 565,570 ****<br>
--- 565,573 ----<br>
       if (brokerageactid.isEmpty()) {<br>
         brokerageactid = file->nameToAccount(<u></u>statementTransactionUnderImpor<u></u>t.m_strBrokerageAccount);<br>
       }<br>
+     if (brokerageactid.isEmpty()) {<br>
+       brokerageactid = file->nameToAccount(<u></u>thisaccount.brokerageName());<br>
+     }<br>
<br>
       // find the security transacted, UNLESS this transaction didn't<br>
       // involve any security.<br>
</blockquote>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
--<br>
<br>
Chris<br>
</blockquote>
______________________________<u></u>______________________________<u></u>________<br>
Hi Chris<br>
<br>
Thanks for this patch, too.  I've found here, though, that<br>
brokerageactid gets over-written a bit later, circa line 1086 in<br>
mymoneystatementreader.cpp -<br>
<br>
  // Add the 'account' split if it's needed<br>
  if (! transfervalue.isZero()) {<br>
    // in case the transaction has a reference to the brokerage<br>
account, we use it<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

   if (!<u></u>statementTransactionUnderImpor<u></u>t.m_strBrokerageAccount.<u></u>isEmpty() && brokerageactid.isEmpty()) {<<<<br>
</blockquote></blockquote></blockquote>
      brokerageactid =<br>
file->nameToAccount(<u></u>statementTransactionUnderImpor<u></u>t.m_strBrokerageAccount);<br>
    }<br>
    if (brokerageactid.isEmpty()) {<br>
      brokerageactid =<br>
file->accountByName(<u></u>statementTransactionUnderImpor<u></u>t.m_strBrokerageAccount).id();<br>
    }<br>
//  There is no BrokerageAccount so have to nowhere to put this split.<br>
    if (!brokerageactid.isEmpty()) {<br>
<br>
and I needed to make the change on the line indicated by >>>....<<<, to<br>
avoid that happening.<br>
<br>
I'm importing via CSV and there is no mapping of OFX source to an<br>
account for the brokerage/checking account so the user gets asked to<br>
enter this.  If he specifies an existing account, all is well.  If,<br>
however, because he makes a mistake or abbreviates a long name, the<br>
transfer will go to the investment account's associated brokerage<br>
account.  Generally, personally, I don't use these brokerage accounts<br>
because for me, they are not real and I would then have to make a manual<br>
transfer.<br>
<br>
It might be interesting to know what happens with your OFX transfers<br>
if/when they reach this code.<br>
<br>
Allan<br>
</blockquote></div>