[16.0] [FIX] l10n_it_fatturapa_import_zip correctly imports all XML files in zip

[rlucia]

Corregge #3958

Con queste modifiche, dopo l'estrazione del contenuto dello zip file:

• non vengono presi in considerazione tutti i file estratti
• vengono presi in considerazione solo XML validi
• vengono presi in considerazione solo file con estensione .p7m (in questa fase mi è sembrato un overkill verificarne anche la correttezza formale)
• viene fatto uno scan di tutti i file anche nelle subdirectory
• aggiunto logging che informa quali file vengono importati e cosa viene scartato

E' stato modificato il file xml_import.zip utilizzato dai test, che ora contiene anche un file non xml e non p7m, ed alcuni xml in una subdirectory.

[SirAionTech]

Grazie della PR!
Ho fatto solo revisione del codice.

Puoi aggiungere un test? Così si evitano regressioni.

Puoi verificare come mai falliscono i test esistenti? Finché falliscono la PR non si può mergiare.

Per il messaggio del commit, puoi seguire https://github.com/OCA/odoo-community.org/blob/8afdb5a5ebfd3314e2fc836c9b9f644468ee044e/website/Contribution/CONTRIBUTING.rst#L1114?

[rlucia]

@SirAionTech grazie. Appena ho un attimo verifico le commit messages rules. Nel frattempo comunque edito il commento della PR.

Btw, il test falliva perché non avevo considerato l'import di file p7m.

Rifatto anche il file xml_import.zip che contiene un file che dovrebbe essere escluso e spostati alcuni file in subdirectory. Cosi' il test effettivamente verifica il codice modificato.

[SirAionTech]

@SirAionTech grazie. Appena ho un attimo verifico le commit messages rules. Nel frattempo comunque edito il commento della PR.

Btw, il test falliva perché non avevo considerato l'import di file p7m.

Rifatto anche il file xml_import.zip che contiene un file che dovrebbe essere escluso e spostati alcuni file in subdirectory. Cosi' il test effettivamente verifica il codice modificato.

Perfetto grazie, quando vuoi che aggiorni la revisione puoi richiederlo direttamente in Github seguendo https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/requesting-a-pull-request-review

[andreampiovesana]

LGTM

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

[TheMule71]

LGTM

[TheMule71]

/ocabot merge patch

[OCA-git-bot]

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-3959-by-TheMule71-bump-patch, awaiting test results.

[OCA-git-bot]

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