[IMP]: Generalization of tooltips: Added tooltips for 'origin' fields, renamed 'Ref...
[odoo/odoo.git] / addons / account / account_move_line.py
index d915a5f..dacca14 100644 (file)
@@ -1,30 +1,21 @@
-# -*- encoding: utf-8 -*-
+# -*- coding: utf-8 -*-
 ##############################################################################
 #
-# Copyright (c) 2004-2008 TINY SPRL. (http://tiny.be) All Rights Reserved.
+#    OpenERP, Open Source Management Solution
+#    Copyright (C) 2004-2009 Tiny SPRL (<http://tiny.be>).
 #
-# $Id$
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU Affero General Public License as
+#    published by the Free Software Foundation, either version 3 of the
+#    License, or (at your option) any later version.
 #
-# WARNING: This program as such is intended to be used by professional
-# programmers who take the whole responsability of assessing all potential
-# consequences resulting from its eventual inadequacies and bugs
-# End users who are looking for a ready-to-use solution with commercial
-# garantees and support are strongly adviced to contract a Free Software
-# Service Company
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU Affero General Public License for more details.
 #
-# This program is Free Software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License
-# as published by the Free Software Foundation; either version 2
-# of the License, or (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
+#    You should have received a copy of the GNU Affero General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
 #
 ##############################################################################
 
@@ -33,6 +24,11 @@ import netsvc
 from osv import fields, osv
 from tools.translate import _
 
+import mx.DateTime
+from mx.DateTime import RelativeDateTime, now, DateTime, localtime
+
+import tools
+
 class account_move_line(osv.osv):
     _name = "account.move.line"
     _description = "Entry lines"
@@ -45,16 +41,22 @@ class account_move_line(osv.osv):
         else:
             fiscalyear_clause = '%s' % context['fiscalyear']
         state=context.get('state',False)
-        where_move_state=''
+        where_move_state = ''
+        where_move_lines_by_date = ''
+
+        if context.get('date_from', False) and context.get('date_to', False):
+            where_move_lines_by_date = " AND " +obj+".move_id in ( select id from account_move  where date >= '" +context['date_from']+"' AND date <= '"+context['date_to']+"')"
+
         if state:
             if state.lower() not in ['all']:
                 where_move_state= " AND "+obj+".move_id in (select id from account_move where account_move.state = '"+state+"')"
 
+
         if context.get('periods', False):
             ids = ','.join([str(x) for x in context['periods']])
-            return obj+".active AND "+obj+".state<>'draft' AND "+obj+".period_id in (SELECT id from account_period WHERE fiscalyear_id in (%s) AND id in (%s)) %s" % (fiscalyear_clause, ids,where_move_state)
+            return obj+".state<>'draft' AND "+obj+".period_id in (SELECT id from account_period WHERE fiscalyear_id in (%s) AND id in (%s)) %s %s" % (fiscalyear_clause, ids,where_move_state,where_move_lines_by_date)
         else:
-            return obj+".active AND "+obj+".state<>'draft' AND "+obj+".period_id in (SELECT id from account_period WHERE fiscalyear_id in (%s) %s)" % (fiscalyear_clause,where_move_state)
+            return obj+".state<>'draft' AND "+obj+".period_id in (SELECT id from account_period WHERE fiscalyear_id in (%s) %s %s)" % (fiscalyear_clause,where_move_state,where_move_lines_by_date)
 
     def default_get(self, cr, uid, fields, context={}):
         data = self._default_get(cr, uid, fields, context)
@@ -63,15 +65,79 @@ class account_move_line(osv.osv):
                 del data[f]
         return data
 
+    def create_analytic_lines(self, cr, uid, ids, context={}):
+        for obj_line in self.browse(cr, uid, ids, context):
+            if obj_line.analytic_account_id:
+                if not obj_line.journal_id.analytic_journal_id:
+                    raise osv.except_osv(_('No Analytic Journal !'),_("You have to define an analytic journal on the '%s' journal!") % (obj_line.journal_id.name,))
+                amt = (obj_line.credit or  0.0) - (obj_line.debit or 0.0)
+                vals_lines={
+                    'name': obj_line.name,
+                    'date': obj_line.date,
+                    'account_id': obj_line.analytic_account_id.id,
+                    'unit_amount':obj_line.quantity,
+                    'product_id': obj_line.product_id and obj_line.product_id.id or False,
+                    'product_uom_id': obj_line.product_uom_id and obj_line.product_uom_id.id or False,
+                    'amount': amt,
+                    'general_account_id': obj_line.account_id.id,
+                    'journal_id': obj_line.journal_id.analytic_journal_id.id,
+                    'ref': obj_line.ref,
+                    'move_id':obj_line.id
+                }
+                new_id = self.pool.get('account.analytic.line').create(cr,uid,vals_lines)
+        return True
+
+    def _default_get_move_form_hook(self, cursor, user, data):
+        '''Called in the end of default_get method for manual entry in account_move form'''
+        if data.has_key('analytic_account_id'):
+            del(data['analytic_account_id'])
+        if data.has_key('account_tax_id'):
+            del(data['account_tax_id'])
+        return data
+
     def _default_get(self, cr, uid, fields, context={}):
         # Compute simple values
         data = super(account_move_line, self).default_get(cr, uid, fields, context)
