codinguser / gnucash-android

Gnucash for Android mobile companion application.
Apache License 2.0
1.23k stars 540 forks source link

Account import fails with no explanation #97

Closed strainu closed 10 years ago

strainu commented 11 years ago

I tried to import a large account structure from my desktop gnucash. The program imported a couple of entries, then failed with no explanation. The accounts already imported functioned poperly.

I would love to provide more data, but i'm not sure how (providing the structure is not an option).

The error message should give information about tbe line/entry that failed and the error encountered.

codinguser commented 11 years ago

Hi, What version of the app are you using? 1.2.0 or 1.2.1. Please if you have 1.2.0, pleas update and try again and let me know.

On 22.06.2013, at 23:51, Andrei Cipu notifications@github.com wrote:

I tried to import a large account structure from my desktop gnucash. The program imported a couple of entries, then failed with no explanation. The accounts already imported functioned poperly.

I would love to provide more data, but i'm not sure how (providing the structure is not an option).

The error message should give information about tbe line/entry that failed and the error encountered.

— Reply to this email directly or view it on GitHubhttps://github.com/codinguser/gnucash-android/issues/97 .

strainu commented 11 years ago

Latest -1.2.1. I had just installed it last night on a lg nexus 4, android 4.2.2

codinguser commented 11 years ago

Fixed in 4dc8cf92cc2979826a72e6aaeeafe7ef435f49a3 Version 1.2.2 includes this changes and will be available soon from the Play store

strainu commented 11 years ago

Still reproducing with 1.2.2. Assuming the accounts are loaded in order, this is the one that fails:

<gnc:account version="2.0.0">
  <act:name>Actiuni Disponibile</act:name>
  <act:id type="guid">cd5e4f74387c32e5ba01f91ce844c06a</act:id>
  <act:type>STOCK</act:type>
  <act:commodity>
    <cmdty:space>NASDAQ</cmdty:space>
    <cmdty:id>XXXX</cmdty:id>
  </act:commodity>
  <act:commodity-scu>10000</act:commodity-scu>
  <act:parent type="guid">cb892499965b49649a8042f72f9a9b35</act:parent>
</gnc:account>

XXXX - I changed the ID, but it exists in the file. Its entry reads:

<gnc:commodity version="2.0.0">
  <cmdty:space>NASDAQ</cmdty:space>
  <cmdty:id>XXXX</cmdty:id>
  <cmdty:name>An imaginary company</cmdty:name>
  <cmdty:fraction>10000</cmdty:fraction>
  <cmdty:get_quotes/>
  <cmdty:quote_source>yahoo</cmdty:quote_source>
  <cmdty:quote_tz/>
</gnc:commodity>
fefe982 commented 11 years ago

Has the same problem with 1.2.2. Only able to import when I manually deleted all account not using real money, such as the ones with manually created currencies in "INET_VIRTUAL" space.

strainu commented 11 years ago

Also failing EQUITY-type accounts

codinguser commented 11 years ago

thanks for the feedback

Mezzaluna commented 11 years ago

FWIW, same problem here with 1.2.2. Running Android 4.1.1. Fairly large account tree: I was importing 1MB XML file--been a gnucash user forever!

codinguser commented 11 years ago

This issue should be fixed in version 1.2.3. Please confirm.

Mezzaluna commented 11 years ago

yes, was able to import the acct list successfully--generated no errors.

however, the exported transactions don't seem to have the account information when the exported OFX file is read back into GNUcash. I get asked to map the account "manually"

On Fri, Aug 30, 2013 at 5:30 AM, Ngewi Fet notifications@github.com wrote:

This issue should be fixed in version 1.2.3. Please confirm.

— Reply to this email directly or view it on GitHubhttps://github.com/codinguser/gnucash-android/issues/97#issuecomment-23550144 .

codinguser commented 10 years ago

QIF export format is now supported and enables easier mapping to accounts in desktop GnuCash