credativ team mailing list archive

["Jagdish Panchal \(Open ERP\)" <912623@xxxxxxxxxxxxxxxxxx>]

Hello hbto(humbertoarocha),

Your patch seems to be correct but I have improved your patch as in one
querey the period.date_start was missing.

Its been fixed in the following branch:- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-912623-jap
Revision ID: jap@xxxxxxxxxxx-20120112092149-ism5impd38iliz2u
Revision no: 6326

It will be merged soon with addons.

Thanks for your contribution.

Thanks,
jap

** Changed in: openobject-addons
       Status: In Progress => Fix Committed

-- 
You received this bug notification because you are a member of OpenERP
Framework Experts, which is subscribed to OpenERP Addons.
https://bugs.launchpad.net/bugs/912623

Title:
  [account] Missing date on fiscalyear clousure

Status in OpenERP Addons (modules):
  Fix Committed

Bug description:
  [account] Missing date on fiscalyear clousure
  wizard account.fiscalyear.close lacks of date
  both for account.move records and account.move.line
  in both cases it just takes the default values given
  by those models, i.e, the day when the wizard is run,

  this could lead to miscorrected account balances, if
  You were to close the fiscal year 2011 at Feb/28th/2012
  It will create an account.move record & 
  account.move.lines with both date from fiscal_year's
  start_date and that of Feb/28th/2012,

  If the account.journal is checked allow_date=True
  with this bug this wizard will even complain because 
  in that date will not be in the range date of that period, 

  All the account.move & account.move.line records created
  during this wizard IMHO should be with date == fiscalyear.start_date
  as those for the account of type == 'detail'

  in the next lines you could find the needed changes to accomplish 
  that what I have exposed,

  Thanks in Advance,

  Hbto

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