+        # Starts: Manual entry from account.move form
+        if context.get('lines',[]):
+
+            total_new=0.00
+            for i in context['lines']:
+                if i[2]:
+                    total_new +=(i[2]['debit'] or 0.00)- (i[2]['credit'] or 0.00)
+                    for item in i[2]:
+                            data[item]=i[2][item]
+            if context['journal']:
+                journal_obj=self.pool.get('account.journal').browse(cr,uid,context['journal'])
+                if journal_obj.type == 'purchase':
+                    if total_new>0:
+                        account = journal_obj.default_credit_account_id
+                    else:
+                        account = journal_obj.default_debit_account_id
+                else:
+                    if total_new>0:
+                        account = journal_obj.default_credit_account_id
+                    else:
+                        account = journal_obj.default_debit_account_id
+
+
+                if account and ((not fields) or ('debit' in fields) or ('credit' in fields)) and 'partner_id' in data and (data['partner_id']):
+                    part = self.pool.get('res.partner').browse(cr, uid, data['partner_id'])
+                    account = self.pool.get('account.fiscal.position').map_account(cr, uid, part and part.property_account_position or False, account.id)
+                    account = self.pool.get('account.account').browse(cr, uid, account)
+                    data['account_id'] =  account.id
+
+            s = -total_new
+            data['debit'] = s>0  and s or 0.0
+            data['credit'] = s<0  and -s or 0.0
+            data = self._default_get_move_form_hook(cr, uid, data)
+            return data
+        # Ends: Manual entry from account.move form
 
         if not 'move_id' in fields: #we are not in manual entry
             return data
 
         period_obj = self.pool.get('account.period')
-        tax_obj=self.pool.get('account.tax')
 
         # Compute the current move
         move_id = False
@@ -82,7 +148,7 @@ class account_move_line(osv.osv):
                     from \
                         account_move_line \
                     where \
-                        journal_id=%d and period_id=%d and create_uid=%d and state=%s \
+                        journal_id=%s and period_id=%s and create_uid=%s and state=%s \
                     order by id desc limit 1',
                     (context['journal_id'], context['period_id'], uid, 'draft'))
                 res = cr.fetchone()
@@ -97,7 +163,7 @@ class account_move_line(osv.osv):
                     from \
                         account_move_line \
                     where \
-                        journal_id=%d and period_id=%d and create_uid=%d \
+                        journal_id=%s and period_id=%s and create_uid=%s \
                     order by id desc',
                     (context['journal_id'], context['period_id'], uid))
                 res = cr.fetchone()
@@ -114,13 +180,15 @@ class account_move_line(osv.osv):
         total = 0
         ref_id = False
         move = self.pool.get('account.move').browse(cr, uid, move_id, context)
-
+        if 'name' in fields:
+            data.setdefault('name', move.line_id[-1].name)
+        acc1 = False
         for l in move.line_id:
+            acc1 = l.account_id
             partner_id = partner_id or l.partner_id.id
             ref_id = ref_id or l.ref
             total += (l.debit or 0.0) - (l.credit or 0.0)
-            if 'name' in fields:
-               data.setdefault('name', l.name)
+
         if 'ref' in fields:
             data['ref'] = ref_id
         if 'partner_id' in fields:
@@ -137,14 +205,35 @@ class account_move_line(osv.osv):
             else:
                 account = move.journal_id.default_debit_account_id
 
-        data['account_id'] = account.id
-        if account and account.tax_ids:
-            for tax in self.pool.get('account.tax').compute_inv(cr,uid,[account.tax_ids[0]],total,1.00):
-                total -= tax['amount']
-            data['account_tax_id'] = account.tax_ids[0].id
+        part = partner_id and self.pool.get('res.partner').browse(cr, uid, partner_id) or False
+        # part = False is acceptable for fiscal position.
+        account = self.pool.get('account.fiscal.position').map_account(cr, uid, part and part.property_account_position or False, account.id)
+        if account:
+            account = self.pool.get('account.account').browse(cr, uid, account)
+
+        if account and ((not fields) or ('debit' in fields) or ('credit' in fields)):
+            data['account_id'] = account.id
+            # Propose the price VAT excluded, the VAT will be added when confirming line
+            if account.tax_ids:
+                taxes = self.pool.get('account.fiscal.position').map_tax(cr, uid, part and part.property_account_position or False, account.tax_ids)
+                tax = self.pool.get('account.tax').browse(cr, uid, taxes)
+                for t in self.pool.get('account.tax').compute_inv(cr, uid, tax, total, 1):
+                    total -= t['amount']
+
         s = -total
         data['debit'] = s>0  and s or 0.0
         data['credit'] = s<0  and -s or 0.0
+
+        if account and account.currency_id:
+            data['currency_id'] = account.currency_id.id
+            acc = account
+            if s>0:
+                acc = acc1
+            v = self.pool.get('res.currency').compute(cr, uid,
+                account.company_id.currency_id.id,
+                data['currency_id'],
+                s, account=acc, account_invert=True)
+            data['amount_currency'] = v
         return data
 
     def _on_create_write(self, cr, uid, id, context={}):
@@ -155,9 +244,9 @@ class account_move_line(osv.osv):
         res={}
         # TODO group the foreach in sql
         for id in ids:
-            cr.execute('SELECT date,account_id FROM account_move_line WHERE id=%d', (id,))
+            cr.execute('SELECT date,account_id FROM account_move_line WHERE id=%s', (id,))
             dt, acc = cr.fetchone()
-            cr.execute('SELECT SUM(debit-credit) FROM account_move_line WHERE account_id=%d AND (date<%s OR (date=%s AND id<=%d)) and active', (acc,dt,dt,id))
+            cr.execute('SELECT SUM(debit-credit) FROM account_move_line WHERE account_id=%s AND (date<%s OR (date=%s AND id<=%s))', (acc,dt,dt,id))
             res[id] = cr.fetchone()[0]
         return res
 
@@ -194,6 +283,17 @@ class account_move_line(osv.osv):
                 result.append((line.id, line.name))
         return result
 
+    def _balance_search(self, cursor, user, obj, name, args):
+        if not len(args):
+            return []
+        where = ' and '.join(map(lambda x: '(abs(sum(debit-credit))'+x[1]+str(x[2])+')',args))
+        cursor.execute('select id, sum(debit-credit) from account_move_line \
+                     group by id,debit,credit having '+where)
+        res = cursor.fetchall()
+        if not len(res):
+            return [('id', '=', '0')]
+        return [('id', 'in', [x[0] for x in res])]
+
     def _invoice_search(self, cursor, user, obj, name, args):
         if not len(args):
             return []
@@ -203,7 +303,7 @@ class account_move_line(osv.osv):
         while i < len(args):
             fargs = args[i][0].split('.', 1)
             if len(fargs) > 1:
