Conversation
|
Hey @gigidn, thank you for your Pull Request. It looks like some users haven't signed our Contributor License Agreement, yet.
Appreciation of efforts, |
| def _check_fiscalcode(self): | ||
|
|
||
| if self.fiscalcode: | ||
| if self.is_company and len(self.fiscalcode) != 11 and not self.individual and not self.fiscalcode.isdigit(): |
| @@ -8,3 +8,4 @@ Lorenzo Battistini <lorenzo.battistini@albatos.com> | |||
| Roberto Onnis <onnis.roberto@gmail.com> | |||
There was a problem hiding this comment.
Please remove this file, as this list is already contained in README.rst
|
Thanks 👍 |
…_location_geonames_import l10n_it_base_location_geonames_import on 9.0
| (check_fiscalcode, | ||
| "The fiscal code doesn't seem to be correct.", ["fiscalcode"]) | ||
| ] | ||
| @api.one |
There was a problem hiding this comment.
Tks for the feedback .... i will remove api.one in the next release
Riesci a fare un rebase? |
… into 90-codicefiscale
|
Credo di aver risolto. |
| if partner.is_company and len(partner.fiscalcode) != 11 and \ | ||
| not partner.individual and not partner.fiscalcode.isdigit(): | ||
| raise ValidationError( | ||
| _("Company fiscal code must be 11 digts lenght.") |
|
@gigidn ho lasciato un commento sul codice |
|
@gigidn guardando il diff Forse alla fine del rebase non hai fatto Se ora il branch è comunque compromesso, ti consiglio di farne uno nuovo e una nuova PR Grazie |
|
Mi sa che ho scazzato con il -f ... in ogni caso visto che sanare il branch è operazione che consuma tempo mi sa che seguo il tuo consiglio @eLBati . In giornata faccio un nuovo branch ed una nuova PR e risolviamo cosi' ne approfitto anche per correggere gli errorini di travis e la segnalazione di @OpenCode (grazie per la segnalazione) |
eLBati
added
needs fixing
|
Penso si possa chiudere questa PR in favore della #274 |
…oice [ADD][l10n_it_fatturapa_out] add depencies from l10n_it_reverse_charge
10 participants
This is a refactor of l10n_it_fiscalcode v8 module and porting to v9. Unfortunately automatic computation of fiscal code produce many wrong code so my proposal is to remove this feature in base module and let a more specialized module like l10n_it_fiscalcode_computation to do it.