c2c-oerpscenario team mailing list archive

["Amit Parik \(OpenERP\)" <amp@xxxxxxxxxxx>]

Hello

"account_payment_extension" is part of the extra-addons, which are
maintained by the community, including all the modules created by
"OpenERP" or "Tiny", as none of these are actively developed by our
teams anymore.

So if you or anyone from the community would like to work on it, please
mark the bug as "In progress" and assign it to yourself (or ask us or
anyone from OpenERP Drivers to do it for you). In the mean time, I will
mark it as "Won't fix" to indicate that no further progress is expected
at the moment. If you require further assistance with any similar
trouble while updating the module, do not hesitate to ask here or via a
question: https://answers.launchpad.net/openobject-addons

By the way the "account_payment_extension" module is not from Tiny So
you can contact with the related author.

Thank you!


** Changed in: openobject-addons
       Status: New => Won't Fix

-- 
You received this bug notification because you are a member of C2C
OERPScenario, which is subscribed to the OpenERP Project Group.
https://bugs.launchpad.net/bugs/747915

Title:
  account_payment_extension: accounting moves are reversed

Status in OpenERP Modules (addons):
  Won't Fix

Bug description:
  A payment from customer implies increasing assets hence a Dr in Bank account.
  Current move create a Cr in bank account when the payement is receive (ie line.amount <0)
  Suggestion in payment.py
  Change:
                  val = {
                      'name': line.move_line_id and line.move_line_id.name or '/',
                      'move_id': move_id,
                      'date': order.date_done,
                      'ref': line.move_line_id and line.move_line_id.ref or False,
                      'partner_id': line.partner_id and line.partner_id.id or False,
                      'account_id': line.account_id.id,
                      'debit': ((amount>0) and amount) or 0.0,
                      'credit': ((amount<0) and -amount) or 0.0,
                      'journal_id': order.mode.journal.id,
                      'period_id': order.period_id.id,
                      'currency_id': currency_id,
                  }
                  
  by 
                  val = {
                      'name': line.move_line_id and line.move_line_id.name or '/',
                      'move_id': move_id,
                      'date': order.date_done,
                      'ref': line.move_line_id and line.move_line_id.ref or False,
                      'partner_id': line.partner_id and line.partner_id.id or False,
                      'account_id': line.account_id.id,
                      'crebit': ((amount>0) and amount) or 0.0,
                      'dedit': ((amount<0) and -amount) or 0.0,
                      'journal_id': order.mode.journal.id,
                      'period_id': order.period_id.id,
                      'currency_id': currency_id,
                  }
                  
   and change:
                  self.pool.get('account.move.line').create(cr, uid, {
                      'name': line.move_line_id and line.move_line_id.name or '/',
                      'move_id': move_id,
                      'date': order.date_done,
                      'ref': line.move_line_id and line.move_line_id.ref or False,
                      'partner_id': line.partner_id and line.partner_id.id or False,
                      'account_id': account_id,
                      'debit': ((amount < 0) and -amount) or 0.0,
                      'credit': ((amount > 0) and amount) or 0.0,
                      'journal_id': order.mode.journal.id,
                      'period_id': order.period_id.id,
                      'amount_currency': amount_currency,
                      'currency_id': move_currency_id,
                  }, context)
  by 
                  self.pool.get('account.move.line').create(cr, uid, {
                      'name': line.move_line_id and line.move_line_id.name or '/',
                      'move_id': move_id,
                      'date': order.date_done,
                      'ref': line.move_line_id and line.move_line_id.ref or False,
                      'partner_id': line.partner_id and line.partner_id.id or False,
                      'account_id': account_id,
                      'crebit': ((amount < 0) and -amount) or 0.0,
                      'dedit': ((amount > 0) and amount) or 0.0,
                      'journal_id': order.mode.journal.id,
                      'period_id': order.period_id.id,
                      'amount_currency': amount_currency,
                      'currency_id': move_currency_id,
                  }, context)