Skip to content

Commit

Permalink
[IMP] l10n_es_aeat_verifactu: pre-commit stuff
Browse files Browse the repository at this point in the history
  • Loading branch information
aritzolea committed Feb 25, 2025
1 parent e2eef3f commit f2a74cd
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 15 deletions.
16 changes: 5 additions & 11 deletions l10n_es_aeat_verifactu/models/account_move.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,15 +2,13 @@
# Copyright 2024 Aures Tic - Jose Zambudio <[email protected]>
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
import logging
from collections import OrderedDict
from time import sleep

from collections import OrderedDict
from datetime import datetime
from hashlib import sha256
from psycopg2 import OperationalError
from time import sleep

import pytz
from psycopg2 import OperationalError

from odoo import _, api, fields, models
from odoo.exceptions import UserError
Expand Down Expand Up @@ -449,12 +447,12 @@ def _post(self, soft=True):
try:
record._set_chaining_invoice()
break
except OperationalError:
except OperationalError as oe:

Check warning on line 450 in l10n_es_aeat_verifactu/models/account_move.py

View check run for this annotation

Codecov / codecov/patch

l10n_es_aeat_verifactu/models/account_move.py#L450

Added line #L450 was not covered by tests
if attempt == SEND_TO_VERIFACTU_MAX_RETRIES - 1:
raise OperationalError(

Check warning on line 452 in l10n_es_aeat_verifactu/models/account_move.py

View check run for this annotation

Codecov / codecov/patch

l10n_es_aeat_verifactu/models/account_move.py#L452

Added line #L452 was not covered by tests
"Failed to chain invoice %s for Verifactu",
record.name,
)
) from oe
else:
sleep(1) # Wait 1 second before next try

Check warning on line 457 in l10n_es_aeat_verifactu/models/account_move.py

View check run for this annotation

Codecov / codecov/patch

l10n_es_aeat_verifactu/models/account_move.py#L457

Added line #L457 was not covered by tests
verifactu_hash_values = record._get_verifactu_hash_string()
Expand All @@ -469,11 +467,7 @@ def _post(self, soft=True):
return res

def _is_verifactu_invoice(self, invoice):
return (
invoice.exists()
and invoice.is_invoice()
and invoice.verifactu_enabled
)
return invoice.exists() and invoice.is_invoice() and invoice.verifactu_enabled

def _should_send_to_verifactu(self, invoice):
return (
Expand Down
11 changes: 7 additions & 4 deletions l10n_es_aeat_verifactu/static/description/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,11 @@

/*
:Author: David Goodger ([email protected])
:Id: $Id: html4css1.css 8954 2022-01-20 10:10:25Z milde $
:Id: $Id: html4css1.css 9511 2024-01-13 09:50:07Z milde $
:Copyright: This stylesheet has been placed in the public domain.
Default cascading style sheet for the HTML output of Docutils.
Despite the name, some widely supported CSS2 features are used.
See https://docutils.sourceforge.io/docs/howto/html-stylesheets.html for how to
customize this style sheet.
Expand Down Expand Up @@ -274,7 +275,7 @@
margin-left: 2em ;
margin-right: 2em }

pre.code .ln { color: grey; } /* line numbers */
pre.code .ln { color: gray; } /* line numbers */
pre.code, code { background-color: #eeeeee }
pre.code .comment, code .comment { color: #5C6576 }
pre.code .keyword, code .keyword { color: #3B0D06; font-weight: bold }
Expand All @@ -300,7 +301,7 @@
span.pre {
white-space: pre }

span.problematic {
span.problematic, pre.problematic {
color: red }

span.section-subtitle {
Expand Down Expand Up @@ -486,7 +487,9 @@ <h2><a class="toc-backref" href="#toc-entry-7">Contributors</a></h2>
<div class="section" id="maintainers">
<h2><a class="toc-backref" href="#toc-entry-8">Maintainers</a></h2>
<p>This module is maintained by the OCA.</p>
<a class="reference external image-reference" href="https://odoo-community.org"><img alt="Odoo Community Association" src="https://odoo-community.org/logo.png" /></a>
<a class="reference external image-reference" href="https://odoo-community.org">
<img alt="Odoo Community Association" src="https://odoo-community.org/logo.png" />
</a>
<p>OCA, or the Odoo Community Association, is a nonprofit organization whose
mission is to support the collaborative development of Odoo features and
promote its widespread use.</p>
Expand Down
2 changes: 2 additions & 0 deletions requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,10 @@ deepdiff<8
pycountry
pycryptodome
qrcode
requests
requests_pkcs12==1.22
suds-py3
unidecode
xmlsig
xmltodict
zeep

0 comments on commit f2a74cd

Please sign in to comment.