audrius/bank-statement-import
1
0
Fork 1
mirror of https://github.com/OCA/bank-statement-import.git synced 2025-01-20 12:37:43 +02:00
Code Issues Packages Projects Releases Wiki Activity

[IMP] *_online_ponto: code organisation

Browse Source 
Check situation that should raise exception immediately, prevent ident on normal flow
This commit is contained in:
Ronald Portier
2021-12-13 21:53:20 +01:00
parent 1afbd2169b
commit 4a8f9acaae
1 changed files with 22 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
account_statement_import_online_ponto/models/online_bank_statement_provider_ponto.py
View File

@@ -156,7 +156,11 @@ class OnlineBankStatementProviderPonto(models.Model):
response = requests.get(
page_url, params=params, headers=self._ponto_header()
)
if response.status_code == 200:
if response.status_code != 200:
raise UserError(
_("Error during get transaction.\n\n%s \n\n %s")
% (response.status_code, response.text)
)
if params.get("before"):
params.pop("before")
data = json.loads(response.text)
@@ -174,16 +178,10 @@ class OnlineBankStatementProviderPonto(models.Model):
)
if date_since <= date < date_until:
current_transactions.append(transaction)
if current_transactions:
if not page_next or (page_next and not latest_identifier):
latest_identifier = current_transactions[0].get("id")
transaction_lines.extend(current_transactions)
else:
raise UserError(
_("Error during get transaction.\n\n%s \n\n %s")
% (response.status_code, response.text)
)
if latest_identifier:
self.ponto_last_identifier = latest_identifier
return transaction_lines