[FIX] l10n_it_intrastat: Permettere agli override di eseguire un'azione

[SirAionTech]

Risolve #3858 per 16.0.

[SirAionTech]

@primes2h @sergiocorato @matteoopenf visto che avete guardato la PR #3857 equivalente per 14.0, magari vi interessa anche questa?

[aleuffre]

Code review, LGTM.

[aleuffre]

Ci sono altre instanze in cui è stato discusso questo punto, anche recentemente, io sono molto a favore di restituire sempre il risultato di super() a meno che non ci siano ragioni forti per non farlo, ma non tutti sono d'accordo.

#4191 (comment)

Potrebbe forse essere discusso come linea guida generale?

[SirAionTech]

Ci sono altre instanze in cui è stato discusso questo punto, anche recentemente, io sono molto a favore di restituire sempre il risultato di super() a meno che non ci siano ragioni forti per non farlo, ma non tutti sono d'accordo.

#4191 (comment)

Potrebbe forse essere discusso come linea guida generale?

Sì, esiste la regola missing-return (vedi in https://github.com/OCA/pylint-odoo) di pylint, che c'è nel template OCA dei repo (https://github.com/OCA/oca-addons-repo-template/blob/f8cceacfd7c6594223087f0b3d67adcedaefb8be/src/.pylintrc-mandatory.jinja#L89) ma non è nel nostro repository, non so come mai e nel nostro repo (

l10n-italy/.pylintrc-mandatory

Line 77 in f3bbdcd

missing-return,

).

Non so come mai non abbia beccato questo pezzo di codice.
Da quel poco che ho capito di https://github.com/OCA/pylint-odoo/blob/35cc929edb224723d330c4422f8109ff1f6f3d13/src/pylint_odoo/checkers/odoo_addons.py#L1248-L1255, potrebbe essere stato fregato dal return True a fine metodo

[tafaRU]

Ci sono dei conflitti da sistemare. Per il resto 👍

[SirAionTech]

Fatto rebase per risolvere i conflitti

[SirAionTech]

/ocabot merge patch

[OCA-git-bot]

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-4311-by-SirAionTech-bump-patch, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at 5210010. Thanks a lot for contributing to OCA. ❤️