-                args[i] = (frags[0], 'in', invoice_obj.search(cursor, user,
+                args[i] = (fargs[0], 'in', invoice_obj.search(cursor, user,
                     [(fargs[1], args[i][1], args[i][2])]))
                 i += 1
                 continue
@@ -220,11 +320,11 @@ class account_move_line(osv.osv):
                 elif (x[2] is False) and (x[1] == '<>' or x[1] == '!='):
                     qu1.append('(i.id IS NOT NULL)')
                 else:
-                    qu1.append('(i.id %s %s)' % (x[1], '%d'))
+                    qu1.append('(i.id %s %s)' % (x[1], '%s'))
                     qu2.append(x[2])
             elif x[1] == 'in':
                 if len(x[2]) > 0:
-                    qu1.append('(i.id in (%s))' % (','.join(['%d'] * len(x[2]))))
+                    qu1.append('(i.id in (%s))' % (','.join(['%s'] * len(x[2]))))
                     qu2 += x[2]
                 else:
                     qu1.append(' (False)')
@@ -240,42 +340,57 @@ class account_move_line(osv.osv):
             return [('id', '=', '0')]
         return [('id', 'in', [x[0] for x in res])]
 
+    def _get_move_lines(self, cr, uid, ids, context={}):
+        result = []
+        for move in self.pool.get('account.move').browse(cr, uid, ids, context=context):
+            for line in move.line_id:
+                result.append(line.id)
+        return result
+
     _columns = {
         'name': fields.char('Name', size=64, required=True),
-        'quantity': fields.float('Quantity', digits=(16,2), help="The optionnal quantity expressed by this line, eg: number of product sold. The quantity is not a legal requirement but is very usefull for some reports."),
-        'debit': fields.float('Debit', digits=(16,2)),
-        'credit': fields.float('Credit', digits=(16,2)),
+        'quantity': fields.float('Quantity', digits=(16,2), help="The optional quantity expressed by this line, eg: number of product sold. The quantity is not a legal requirement but is very useful for some reports."),
+        'product_uom_id': fields.many2one('product.uom', 'UoM'),
+        'product_id': fields.many2one('product.product', 'Product'),
+        'debit': fields.float('Debit', digits=(16,int(tools.config['price_accuracy']))),
+        'credit': fields.float('Credit', digits=(16,int(tools.config['price_accuracy']))),
         'account_id': fields.many2one('account.account', 'Account', required=True, ondelete="cascade", domain=[('type','<>','view'), ('type', '<>', 'closed')], select=2),
         'move_id': fields.many2one('account.move', 'Move', ondelete="cascade", states={'valid':[('readonly',True)]}, help="The move of this entry line.", select=2),
 
-        'ref': fields.char('Ref.', size=32),
+        'ref': fields.char('Reference', size=32),
         'statement_id': fields.many2one('account.bank.statement', 'Statement', help="The bank statement used for bank reconciliation", select=1),
         'reconcile_id': fields.many2one('account.move.reconcile', 'Reconcile', readonly=True, ondelete='set null', select=2),
         'reconcile_partial_id': fields.many2one('account.move.reconcile', 'Partial Reconcile', readonly=True, ondelete='set null', select=2),
-        'amount_currency': fields.float('Amount Currency', help="The amount expressed in an optionnal other currency if it is a multi-currency entry."),
-        'currency_id': fields.many2one('res.currency', 'Currency', help="The optionnal other currency if it is a multi-currency entry."),
+        'amount_currency': fields.float('Amount Currency', help="The amount expressed in an optional other currency if it is a multi-currency entry.", digits=(16,int(tools.config['price_accuracy']))),
+        'currency_id': fields.many2one('res.currency', 'Currency', help="The optional other currency if it is a multi-currency entry."),
 
         'period_id': fields.many2one('account.period', 'Period', required=True, select=2),
         'journal_id': fields.many2one('account.journal', 'Journal', required=True, select=1),
         'blocked': fields.boolean('Litigation', help="You can check this box to mark the entry line as a litigation with the associated partner"),
 
-        'partner_id': fields.many2one('res.partner', 'Partner Ref.'),
+        'partner_id': fields.many2one('res.partner', 'Partner'),
         'date_maturity': fields.date('Maturity date', help="This field is used for payable and receivable entries. You can put the limit date for the payment of this entry line."),
-        'date': fields.date('Effective date', required=True),
+        'date': fields.related('move_id','date', string='Effective date', type='date', required=True,
+            store={
+                'account.move': (_get_move_lines, ['date'], 20)
+            }),
         'date_created': fields.date('Creation date'),
         'analytic_lines': fields.one2many('account.analytic.line', 'move_id', 'Analytic lines'),
         'centralisation': fields.selection([('normal','Normal'),('credit','Credit Centralisation'),('debit','Debit Centralisation')], 'Centralisation', size=6),
-        'balance': fields.function(_balance, method=True, string='Balance'),
-        'active': fields.boolean('Active'),
-        'state': fields.selection([('draft','Draft'), ('valid','Valid')], 'State', readonly=True),
-        'tax_code_id': fields.many2one('account.tax.code', 'Tax Account'),
-        'tax_amount': fields.float('Tax/Base Amount', digits=(16,2), select=True),
+        'balance': fields.function(_balance, fnct_search=_balance_search, method=True, string='Balance'),
+        'state': fields.selection([('draft','Draft'), ('valid','Valid')], 'State', readonly=True,
+                                  help='When new move line is created the state will be \'Draft\'.\n* When all the payments are done it will be in \'Valid\' state.'),
+        'tax_code_id': fields.many2one('account.tax.code', 'Tax Account', help="The Account can either be a base tax code or tax code account."),
+        'tax_amount': fields.float('Tax/Base Amount', digits=(16,int(tools.config['price_accuracy'])), select=True, help="If the Tax account is a tax code account, this field will contain the taxed amount.If the tax account is base tax code,\
+                    this field will contain the basic amount(without tax)."),
         'invoice': fields.function(_invoice, method=True, string='Invoice',
             type='many2one', relation='account.invoice', fnct_search=_invoice_search),
         'account_tax_id':fields.many2one('account.tax', 'Tax'),
         'analytic_account_id' : fields.many2one('account.analytic.account', 'Analytic Account'),
 #TODO: remove this
-        'amount_taxed':fields.float("Taxed Amount",digits=(16,2)),
+        'amount_taxed':fields.float("Taxed Amount",digits=(16,int(tools.config['price_accuracy']))),
+        'company_id': fields.related('account_id','company_id',type='many2one',relation='res.company',string='Company',store=True)
+
     }
 
     def _get_date(self, cr, uid, context):
@@ -283,7 +398,7 @@ class account_move_line(osv.osv):
         dt = time.strftime('%Y-%m-%d')
         if ('journal_id' in context) and ('period_id' in context):
             cr.execute('select date from account_move_line ' \
-                    'where journal_id=%d and period_id=%d ' \
+                    'where journal_id=%s and period_id=%s ' \
                     'order by id desc limit 1',
                     (context['journal_id'], context['period_id']))
             res = cr.fetchone()
@@ -294,15 +409,22 @@ class account_move_line(osv.osv):
                         context=context)
                 dt = period.date_start
         return dt
+    def _get_currency(self, cr, uid, context={}):
+        if not context.get('journal_id', False):
+            return False
+        cur = self.pool.get('account.journal').browse(cr, uid, context['journal_id']).currency
+        return cur and cur.id or False
+
     _defaults = {
         'blocked': lambda *a: False,
-        'active': lambda *a: True,
         'centralisation': lambda *a: 'normal',
         'date': _get_date,
         'date_created': lambda *a: time.strftime('%Y-%m-%d'),
         'state': lambda *a: 'draft',
+        'currency_id': _get_currency,
         'journal_id': lambda self, cr, uid, c: c.get('journal_id', False),
         'period_id': lambda self, cr, uid, c: c.get('period_id', False),
+        'company_id': lambda self,cr,uid,c: self.pool.get('res.company')._company_default_get(cr, uid, 'account.move.line', c)
     }
     _order = "date desc,id desc"
     _sql_constraints = [
@@ -338,20 +460,63 @@ class account_move_line(osv.osv):
 
     #TODO: ONCHANGE_ACCOUNT_ID: set account_tax_id
 
-    def onchange_partner_id(self, cr, uid, ids, move_id, partner_id, account_id=None, debit=0, credit=0, journal=False):
-        if (not partner_id) or account_id:
+    def onchange_currency(self, cr, uid, ids, account_id, amount, currency_id, date=False, journal=False):
+        if (not currency_id) or (not account_id):
             return {}
+        result = {}
+        acc =self.pool.get('account.account').browse(cr, uid, account_id)
+        if (amount>0) and journal:
+            x = self.pool.get('account.journal').browse(cr, uid, journal).default_credit_account_id
+            if x: acc = x
+        v = self.pool.get('res.currency').compute(cr, uid, currency_id,acc.company_id.currency_id.id, amount, account=acc)
+        result['value'] = {
+            'debit': v>0 and v or 0.0,
+            'credit': v<0 and -v or 0.0
+        }
+        return result
+
+    def onchange_partner_id(self, cr, uid, ids, move_id, partner_id, account_id=None, debit=0, credit=0, date=False, journal=False):
+        val = {}
+        val['date_maturity'] = False
+
+        if not partner_id:
+            return {'value':val}
+        if not date:
+            date = now().strftime('%Y-%m-%d')
         part = self.pool.get('res.partner').browse(cr, uid, partner_id)
-        id1 = part.property_account_payable.id
-        id2 =  part.property_account_receivable.id
+
+        if part.property_payment_term and part.property_payment_term.line_ids:
+            payterm = part.property_payment_term.line_ids[0]
+            res = self.pool.get('account.payment.term').compute(cr, uid, payterm.id, 100, date)
+            if res:
+                val['date_maturity'] = res[0][0]
+        if not account_id:
+            id1 = part.property_account_payable.id
+            id2 =  part.property_account_receivable.id
+            if journal:
+                jt = self.pool.get('account.journal').browse(cr, uid, journal).type
+                if jt=='sale':
+                    val['account_id'] = self.pool.get('account.fiscal.position').map_account(cr, uid, part and part.property_account_position or False, id2)
+
+                elif jt=='purchase':
+                    val['account_id'] = self.pool.get('account.fiscal.position').map_account(cr, uid, part and part.property_account_position or False, id1)
+                if val.get('account_id', False):
+                    d = self.onchange_account_id(cr, uid, ids, val['account_id'])
+                    val.update(d['value'])
+
+        return {'value':val}
+
+    def onchange_account_id(self, cr, uid, ids, account_id=False, partner_id=False):
         val = {}
-        if journal:
-            jt = self.pool.get('account.journal').browse(cr, uid, journal).type
-            if jt=='sale':
-                val['account_id'] =  id2
-            elif jt=='purchase':
-                val['account_id'] =  id1
-        # Compute Maturity Date in val !
+        if account_id:
+            res = self.pool.get('account.account').browse(cr, uid, account_id)
+            tax_ids = res.tax_ids
+            if tax_ids and partner_id:
+                part = self.pool.get('res.partner').browse(cr, uid, partner_id)
+                tax_id = self.pool.get('account.fiscal.position').map_tax(cr, uid, part and part.property_account_position or False, tax_ids)[0]
+            else:
+                tax_id = tax_ids and tax_ids[0].id or False
+            val['account_tax_id'] = tax_id
         return {'value':val}
 
     #
@@ -367,7 +532,7 @@ class account_move_line(osv.osv):
         merges_rec = []
         for line in self.browse(cr, uid, ids, context):
             if line.reconcile_id:
-                raise _('Already Reconciled')
+                raise osv.except_osv(_('Already Reconciled'), _('Already Reconciled'))
             if line.reconcile_partial_id:
                 for line2 in line.reconcile_partial_id.line_partial_ids:
                     if not line2.reconcile_id:
@@ -378,7 +543,8 @@ class account_move_line(osv.osv):
                 unmerge.append(line.id)
                 total += (line.debit or 0.0) - (line.credit or 0.0)
         if not total:
-            return self.reconcile(cr, uid, merges+unmerge, context=context)
+            res = self.reconcile(cr, uid, merges+unmerge, context=context)
+            return res
         r_id = self.pool.get('account.move.reconcile').create(cr, uid, {
             'type': type,
             'line_partial_ids': map(lambda x: (4,x,False), merges+unmerge)
@@ -388,6 +554,7 @@ class account_move_line(osv.osv):
 
     def reconcile(self, cr, uid, ids, type='auto', writeoff_acc_id=False, writeoff_period_id=False, writeoff_journal_id=False, context={}):
         id_set = ','.join(map(str, ids))
+
         lines = self.browse(cr, uid, ids, context=context)
         unrec_lines = filter(lambda x: not x['reconcile_id'], lines)
         credit = debit = 0.0
@@ -416,11 +583,13 @@ class account_move_line(osv.osv):
                 GROUP BY account_id,reconcile_id')
         r = cr.fetchall()
 #TODO: move this check to a constraint in the account_move_reconcile object
-        if len(r) != 1:
+        if (len(r) != 1) and not context.get('fy_closing', False):
             raise osv.except_osv(_('Error'), _('Entries are not of the same account or already reconciled ! '))
+        if not unrec_lines:
+            raise osv.except_osv(_('Error'), _('Entry is already reconciled'))
         account = self.pool.get('account.account').browse(cr, uid, account_id, context=context)
-        if not account.reconcile:
-            raise osv.except_osv(_('Error'), _('The account is not defined to be reconcile !'))
+        if not context.get('fy_closing', False) and not account.reconcile:
+            raise osv.except_osv(_('Error'), _('The account is not defined to be reconciled !'))
         if r[0][1] != None:
             raise osv.except_osv(_('Error'), _('Some entries are already reconciled !'))
 
@@ -440,7 +609,7 @@ class account_move_line(osv.osv):
                 self_debit = -writeoff
 
             # If comment exist in context, take it
-            if context['comment']:
+            if 'comment' in context and context['comment']:
                 libelle=context['comment']
             else:
                 libelle='Write-Off'
@@ -461,22 +630,16 @@ class account_move_line(osv.osv):
                     'debit':debit,
                     'credit':credit,
                     'account_id':writeoff_acc_id,
+                    'analytic_account_id': context.get('analytic_id', False),
                     'date':date,
                     'partner_id':partner_id
                 })
             ]
 
-            name = 'Write-Off'
-            if writeoff_journal_id:
-                journal = self.pool.get('account.journal').browse(cr, uid, writeoff_journal_id)
-                if journal.sequence_id:
-                    name = self.pool.get('ir.sequence').get_id(cr, uid, journal.sequence_id.id)
-
             writeoff_move_id = self.pool.get('account.move').create(cr, uid, {
-                'name': name,
                 'period_id': writeoff_period_id,
                 'journal_id': writeoff_journal_id,
-
+                'date':date,
                 'state': 'draft',
                 'line_id': writeoff_lines
             })
@@ -490,31 +653,31 @@ class account_move_line(osv.osv):
             'line_id': map(lambda x: (4,x,False), ids),
             'line_partial_ids': map(lambda x: (3,x,False), ids)
         })
+        wf_service = netsvc.LocalService("workflow")
         # the id of the move.reconcile is written in the move.line (self) by the create method above
         # because of the way the line_id are defined: (4, x, False)
-        wf_service = netsvc.LocalService("workflow")
         for id in ids:
             wf_service.trg_trigger(uid, 'account.move.line', id, cr)
         return r_id
 
     def view_header_get(self, cr, user, view_id, view_type, context):
         if context.get('account_id', False):
-            cr.execute('select code from account_account where id=%d', (context['account_id'],))
+            cr.execute('select code from account_account where id=%s', (context['account_id'],))
             res = cr.fetchone()
             res = _('Entries: ')+ (res[0] or '')
             return res
         if (not context.get('journal_id', False)) or (not context.get('period_id', False)):
             return False
-        cr.execute('select code from account_journal where id=%d', (context['journal_id'],))
+        cr.execute('select code from account_journal where id=%s', (context['journal_id'],))
         j = cr.fetchone()[0] or ''
-        cr.execute('select code from account_period where id=%d', (context['period_id'],))
+        cr.execute('select code from account_period where id=%s', (context['period_id'],))
         p = cr.fetchone()[0] or ''
         if j or p:
             return j+(p and (':'+p) or '')
         return False
 
-    def fields_view_get(self, cr, uid, view_id=None, view_type='form', context={}, toolbar=False):
-        result = super(osv.osv, self).fields_view_get(cr, uid, view_id,view_type,context)
+    def fields_view_get(self, cr, uid, view_id=None, view_type='form', context={}, toolbar=False, submenu=False):
+        result = super(osv.osv, self).fields_view_get(cr, uid, view_id,view_type,context,toolbar=toolbar, submenu=submenu)
         if view_type=='tree' and 'journal_id' in context:
             title = self.view_header_get(cr, uid, view_id, view_type, context)
             journal = self.pool.get('account.journal').browse(cr, uid, context['journal_id'])
@@ -544,16 +707,21 @@ class account_move_line(osv.osv):
                     attrs.append('sum="Total debit"')
                 elif field.field=='credit':
                     attrs.append('sum="Total credit"')
+                elif field.field=='account_tax_id':
+                    attrs.append('domain="[(\'parent_id\',\'=\',False)]"')
                 elif field.field=='account_id' and journal.id:
-                    attrs.append('domain="[(\'journal_id\', \'=\', '+str(journal.id)+'),(\'type\',\'&lt;&gt;\',\'view\'), (\'type\',\'&lt;&gt;\',\'closed\')]"')
+                    attrs.append('domain="[(\'journal_id\', \'=\', '+str(journal.id)+'),(\'type\',\'&lt;&gt;\',\'view\'), (\'type\',\'&lt;&gt;\',\'closed\')]" on_change="onchange_account_id(account_id, partner_id)"')
+                elif field.field == 'partner_id':
+                    attrs.append('on_change="onchange_partner_id(move_id,partner_id,account_id,debit,credit,date,((\'journal_id\' in context) and context[\'journal_id\']) or {})"')
                 if field.readonly:
                     attrs.append('readonly="1"')
                 if field.required:
                     attrs.append('required="1"')
                 else:
                     attrs.append('required="0"')
-                if field.field == 'partner_id':
-                    attrs.append('on_change="onchange_partner_id(move_id,partner_id,account_id,debit,credit,((\'journal_id\' in context) and context[\'journal_id\']) or {})"')
+                if field.field in ('amount_currency','currency_id'):
+                    attrs.append('on_change="onchange_currency(account_id,amount_currency,currency_id,date,((\'journal_id\' in context) and context[\'journal_id\']) or {})"')
+
                 if field.field in widths:
                     attrs.append('width="'+str(widths[field.field])+'"')
                 xml += '''<field name="%s" %s/>\n''' % (field.field,' '.join(attrs))
@@ -577,36 +745,34 @@ class account_move_line(osv.osv):
     def write(self, cr, uid, ids, vals, context=None, check=True, update_check=True):
         if not context:
             context={}
-        raise_ex=False
-        account_obj = self.pool.get('account.account')
-        acc=account_obj.browse(cr,uid,ids)[0]
-
-        if ('debit' in vals and 'credit' in vals)  and not vals['debit'] and not vals['credit']:
-            raise_ex=True
-        if ('debit' in vals and 'credit' not in vals) and  not vals['debit'] and not acc.credit:
-            raise_ex=True
-        if ('credit' in vals and 'debit' not in vals) and  not vals['credit'] and not acc.debit:
-            raise_ex=True
-
-        if raise_ex:
-            raise osv.except_osv(_('Wrong Accounting Entry!'), _('Both Credit and Debit cannot be zero!'))
+        if vals.get('account_tax_id', False):
+            raise osv.except_osv(_('Unable to change tax !'), _('You can not change the tax, you should remove and recreate lines !'))
 
+        account_obj = self.pool.get('account.account')
         if ('account_id' in vals) and not account_obj.read(cr, uid, vals['account_id'], ['active'])['active']:
             raise osv.except_osv(_('Bad account!'), _('You can not use an inactive account!'))
         if update_check:
             if ('account_id' in vals) or ('journal_id' in vals) or ('period_id' in vals) or ('move_id' in vals) or ('debit' in vals) or ('credit' in vals) or ('date' in vals):
                 self._update_check(cr, uid, ids, context)
