Merge lp:~openbig/bigconsulting/milestone2_manually_entries into lp:bigconsulting

Proposed by gpa(OpenERP)
Status: Merged
Merged at revision: 62
Proposed branch: lp:~openbig/bigconsulting/milestone2_manually_entries
Merge into: lp:bigconsulting
Diff against target: 182 lines (+85/-10)
2 files modified
account_invoice_cash_discount/account_invoice_cash_discount.py (+46/-10)
account_invoice_cash_discount/account_invoice_cash_discount_view.xml (+39/-0)
To merge this branch: bzr merge lp:~openbig/bigconsulting/milestone2_manually_entries
Reviewer Review Type Date Requested Status
openbig Pending
Review via email: mp+31821@code.launchpad.net

This proposal supersedes a proposal from 2010-08-05.

Description of the change

changes added for milestone2 for manually entries

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'account_invoice_cash_discount/account_invoice_cash_discount.py'
2--- account_invoice_cash_discount/account_invoice_cash_discount.py 2010-08-03 12:11:28 +0000
3+++ account_invoice_cash_discount/account_invoice_cash_discount.py 2010-08-05 07:58:42 +0000
4@@ -529,6 +529,7 @@
5
6 def button_confirm(self, cr, uid, ids, context={}):
7 done = []
8+ tax_obj = self.pool.get('account.tax')
9 res_currency_obj = self.pool.get('res.currency')
10 res_users_obj = self.pool.get('res.users')
11 account_move_obj = self.pool.get('account.move')
12@@ -579,10 +580,15 @@
13 amount = res_currency_obj.compute(cr, uid, st.currency.id,
14 company_currency_id, move.amount, context=context,
15 account=acc_cur)
16-# if move.reconcile_id and move.reconcile_id.line_new_ids:
17-# for newline in move.reconcile_id.line_new_ids:
18-# amount += newline.amount
19
20+ st_tax_amount = 0.00
21+ st_base_code_id = False
22+
23+ if move.tax_id:
24+ tax = tax_obj.compute(cr, uid, [move.tax_id], move.amount, 1.00)[0]
25+ st_tax_amount = tax['amount']
26+ st_base_code_id = tax['base_code_id']
27+
28 val = {
29 'name': move.name,
30 'date': move.date,
31@@ -596,8 +602,10 @@
32 'journal_id': st.journal_id.id,
33 'period_id': st.period_id.id,
34 'currency_id': st.currency.id,
35+ 'tax_code_id':st_base_code_id,
36+ 'account_tax_id':move.tax_id.id,
37+ 'tax_amount':move.amount,
38 }
39-
40 amount = res_currency_obj.compute(cr, uid, st.currency.id,
41 company_currency_id, move.amount, context=context,
42 account=acc_cur)
43@@ -617,7 +625,8 @@
44 account=acc_cur)
45 val['amount_currency'] = amount_cur
46
47- torec.append(account_move_line_obj.create(cr, uid, val , context=context))
48+ torec.append(account_move_line_obj.create(cr, uid, val, context=context))
49+
50 newline_sum = 0.0
51 if move.reconcile_id and move.reconcile_id.line_new_ids:
52 for newline in move.reconcile_id.line_new_ids:
53@@ -723,7 +732,7 @@
54 'amount_currency': amount_currency,
55 'currency_id': currency_id,
56 }, context=context)
57-
58+
59 account_move_line_obj.create(cr, uid, {
60 'name': move.name,
61 'date': move.date,
62@@ -731,8 +740,8 @@
63 'move_id': move_id,
64 'partner_id': ((move.partner_id) and move.partner_id.id) or False,
65 'account_id': account_id,
66- 'credit': (((amount-newline_sum) < 0) and -(amount-newline_sum)) or 0.0,
67- 'debit': (((amount-newline_sum) > 0) and (amount-newline_sum)) or 0.0,
68+ 'credit': (((amount+st_tax_amount-newline_sum) < 0) and -(amount+st_tax_amount-newline_sum)) or 0.0,
69+ 'debit': (((amount+st_tax_amount-newline_sum) > 0) and (amount+st_tax_amount-newline_sum)) or 0.0,
70 'statement_id': st.id,
71 'journal_id': st.journal_id.id,
72 'period_id': st.period_id.id,
73@@ -740,10 +749,15 @@
74 'currency_id': currency_id,
75 }, context=context)
76
77+ account_move_line_obj.write(cr, uid, [x.id for x in
78+ account_move_obj.browse(cr, uid, move_id,
79+ context=context).line_id], {'statement_id': st.id,})
80+
81 for line in account_move_line_obj.browse(cr, uid, [x.id for x in
82 account_move_obj.browse(cr, uid, move_id,
83 context=context).line_id],
84 context=context):
85+
86 if line.state <> 'valid':
87 raise osv.except_osv(_('Error !'),
88 _('Account move line "%s" is not valid') % line.name)
89@@ -752,13 +766,11 @@
90 torec += map(lambda x: x.id, move.reconcile_id.line_ids)
91 #try:
92 if abs(move.reconcile_amount-move.amount)<0.0001:
93-
94 writeoff_acc_id = False
95 #There should only be one write-off account!
96 for entry in move.reconcile_id.line_new_ids:
97 writeoff_acc_id = entry.account_id.id
98 break
99-
100 account_move_line_obj.reconcile(cr, uid, torec, 'statement', writeoff_acc_id=writeoff_acc_id, writeoff_period_id=st.period_id.id, writeoff_journal_id=st.journal_id.id, context=context)
101 else:
102 account_move_line_obj.reconcile_partial(cr, uid, torec, 'statement', context)
103@@ -774,6 +786,30 @@
104
105 account_bank_statement()
106
107+class account_bank_statement_line(osv.osv):
108+ _inherit="account.bank.statement.line"
109+
110+ def _get_account(self, cr, uid, context={}):
111+ account_id = self.pool.get('account.account').search(cr, uid, [('type','!=','view')])[0]
112+ if account_id:
113+ return account_id
114+ else:
115+ return False
116+
117+ def _get_tax(self, cr, uid, context={}):
118+ tax_id = self.pool.get('account.tax').search(cr, uid, [])
119+ if tax_id:
120+ return tax_id[0]
121+ else:
122+ return False
123+ _columns = {
124+ 'tax_id': fields.many2one('account.tax', 'Tax'),
125+ }
126+ _defaults = {
127+ 'account_id': _get_account,
128+ 'tax_id': _get_tax,
129+ }
130+account_bank_statement_line()
131
132 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
133
134
135=== modified file 'account_invoice_cash_discount/account_invoice_cash_discount_view.xml'
136--- account_invoice_cash_discount/account_invoice_cash_discount_view.xml 2010-08-03 12:11:28 +0000
137+++ account_invoice_cash_discount/account_invoice_cash_discount_view.xml 2010-08-05 07:58:42 +0000
138@@ -162,5 +162,44 @@
139 <field name="action_id" ref="action_res_company_wizard"/>
140 </record>
141
142+ <record id="view_move_line_tree1" model="ir.ui.view">
143+ <field name="name">account.move.line.tree</field>
144+ <field name="model">account.move.line</field>
145+ <field name="type">tree</field>
146+ <field eval="1" name="priority"/>
147+ <field name="inherit_id" ref="account.view_move_line_tree"/>
148+ <field name="arch" type="xml">
149+ <field name="account_tax_id" position="after">
150+ <field name="tax_code_id"/>
151+ <field name="tax_amount"/>
152+ </field>
153+ </field>
154+ </record>
155+
156+ <record model="ir.ui.view" id="view_bank_statement_form111">
157+ <field name="name">account.bank.statement.form1</field>
158+ <field name="model">account.bank.statement</field>
159+ <field name="inherit_id" ref="account.view_bank_statement_form" />
160+ <field name="type">form</field>
161+ <field name="arch" type="xml">
162+ <xpath expr="//field[@name='line_ids']/form/field[@name='ref']" position="after">
163+ <field name="tax_id"/>
164+ </xpath>
165+ </field>
166+ </record>
167+
168+ <record model="ir.ui.view" id="view_bank_statement_form222">
169+ <field name="name">account.bank.statement.tree1</field>
170+ <field name="model">account.bank.statement</field>
171+ <field name="inherit_id" ref="account.view_bank_statement_form" />
172+ <field name="type">form</field>
173+ <field name="arch" type="xml">
174+ <xpath expr="//field[@name='line_ids']/tree/field[@name='reconcile_amount']" position="after">
175+ <field name="tax_id"/>
176+ </xpath>
177+ </field>
178+ </record>
179+
180+
181 </data>
182 </openerp>

Subscribers

People subscribed via source and target branches