← Back to team overview

c2c-oerpscenario team mailing list archive

[Bug 812540] Re: (account_voucher) action_move_line_create refactoring

 

Update patch

** Patch added: "action_move_line_create refactoring4"
   https://bugs.launchpad.net/openobject-addons/+bug/812540/+attachment/2234431/+files/account_voucher4.diff

-- 
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/812540

Title:
  (account_voucher) action_move_line_create refactoring

Status in OpenERP Modules (addons):
  Confirmed

Bug description:
  Version: 6.0
  Addons:
  revno: 4715
  revision-id: launchpad_translations_on_behalf_of_openerp-20110718051317-5w1lj4e12edvvqv7
  Module: account_voucher
  arch: account_voucher.py

  Hi,

  I think IMHO the action_move_line_create method need refactoring, it is too long and pretty
  confused, it has over 160 lines also there is a variable that doesn't follow the OpenERP Specific Guidelines
  (e.g. inv when you talk about voucher). I attached a patch file where this one
  becomes six smaller methods easier to understand and to use and to improve by others. I hope this can help
  a little to has a better readable/maintainable code.

  Regards,

  http://doc.openerp.com/v6.0/contribute/15_guidelines/coding_guidelines_framework.html
  http://doc.openerp.com/v6.0/contribute/15_guidelines/coding_guidelines_framework.html#call-your-fish-a-fish
  http://doc.openerp.com/v6.0/contribute/15_guidelines/coding_guidelines_framework.html#keep-your-methods-short-simple-when-possible

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


References