Merge lp:~openerp-dev/openobject-addons/trunk-bug-730885-skh into lp:openobject-addons

Proposed by Somesh Khare
Status: Rejected
Rejected by: qdp (OpenERP)
Proposed branch: lp:~openerp-dev/openobject-addons/trunk-bug-730885-skh
Merge into: lp:openobject-addons
Diff against target: 69 lines (+15/-15)
2 files modified
l10n_be/account_pcmn_belgium.xml (+13/-13)
l10n_be/wizard/l10n_be_vat_intra.py (+2/-2)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/trunk-bug-730885-skh
Reviewer Review Type Date Requested Status
qdp (OpenERP) Needs Information
Somesh Khare (community) Needs Resubmitting
Mustufa Rangwala (Open ERP) (community) Approve
Review via email: mp+55318@code.launchpad.net

Description of the change

Hello,

Fix : l10n_be: vat_intra report: used tax code are wrong.

Thanks,
skh

To post a comment you must log in.
4608. By Somesh Khare

[FIX] l10n_be: vat_intra report: used tax code are wrong

Revision history for this message
Somesh Khare (somesh.khare) wrote :

Hello,

[Fix] : I have made the changes according to the last comment.

Thanks,
skh

review: Needs Resubmitting
4609. By Floris

[FIX] l10n_be: vat_intra report: used tax code are wrong

Revision history for this message
Somesh Khare (somesh.khare) wrote :

Hello,

[Fix] : I have made the changes in the account_pcmn_belgium.xml according to the last comment.

Thanks,
skh

review: Needs Resubmitting
Revision history for this message
Mustufa Rangwala (Open ERP) (mra-tinyerp) :
review: Approve
4610. By Floris

[FIX] l10n_be: vat_intra report: used tax code are wrong

Revision history for this message
Somesh Khare (somesh.khare) wrote :

Hello,

[Fix] : These strings need to be verified by qdp.

Thanks,
skh

review: Needs Resubmitting
Revision history for this message
qdp (OpenERP) (qdp) wrote :

i should already have fixed that 2 weeks ago, so i'm rejecting this merge prop for the sake of clarity in LP. Please resubmit if the bug is confirmed again.

Thanks

review: Needs Information

Unmerged revisions

4610. By Floris

[FIX] l10n_be: vat_intra report: used tax code are wrong

4609. By Floris

[FIX] l10n_be: vat_intra report: used tax code are wrong

4608. By Somesh Khare

[FIX] l10n_be: vat_intra report: used tax code are wrong

4607. By Somesh Khare

[FIX] l10n_be: vat_intra report: used tax code are wrong

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'l10n_be/account_pcmn_belgium.xml'
2--- l10n_be/account_pcmn_belgium.xml 2011-01-14 00:11:01 +0000
3+++ l10n_be/account_pcmn_belgium.xml 2011-03-31 05:46:34 +0000
4@@ -6479,18 +6479,8 @@
5 </record>
6 <record id="vat_code_a44" model="account.tax.code.template">
7 <field name="code">44</field>
8- <field name="name">Services intra-communautaires et ABC</field>
9- <field name="parent_id" ref="vat_code_out"/>
10- </record>
11- <record id="vat_code_a44a" model="account.tax.code.template">
12- <field name="code">44a</field>
13 <field name="name">Services intra-communautaires</field>
14- <field name="parent_id" ref="vat_code_a44"/>
15- </record>
16- <record id="vat_code_a44b" model="account.tax.code.template">
17- <field name="code">44b</field>
18- <field name="name">ABC intra-communautaires</field>
19- <field name="parent_id" ref="vat_code_a44"/>
20+ <field name="parent_id" ref="vat_code_out"/>
21 </record>
22
23 <record id="vat_code_a45" model="account.tax.code.template">
24@@ -6498,11 +6488,21 @@
25 <field name="name">Operations avec TVA due par le cocontractant</field>
26 <field name="parent_id" ref="vat_code_out"/>
27 </record>
28- <record id="vat_code_a46" model="account.tax.code.template">
29+ <record id="vat_code_a46" model="account.tax.code.template">
30 <field name="code">46</field>
31- <field name="name">Livraisons intra-communautaire exemptees</field>
32+ <field name="name">Livraisons intra-communautaire et ABC</field>
33 <field name="parent_id" ref="vat_code_out"/>
34 </record>
35+ <record id="vat_code_a46a" model="account.tax.code.template">
36+ <field name="code">46a</field>
37+ <field name="name">Livraisons intra-communautaire</field>
38+ <field name="parent_id" ref="vat_code_a44"/>
39+ </record>
40+ <record id="vat_code_a46b" model="account.tax.code.template">
41+ <field name="code">46b</field>
42+ <field name="name">Intra-communautaire ABC transactions</field>
43+ <field name="parent_id" ref="vat_code_a44"/>
44+ </record>
45 <record id="vat_code_a47" model="account.tax.code.template">
46 <field name="code">47</field>
47 <field name="name">Autres operations exemptees</field>
48
49=== modified file 'l10n_be/wizard/l10n_be_vat_intra.py'
50--- l10n_be/wizard/l10n_be_vat_intra.py 2011-01-14 00:11:01 +0000
51+++ l10n_be/wizard/l10n_be_vat_intra.py 2011-03-31 05:46:34 +0000
52@@ -127,7 +127,7 @@
53 FROM account_move_line l
54 LEFT JOIN account_tax_code t ON (l.tax_code_id = t.id)
55 LEFT JOIN res_partner p ON (l.partner_id = p.id)
56- WHERE t.code IN ('44a','44b','88')
57+ WHERE t.code IN ('44','46b','46a')
58 AND l.period_id IN %s
59 GROUP BY p.name, l.partner_id, p.vat, t.code''', (tuple(data['period_ids']), ))
60 for row in cursor.dictfetchall():
61@@ -138,7 +138,7 @@
62 amt = row['amount'] or 0
63 amt = int(round(amt * 100))
64 amount_sum += amt
65- intra_code = row['intra_code'] == '88' and 'L' or (row['intra_code'] == '44b' and 'T' or (row['intra_code'] == '44a' and 'S' or ''))
66+ intra_code = row['intra_code'] == '46a' and 'L' or (row['intra_code'] == '46b' and 'T' or (row['intra_code'] == '44' and 'S' or ''))
67 data_clientinfo +='\n\t\t<ClientList SequenceNum="'+str(seq)+'">\n\t\t\t<CompanyInfo>\n\t\t\t\t<VATNum>'+row['vat'][2:] +'</VATNum>\n\t\t\t\t<Country>'+row['vat'][:2] +'</Country>\n\t\t\t</CompanyInfo>\n\t\t\t<Amount>'+str(amt) +'</Amount>\n\t\t\t<Code>'+str(intra_code) +'</Code>\n\t\t</ClientList>'
68 amount_sum = int(amount_sum)
69 data_decl = '\n\t<DeclarantList SequenceNum="1" DeclarantNum="'+ dnum + '" ClientNbr="'+ str(seq) +'" AmountSum="'+ str(amount_sum) +'" >'

Subscribers

People subscribed via source and target branches

to all changes: