Merge lp:~vauxoo/openerp-venezuela-localization/ovl60-bug-1187089 into lp:openerp-venezuela-localization/6.0

Proposed by Katherine Zaoral (Vauxoo)
Status: Merged
Merged at revision: 692
Proposed branch: lp:~vauxoo/openerp-venezuela-localization/ovl60-bug-1187089
Merge into: lp:openerp-venezuela-localization/6.0
Diff against target: 123 lines (+13/-40)
4 files modified
l10n_ve_fiscal_requirements/partner.py (+6/-0)
l10n_ve_fiscal_requirements/seniat_url.py (+6/-14)
l10n_ve_fiscal_requirements/wizard/search_info_partner_seniat.py (+1/-3)
l10n_ve_withholding_iva/model/partner.py (+0/-23)
To merge this branch: bzr merge lp:~vauxoo/openerp-venezuela-localization/ovl60-bug-1187089
Reviewer Review Type Date Requested Status
hbto [Vauxoo] http://www.vauxoo.com Approve
Review via email: mp+167360@code.launchpad.net

This proposal supersedes a proposal from 2013-06-04.

Description of the change

To post a comment you must log in.
Revision history for this message
hbto [Vauxoo] http://www.vauxoo.com (humbertoarocha) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'l10n_ve_fiscal_requirements/partner.py'
2--- l10n_ve_fiscal_requirements/partner.py 2012-07-16 18:55:40 +0000
3+++ l10n_ve_fiscal_requirements/partner.py 2013-06-04 18:10:38 +0000
4@@ -32,6 +32,7 @@
5
6 from osv import fields, osv
7 from tools.translate import _
8+import decimal_precision as dp
9 import re
10
11 class res_partner_address(osv.osv):
12@@ -65,6 +66,11 @@
13
14 _columns = {
15 'seniat_updated': fields.boolean('Seniat Updated', help="This field indicates if partner was updated using SENIAT button"),
16+ 'wh_iva_agent': fields.boolean('Wh. Agent', help="Indicate if the partner is a withholding vat agent"),
17+ 'wh_iva_rate': fields.float(
18+ string='Rate',
19+ digits_compute=dp.get_precision('Withhold'),
20+ help="Withholding vat rate"),
21 }
22
23 _default = {
24
25=== modified file 'l10n_ve_fiscal_requirements/seniat_url.py'
26--- l10n_ve_fiscal_requirements/seniat_url.py 2012-07-16 18:55:40 +0000
27+++ l10n_ve_fiscal_requirements/seniat_url.py 2013-06-04 18:10:38 +0000
28@@ -64,31 +64,23 @@
29 pass
30 retries -= 1
31 return str_error
32-
33- def _buscar_porcentaje(self,rif,url):
34- context={}
35- html_data = self.pool.get('seniat.url')._load_url(3,url %rif)
36- html_data = unicode(html_data, 'ISO-8859-1').encode('utf-8')
37- search_str='La condición de este contribuyente requiere la retención del '
38- pos = html_data.find(search_str)
39- if pos > 0:
40- pos += len(search_str)
41- pct = html_data[pos:pos+4].replace('%','').replace(' ','')
42- return float(pct)
43- else:
44- return 0.0
45
46 def _parse_dom(self,dom,rif,url_seniat,context={}):
47 rif_aux = dom.childNodes[0].getAttribute('rif:numeroRif')
48 name = dom.childNodes[0].childNodes[0].firstChild.data
49 wh_agent = dom.childNodes[0].childNodes[1].firstChild.data.upper()=='SI' and True or False
50 vat_subjected = dom.childNodes[0].childNodes[2].firstChild.data.upper()=='SI' and True or False
51+ wh_rate = dom.childNodes[0].childNodes[3].firstChild.data
52 self.logger.notifyChannel("info", netsvc.LOG_INFO,
53 "RIF: %s Found" % rif)
54 if name.count('(') > 0:
55 name = name[:name.index('(')].rstrip()
56 if context.get('spf_info'):
57- res= {'name': name,'vat_subjected': vat_subjected,'vat':'VE'+rif_aux,'wh_iva_agent':wh_agent}
58+ res= {'name': name,
59+ 'vat_subjected': vat_subjected,
60+ 'vat': 'VE'+rif_aux,
61+ 'wh_iva_rate': wh_rate,
62+ 'wh_iva_agent': wh_agent}
63 else:
64 res= {'name': name,'vat_subjected': vat_subjected,'vat':'VE'+rif_aux}
65 return res
66
67=== modified file 'l10n_ve_fiscal_requirements/wizard/search_info_partner_seniat.py'
68--- l10n_ve_fiscal_requirements/wizard/search_info_partner_seniat.py 2012-07-16 18:55:40 +0000
69+++ l10n_ve_fiscal_requirements/wizard/search_info_partner_seniat.py 2013-06-04 18:10:38 +0000
70@@ -52,9 +52,7 @@
71 if rp_obj.check_vat_ve(aux,context):
72 context.update({'spf_info':True})
73 res = su_obj._dom_giver(url1,url2,url3, aux,context)
74- if res:
75- res.update({'wh_iva_rate':su_obj._buscar_porcentaje(aux,url2)})
76- else:
77+ if not res:
78 raise osv.except_osv(_('Error'),_("Does not exist the contributor requested"))
79 self.write(cr,uid,vat,res)
80
81
82=== modified file 'l10n_ve_withholding_iva/model/partner.py'
83--- l10n_ve_withholding_iva/model/partner.py 2012-07-25 19:19:14 +0000
84+++ l10n_ve_withholding_iva/model/partner.py 2013-06-04 18:10:38 +0000
85@@ -23,19 +23,11 @@
86 # along with this program. If not, see <http://www.gnu.org/licenses/>.
87 ################################################################################
88 from osv import fields, osv
89-import decimal_precision as dp
90-from tools.translate import _
91-import urllib
92-from xml.dom.minidom import parseString
93 import netsvc
94
95 class res_partner(osv.osv):
96 _inherit = 'res.partner'
97 logger = netsvc.Logger()
98- _columns = {
99- 'wh_iva_agent': fields.boolean('Wh. Agent', help="Indicate if the partner is a withholding vat agent"),
100- 'wh_iva_rate': fields.float(string='Rate', digits_compute= dp.get_precision('Withhold'), help="Withholding vat rate"),
101- }
102 _defaults = {
103 'wh_iva_rate': lambda *a: 0,
104 }
105@@ -47,18 +39,3 @@
106 return su_obj.update_rif(cr, uid, ids, context=context)
107
108 res_partner()
109-
110-class seniat_url(osv.osv):
111-
112- _inherit = 'seniat.url'
113-
114- def _parse_dom(self,dom,rif,url_seniat,context=None):
115- su_obj = self.pool.get('seniat.url')
116- wh_agent = dom.childNodes[0].childNodes[1].firstChild.data.upper()=='SI' and True or False
117- wh_rate = su_obj._buscar_porcentaje(rif,url_seniat)
118- self.logger.notifyChannel("info", netsvc.LOG_INFO,
119- "RIF: %s Found" % rif)
120- data = {'wh_iva_agent':wh_agent,'wh_iva_rate':wh_rate}
121- return dict(data.items() + super(seniat_url,self)._parse_dom(dom,rif,url_seniat,context=context).items())
122-
123-seniat_url()

Subscribers

People subscribed via source and target branches