banking-addons-team team mailing list archive

["Stefan Rijnhart \(Therp\)" <1112612@xxxxxxxxxxxxxxxxxx>]

Can't reproduce, and journal is returned from get_company_bank_account,
taken from the account settings for bankimport in this line:
http://bazaar.launchpad.net/~banking-addons-team/banking-addons/banking-
addons-70/view/head:/account_banking/wizard/banktools.py#L176


** Changed in: banking-addons
       Status: New => Invalid

-- 
You received this bug notification because you are a member of Banking
Addons Core Editors, which is subscribed to Banking Addons.
https://bugs.launchpad.net/bugs/1112612

Title:
  Statements found account but no default journal was defined.

Status in OpenERP Banking Addons:
  Invalid

Bug description:
  Importing statement always give error "Statements found account but no
  default journal was defined."

  On investigating code, it seems journal_id is never set in return
  value  of function get_company_bank_account.

  Please find the patch as below

  === modified file 'account_banking/wizard/bank_import.py'
  --- account_banking/wizard/bank_import.py	2012-07-11 10:37:31 +0000
  +++ account_banking/wizard/bank_import.py	2013-02-01 15:45:30 +0000
  @@ -202,7 +202,7 @@
                       error_accounts[statement.local_account] = True
                       results.error_cnt += 1
                       continue
  -                if 'journal_id' not in account_info.keys():
  +                if not account_info['account'].journal_id:
                       results.log.append(
                           _('Statements found for account %(bank_account)s, '
                             'but no default journal was defined.'

To manage notifications about this bug go to:
https://bugs.launchpad.net/banking-addons/+bug/1112612/+subscriptions