-        result = super(osv.osv, self).write(cr, uid, ids, vals, context)
+
+        todo_date = None
+        if vals.get('date', False):
+            todo_date = vals['date']
+            del vals['date']
+        result = super(account_move_line, self).write(cr, uid, ids, vals, context)
+
         if check:
             done = []
             for line in self.browse(cr, uid, ids):
                 if line.move_id.id not in done:
                     done.append(line.move_id.id)
                     self.pool.get('account.move').validate(cr, uid, [line.move_id.id], context)
+                    if todo_date:
+                        self.pool.get('account.move').write(cr, uid, [line.move_id.id], {'date': todo_date}, context=context)
         return result
 
     def _update_journal_check(self, cr, uid, journal_id, period_id, context={}):
-        cr.execute('select state from account_journal_period where journal_id=%d and period_id=%d', (journal_id, period_id))
+        cr.execute('select state from account_journal_period where journal_id=%s and period_id=%s', (journal_id, period_id))
         result = cr.fetchall()
         for (state,) in result:
             if state=='done':
@@ -637,42 +803,42 @@ class account_move_line(osv.osv):
     def create(self, cr, uid, vals, context=None, check=True):
         if not context:
             context={}
-
         account_obj = self.pool.get('account.account')
         tax_obj=self.pool.get('account.tax')
-
         if ('account_id' in vals) and not account_obj.read(cr, uid, vals['account_id'], ['active'])['active']:
             raise osv.except_osv(_('Bad account!'), _('You can not use an inactive account!'))
         if 'journal_id' in vals and 'journal_id' not in context:
             context['journal_id'] = vals['journal_id']
         if 'period_id' in vals and 'period_id' not in context:
             context['period_id'] = vals['period_id']
-        if 'journal_id' not in context and 'move_id' in vals:
+        if ('journal_id' not in context) and ('move_id' in vals) and vals['move_id']:
             m = self.pool.get('account.move').browse(cr, uid, vals['move_id'])
             context['journal_id'] = m.journal_id.id
             context['period_id'] = m.period_id.id
 
         self._update_journal_check(cr, uid, context['journal_id'], context['period_id'], context)
+        company_currency = self.pool.get('res.users').browse(cr, uid, uid, context=context).company_id.currency_id.id
 
         move_id = vals.get('move_id', False)
         journal = self.pool.get('account.journal').browse(cr, uid, context['journal_id'])
+        is_new_move = False
         if not move_id:
             if journal.centralisation:
                 # use the first move ever created for this journal and period
