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
=== modified file 'l10n_be/account_pcmn_belgium.xml'
--- l10n_be/account_pcmn_belgium.xml 2011-01-14 00:11:01 +0000
+++ l10n_be/account_pcmn_belgium.xml 2011-03-31 05:46:34 +0000
@@ -6479,18 +6479,8 @@
6479 </record>6479 </record>
6480 <record id="vat_code_a44" model="account.tax.code.template">6480 <record id="vat_code_a44" model="account.tax.code.template">
6481 <field name="code">44</field>6481 <field name="code">44</field>
6482 <field name="name">Services intra-communautaires et ABC</field>
6483 <field name="parent_id" ref="vat_code_out"/>
6484 </record>
6485 <record id="vat_code_a44a" model="account.tax.code.template">
6486 <field name="code">44a</field>
6487 <field name="name">Services intra-communautaires</field>6482 <field name="name">Services intra-communautaires</field>
6488 <field name="parent_id" ref="vat_code_a44"/>6483 <field name="parent_id" ref="vat_code_out"/>
6489 </record>
6490 <record id="vat_code_a44b" model="account.tax.code.template">
6491 <field name="code">44b</field>
6492 <field name="name">ABC intra-communautaires</field>
6493 <field name="parent_id" ref="vat_code_a44"/>
6494 </record>6484 </record>
64956485
6496 <record id="vat_code_a45" model="account.tax.code.template">6486 <record id="vat_code_a45" model="account.tax.code.template">
@@ -6498,11 +6488,21 @@
6498 <field name="name">Operations avec TVA due par le cocontractant</field>6488 <field name="name">Operations avec TVA due par le cocontractant</field>
6499 <field name="parent_id" ref="vat_code_out"/>6489 <field name="parent_id" ref="vat_code_out"/>
6500 </record>6490 </record>
6501 <record id="vat_code_a46" model="account.tax.code.template">6491 <record id="vat_code_a46" model="account.tax.code.template">
6502 <field name="code">46</field>6492 <field name="code">46</field>
6503 <field name="name">Livraisons intra-communautaire exemptees</field>6493 <field name="name">Livraisons intra-communautaire et ABC</field>
6504 <field name="parent_id" ref="vat_code_out"/>6494 <field name="parent_id" ref="vat_code_out"/>
6505 </record>6495 </record>
6496 <record id="vat_code_a46a" model="account.tax.code.template">
6497 <field name="code">46a</field>
6498 <field name="name">Livraisons intra-communautaire</field>
6499 <field name="parent_id" ref="vat_code_a44"/>
6500 </record>
6501 <record id="vat_code_a46b" model="account.tax.code.template">
6502 <field name="code">46b</field>
6503 <field name="name">Intra-communautaire ABC transactions</field>
6504 <field name="parent_id" ref="vat_code_a44"/>
6505 </record>
6506 <record id="vat_code_a47" model="account.tax.code.template">6506 <record id="vat_code_a47" model="account.tax.code.template">
6507 <field name="code">47</field>6507 <field name="code">47</field>
6508 <field name="name">Autres operations exemptees</field>6508 <field name="name">Autres operations exemptees</field>
65096509
=== modified file 'l10n_be/wizard/l10n_be_vat_intra.py'
--- l10n_be/wizard/l10n_be_vat_intra.py 2011-01-14 00:11:01 +0000
+++ l10n_be/wizard/l10n_be_vat_intra.py 2011-03-31 05:46:34 +0000
@@ -127,7 +127,7 @@
127 FROM account_move_line l127 FROM account_move_line l
128 LEFT JOIN account_tax_code t ON (l.tax_code_id = t.id)128 LEFT JOIN account_tax_code t ON (l.tax_code_id = t.id)
129 LEFT JOIN res_partner p ON (l.partner_id = p.id)129 LEFT JOIN res_partner p ON (l.partner_id = p.id)
130 WHERE t.code IN ('44a','44b','88')130 WHERE t.code IN ('44','46b','46a')
131 AND l.period_id IN %s131 AND l.period_id IN %s
132 GROUP BY p.name, l.partner_id, p.vat, t.code''', (tuple(data['period_ids']), ))132 GROUP BY p.name, l.partner_id, p.vat, t.code''', (tuple(data['period_ids']), ))
133 for row in cursor.dictfetchall():133 for row in cursor.dictfetchall():
@@ -138,7 +138,7 @@
138 amt = row['amount'] or 0138 amt = row['amount'] or 0
139 amt = int(round(amt * 100))139 amt = int(round(amt * 100))
140 amount_sum += amt140 amount_sum += amt
141 intra_code = row['intra_code'] == '88' and 'L' or (row['intra_code'] == '44b' and 'T' or (row['intra_code'] == '44a' and 'S' or ''))141 intra_code = row['intra_code'] == '46a' and 'L' or (row['intra_code'] == '46b' and 'T' or (row['intra_code'] == '44' and 'S' or ''))
142 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>'142 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>'
143 amount_sum = int(amount_sum)143 amount_sum = int(amount_sum)
144 data_decl = '\n\t<DeclarantList SequenceNum="1" DeclarantNum="'+ dnum + '" ClientNbr="'+ str(seq) +'" AmountSum="'+ str(amount_sum) +'" >'144 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: