[12.0] [IMP] l10n_it_fiscalcode: add codicefiscale.isvalid()

[odooNextev]

#3719

[primes2h]

/ocabot rebase

[OCA-git-bot]

Congratulations, PR rebased to 12.0.

[primes2h]

/ocabot rebase

[OCA-git-bot]

Congratulations, PR rebased to 12.0.

[primes2h]

@odooNextev
Test ❌ su un codice fiscale, sembra che questa PR funzioni a dovere. :-)

[odooNextev]

Abbiamo corretto il test di l10n_it_fatturapa_in che impostava sul partner una stringa che non era un CF valido

[odooNextev]

@OCA/local-italy-developers si può mergiare?

[SirAionTech]

@OCA/local-italy-developers si può mergiare?

Serve almeno una revisione.
@primes2h questa #3724 (comment) potrebbe essere una revisione?

[odooNextev]

@OCA/local-italy-developers si può mergiare?

Serve almeno una revisione. @primes2h questa #3724 (comment) potrebbe essere una revisione?

Hai ragione, non avevo notato.
Per testarlo basta inserire un codice fiscale non valido non solo come numero di caratteri (quello c'era già senza questa PR), ma anche come formato (lettere dove ci dovrebbero essere numeri o viceversa).

[primes2h]

@OCA/local-italy-developers si può mergiare?

Serve almeno una revisione. @primes2h questa #3724 (comment) potrebbe essere una revisione?

Ops, visto ora. 😅
Attendiamo solo #3724 (comment), poi possiamo procedere con il merge.

[odooNextev]

@primes2h dovremmo aver risolto

[primes2h]

@odooNextev
Dal test su runboat ho notato un comportamento anomalo.
Se parto da un partner di tipo azienda che contiene un CF non corretto e cambio la tipologia in persona fisica, al salvataggio non viene eseguito alcun controllo.

Ho visto però che il problema era presente anche prima di questa PR, quindi possiamo comunque procedere con il merge.

È giusto però segnalare la cosa, casomai qualcuno volesse darci un'occhiata.

[primes2h]

/ocabot merge patch

[OCA-git-bot]

On my way to merge this fine PR!
Prepared branch 12.0-ocabot-merge-pr-3724-by-primes2h-bump-patch, awaiting test results.

[SirAionTech]

@odooNextev Dal test su runboat ho notato un comportamento anomalo. Se parto da un partner di tipo azienda che contiene un CF non corretto e cambio la tipologia in persona fisica, al salvataggio non viene eseguito alcun controllo.

Ho visto però che il problema era presente anche prima di questa PR, quindi possiamo comunque procedere con il merge.

È giusto però segnalare la cosa, casomai qualcuno volesse darci un'occhiata.

Puoi aprire una issue (https://github.com/OCA/l10n-italy/wiki/Team-di-sviluppo#apertura-issue)? Sarà comunque difficile che qualcuno si metta a correggerlo, ma almeno ha più visibilità di un commento in una PR

[OCA-git-bot]

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

[primes2h]

@odooNextev Dal test su runboat ho notato un comportamento anomalo. Se parto da un partner di tipo azienda che contiene un CF non corretto e cambio la tipologia in persona fisica, al salvataggio non viene eseguito alcun controllo.
Ho visto però che il problema era presente anche prima di questa PR, quindi possiamo comunque procedere con il merge.
È giusto però segnalare la cosa, casomai qualcuno volesse darci un'occhiata.

Puoi aprire una issue (https://github.com/OCA/l10n-italy/wiki/Team-di-sviluppo#apertura-issue)? Sarà comunque difficile che qualcuno si metta a correggerlo, ma almeno ha più visibilità di un commento in una PR

Fatto #4050