-                cr.execute('select id, state, name from account_move where journal_id=%d and period_id=%d order by id limit 1', (context['journal_id'],context['period_id']))
+                cr.execute('select id, state, name from account_move where journal_id=%s and period_id=%s order by id limit 1', (context['journal_id'],context['period_id']))
                 res = cr.fetchone()
                 if res:
                     if res[1] != 'draft':
                         raise osv.except_osv(_('UserError'),
-                                _('The account move (%s) for centralisation ' \
+                                _('The Ledger Posting (%s) for centralisation ' \
                                         'has been confirmed!') % res[2])
                     vals['move_id'] = res[0]
 
             if not vals.get('move_id', False):
                 if journal.sequence_id:
-                    name = self.pool.get('ir.sequence').get_id(cr, uid, journal.sequence_id.id)
+                    #name = self.pool.get('ir.sequence').get_id(cr, uid, journal.sequence_id.id)
                     v = {
-                        'name': name,
+                        'date': vals.get('date', time.strftime('%Y-%m-%d')),
                         'period_id': context['period_id'],
                         'journal_id': context['journal_id']
                     }
@@ -680,14 +846,15 @@ class account_move_line(osv.osv):
                     vals['move_id'] = move_id
                 else:
                     raise osv.except_osv(_('No piece number !'), _('Can not create an automatic sequence for this piece !\n\nPut a sequence in the journal definition for automatic numbering or create a sequence manually for this piece.'))
+            is_new_move = True
 
         ok = not (journal.type_control_ids or journal.account_control_ids)
         if ('account_id' in vals):
             account = account_obj.browse(cr, uid, vals['account_id'])
             if journal.type_control_ids:
-                type = account.type
+                type = account.user_type
                 for t in journal.type_control_ids:
-                    if type==t.code:
+                    if type.code == t.code:
                         ok = True
                         break
             if journal.account_control_ids and not ok:
@@ -695,59 +862,112 @@ class account_move_line(osv.osv):
                     if a.id==vals['account_id']:
                         ok = True
                         break
-            if (account.currency_id) and 'amount_currency' not in vals:
+            if (account.currency_id) and 'amount_currency' not in vals and account.currency_id.id <> company_currency:
                 vals['currency_id'] = account.currency_id.id
                 cur_obj = self.pool.get('res.currency')
                 ctx = {}
                 if 'date' in vals:
                     ctx['date'] = vals['date']
-                vals['amount_currency'] = cur_obj.compute(cr, uid, account.company_id.currency_id.id, account.currency_id.id, vals.get('debit', 0.0)-vals.get('credit', 0.0), context=ctx)
+                vals['amount_currency'] = cur_obj.compute(cr, uid, account.company_id.currency_id.id,
+                    account.currency_id.id, vals.get('debit', 0.0)-vals.get('credit', 0.0),
+                    context=ctx)
         if not ok:
             raise osv.except_osv(_('Bad account !'), _('You can not use this general account in this journal !'))
 
-        result = super(osv.osv, self).create(cr, uid, vals, context)
         if 'analytic_account_id' in vals and vals['analytic_account_id']:
             if journal.analytic_journal_id:
                 vals['analytic_lines'] = [(0,0, {
                         'name': vals['name'],
-                        'date': vals['date'],
+                        'date': vals.get('date', time.strftime('%Y-%m-%d')),
                         'account_id': vals['analytic_account_id'],
-                        'unit_amount': vals['quantity'],
+                        'unit_amount':'quantity' in vals and vals['quantity'] or 1.0,
                         'amount': vals['debit'] or vals['credit'],
                         'general_account_id': vals['account_id'],
                         'journal_id': journal.analytic_journal_id.id,
-                        'ref': vals['ref'],
+                        'ref': vals.get('ref', False),
                     })]
+            #else:
+            #    raise osv.except_osv(_('No analytic journal !'), _('Please set an analytic journal on this financial journal !'))
 
+        #if not 'currency_id' in vals:
+        #    vals['currency_id'] = account.company_id.currency_id.id
+
+        result = super(osv.osv, self).create(cr, uid, vals, context)
         # CREATE Taxes
         if 'account_tax_id' in vals and vals['account_tax_id']:
             tax_id=tax_obj.browse(cr,uid,vals['account_tax_id'])
-            total = vals['credit'] or (-vals['debit'])
+            total = vals['debit'] - vals['credit']
+            if journal.refund_journal:
+                base_code = 'ref_base_code_id'
+                tax_code = 'ref_tax_code_id'
+                account_id = 'account_paid_id'
+                base_sign = 'ref_base_sign'
+                tax_sign = 'ref_tax_sign'
+            else:
+                base_code = 'base_code_id'
+                tax_code = 'tax_code_id'
+                account_id = 'account_collected_id'
+                base_sign = 'base_sign'
+                tax_sign = 'tax_sign'
+
+            tmp_cnt = 0
             for tax in tax_obj.compute(cr,uid,[tax_id],total,1.00):
-                self.write(cr, uid,[result], {
-                    'tax_code_id': tax['base_code_id'],
-                    'tax_amount': tax['base_sign'] * total
-                })
+                #create the base movement
+                if tmp_cnt == 0:
+                    if tax[base_code]:
+                        tmp_cnt += 1
+                        self.write(cr, uid,[result], {
+                            'tax_code_id': tax[base_code],
+                            'tax_amount': tax[base_sign] * abs(total)
+                        })
+                else:
+                    data = {
+                        'move_id': vals['move_id'],
+                        'journal_id': vals['journal_id'],
+                        'period_id': vals['period_id'],
+                        'name': tools.ustr(vals['name'] or '') + ' ' + tools.ustr(tax['name'] or ''),
+                        'date': vals['date'],
+                        'partner_id': vals.get('partner_id',False),
+                        'ref': vals.get('ref',False),
+                        'account_tax_id': False,
+                        'tax_code_id': tax[base_code],
+                        'tax_amount': tax[base_sign] * abs(total),
+                        'account_id': vals['account_id'],
+                        'credit': 0.0,
+                        'debit': 0.0,
+                    }
+                    if data['tax_code_id']:
+                        self.create(cr, uid, data, context)
+
+                #create the VAT movement
                 data = {
                     'move_id': vals['move_id'],
                     'journal_id': vals['journal_id'],
                     'period_id': vals['period_id'],
-                    'name': vals['name']+' '+tax['name'],
+                    'name': tools.ustr(vals['name'] or '') + ' ' + tools.ustr(tax['name'] or ''),
                     'date': vals['date'],
                     'partner_id': vals.get('partner_id',False),
                     'ref': vals.get('ref',False),
                     'account_tax_id': False,
-                    'tax_code_id': tax['tax_code_id'],
-                    'tax_amount': tax['tax_sign'] * tax['amount'],
-                    'account_id': tax['account_paid_id'], # or collected ?
-                    'credit': tax['amount']>0 and tax['amount'] or 0.0,
-                    'debit': tax['amount']<0 and -tax['amount'] or 0.0,
+                    'tax_code_id': tax[tax_code],
+                    'tax_amount': tax[tax_sign] * abs(tax['amount']),
+                    'account_id': tax[account_id] or vals['account_id'],
+                    'credit': tax['amount']<0 and -tax['amount'] or 0.0,
+                    'debit': tax['amount']>0 and tax['amount'] or 0.0,
                 }
-                self.create(cr, uid, data, context)
-        if check:
+                if data['tax_code_id']:
+                    self.create(cr, uid, data, context)
+            del vals['account_tax_id']
+
+        # No needed, related to the job
+        #if not is_new_move and 'date' in vals:
+        #    if context and ('__last_update' in context):
+        #        del context['__last_update']
+        #    self.pool.get('account.move').write(cr, uid, [move_id], {'date':vals['date']}, context=context)
+        if check and not context.get('no_store_function'):
             tmp = self.pool.get('account.move').validate(cr, uid, [vals['move_id']], context)
             if journal.entry_posted and tmp:
-                self.pool.get('account.move').write(cr,uid, [vals['move_id']],{'state':'posted'})
+                self.pool.get('account.move').button_validate(cr,uid, [vals['move_id']],context)
         return result
 account_move_line()