nojhan/weboob-devel
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

[vicsec] Handle new numbers notation. Closes #1956

Browse source
This commit is contained in:
Oleg Plakhotniuk 2015-05-30 15:46:46 -05:00 • committed by Romain Bignon
commit 9b24f55266
1 changed files with 9 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

13
modules/vicsec/browser.py
View file

@ -85,11 +85,13 @@ class OrderPage(VicSecPage):
for tr in self.doc.xpath('//tbody[@class="payment-summary"]' for tr in self.doc.xpath('//tbody[@class="payment-summary"]'
'//th[text()="Payment Summary"]/../../../tbody/tr'): '//th[text()="Payment Summary"]/../../../tbody/tr'):
method = tr.xpath('td[1]/text()')[0] method = tr.xpath('td[1]/text()')[0]
amount = tr.xpath('td[2]')[0].text_content().strip() amnode = tr.xpath('td[2]')[0]
amsign = -1 if amnode.xpath('em') else 1
amount = amnode.text_content().strip()
pmt = Payment() pmt = Payment()
pmt.date = self.order_date() pmt.date = self.order_date()
pmt.method = unicode(method) pmt.method = unicode(method)
pmt.amount = AmTr.decimal_amount(amount) pmt.amount = amsign * AmTr.decimal_amount(amount)
yield pmt yield pmt
def items(self): def items(self):
@ -137,10 +139,13 @@ class OrderPage(VicSecPage):
return dcnt + subt + rett - items return dcnt + subt + rett - items
def payment_part(self, which): def payment_part(self, which):
# The numbers notation on VS is super wierd.
# Sometimes negative amounts are represented by <em> element.
for node in self.doc.xpath('//tbody[@class="payment-summary"]' for node in self.doc.xpath('//tbody[@class="payment-summary"]'
'//td[contains(text(),"%s")]/../td[2]' % which): '//td[contains(text(),"%s")]/../td[2]' % which):
x = node.text_content().strip() strv = node.text_content().strip()
return Decimal(0) if x == u'FREE' else AmTr.decimal_amount(x) v = Decimal(0) if strv == u'FREE' else AmTr.decimal_amount(strv)
return -v if node.xpath('em') and v > 0 else v
return Decimal(0) return Decimal(0)