c2c-oerpscenario team mailing list archive

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

** Summary changed:

- [6.0](account_voucher) action_move_line_create refactoring
+ (account_voucher) action_move_line_create refactoring

** Changed in: openobject-addons
   Importance: Undecided => Wishlist

** Changed in: openobject-addons
       Status: New => Confirmed

** Changed in: openobject-addons
     Assignee: (unassigned) => OpenERP R&D Addons Team 3 (openerp-dev-addons3)

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