Migration to Version 9.0 of l10n_it_fiscalcode by hurrinico · Pull Request #274 · OCA/l10n-italy

hurrinico · 2016-10-25T08:33:39Z

No description provided.

hurrinico · 2016-10-25T12:48:37Z

@OpenCode @eLBati if this pr is ok i think that #203 by @gigidn must be closed

andrea4ever · 2016-10-25T12:51:53Z

@@ -0,0 +1,75 @@
+.. image:: https://img.shields.io/badge/licence-AGPL--3-blue.svg
+    :alt: License


Add this :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html, on line 2

andrea4ever · 2016-10-25T12:52:20Z

+.. image:: https://img.shields.io/badge/licence-AGPL--3-blue.svg
+    :alt: License
+
+Italian Localization - FiscalCode


=================================, before on line before module name

andrea4ever · 2016-10-25T12:53:28Z

+============
+
+This module requires codicefiscale
+https://pypi.python.org/pypi/codicefiscale


replace with codicefiscale https://pypi.python.org/pypi/codicefiscale

andrea4ever · 2016-10-25T12:54:59Z

-    'version': '8.0.0.1.0',
+    'version': '9.0.0.1.0',
    'category': 'Localisation/Italy',
    'description': """


this key can be removed.

hurrinico · 2016-10-25T16:05:07Z

@andrea4ever i've made the requested changes

andrea4ever · 2016-10-25T16:17:36Z

@hurrinico I miss to escape string in my review for link to python depends codicefiscale, the correct string is codicefiscale https://pypi.python.org/pypi/codicefiscale.

OpenCode · 2016-10-26T09:37:04Z

        "security/ir.model.access.csv"
-        ],
+    ],
    'qweb': [],


Remove empty keys

OpenCode · 2016-10-26T09:37:41Z

+#: model:ir.model.fields,help:l10n_it_fiscalcode.field_res_partner_individual
 msgid "If checked the C.F. is referred to a Individual Person"
 msgstr "Se selezionato il Cofice Fiscale farà riferimento a quello di una Persona Fisica"



Typo Cofice -> Codice

OpenCode · 2016-10-26T09:37:59Z

 #: selection:wizard.compute.fc,sex:0
 msgid "Male"
 msgstr "Maschio"



Forse è meglio Uomo

OpenCode · 2016-10-26T09:38:12Z

@@ -87,71 +95,84 @@ msgid "Female"
 msgstr "Femmina"



Forse è meglio Donna

hurrinico · 2016-10-27T08:46:44Z

@OpenCode se non mi ricordo male sui documenti è indicato M/F... però forse mi sbaglio
@andrea4ever mi sa che non ho capito cosa cambiare riguardo la libreria codicefiscale

OpenCode

Tested on Runbot.
Insert a fiscalcode on partner: Ok
Calculate fiscalcode with wizard: Ok
Get an error whn calculate a fiscalcode on partner that has one yet: Ok

andrea4ever · 2016-10-27T15:46:11Z

@hurrinico usa la stringa "codicefiscale" come etichetta per il web url in modo che non sia visibile, così come avviene già nella sezione Bug Tracker con il link alle Issues del repository

hurrinico · 2016-11-04T11:01:45Z

@andrea4ever ho fixato quanto chiesto

andrea4ever · 2016-11-04T14:01:14Z

👍

eLBati

just a couple of remarks

eLBati · 2016-11-04T16:19:56Z

+Images
+------
+
+* Odoo Community Association: `Icon <https://github.com/OCA/maintainer-tools/blob/master/template/module/static/description/icon.svg>`_.


Questa sezione è superflua, poichè che l'icona OCA non è usata

eLBati · 2016-11-04T16:20:27Z

    'name': 'Italian Localisation - Fiscal Code',
-    'version': '8.0.0.1.0',
    'category': 'Localisation/Italy',
    'description': """


Essendoci il README, l'elemento description è da rimuovere

eLBati · 2016-11-04T16:24:40Z

 ##############################################################################
 {
    'name': 'Italian Localisation - Fiscal Code',
-    'version': '8.0.0.1.0',


Mi pare manchi l'elemento version

Volevo segnalare che ho iniziato una PR sul modulo [10] codicefiscale, che potrebbe interessare anche al backport verso la 9.0.
##278

LE modifiche riguardano:

eliminazione del campo _constrain [deprecated] e del metodo collegato _check_codicefiscale() , e passaggio alla #api.constrains(‘fiscalcode’)

modifica alla validazione dell’algoritmo di validazione codice inserito (1)

eliminazione del campo inidividual ed utilizzo del campo “nativo” is_company, per determinare se il partner è una company o una persona fisica, da cui l’applicabilità del calcolo automatico nonchè tipo di verifica del codice fiscale inserito

visualizzazione on/off del button per il calcolo del codice fiscale

alcune modifiche al modulo di test

Vi volevo invitare a dare una occhiata per unire gli sforzi…

(1) Dal dicembre 2001 in Italia il codice fiscale è posto uguale alla PIVA per società di persone e capitali, associazioni, fondazionie etc..
Ad oggi il modulo codice fiscale impedisce di inserire la PIVA come valore valido di codice fiscale per le aziende (vede la PIVA come un CF sbagliato...) . Con il campo is_company si può distinguere tra persona fisica (codice fiscale) e azienda (partita IVA).

Tutti a rifa' sto cidice_fiscale :D cmq qui trovi il lavoro che a suo tempo avevo fatto per la v9 ed il porting alla v10:

https://github.com/gigidn/l10n-italy/tree/v10-fiscalcode

Il campo individual serve perchè le ditte individuali hanno codice fiscale = codice_fiscale_proprietario e non alla partita iva.

Il codice fiscale delle amministrazioni pubbliche è composto da soli numeri ma non è una partita iva valida se applichi le regole di verifica della partita iva.

Come dissi tempo fa a me non piace un modulo codice_fiscale con calcolo integrato ... meglio dividere in due il modulo, uno con la sola funzione del codice fiscale e l'altro che lo estende ed aggiunge la creazione automatica cosi' che se uno vuole evita di installare questo ultimo perchè fonte di mille problemi.

In ogni modo se puo' essere da spunto i codici son li .... che poi son gli stessi o quasi della PR di molti mesi fa.

hurrinico · 2016-11-07T08:26:28Z

@eLBati fatto.
@Giuliano69 ho visto che c'è un po di discussione sugli improve alla versione 10, procederei con il porting dalla versione 8 alla 9 e quando verrà mergiata la tua pr con un git cherry-pick possiamo portare la modifica anche sulla 9

eLBati · 2016-11-07T10:37:11Z

@hurrinico grazie. Puoi fare lo squash dei commit in base alla logica delle modifiche?

hurrinico · 2016-11-07T10:43:06Z

@eLBati fatto

hurrinico · 2016-11-07T13:20:45Z

Ho fatto degli errori in fase di squash.
Ho aperto questa #281

Giuliano69 · 2016-11-07T14:39:24Z

ho aperto una discussione sulle funzionalità e caratteristiche nella PR
##278

oca-transbot added 5 commits October 25, 2016 09:51

OCA Transbot updated translations from Transifex

0d26bd5

OCA Transbot updated translations from Transifex

fa18b4e

OCA Transbot updated translations from Transifex

6de8d5a

OCA Transbot updated translations from Transifex

249d7a3

OCA Transbot updated translations from Transifex

5d38218

OpenCode added work in progress 9.0 needs review and removed work in progress labels Oct 25, 2016

hurrinico mentioned this pull request Oct 25, 2016

[ADD] Refactoring of module l10n_it_website_sale_fiscalcode #275

Closed

andrea4ever suggested changes Oct 25, 2016

View reviewed changes

andrea4ever mentioned this pull request Oct 25, 2016

l10n_it_fiscalcode refactor and port to v9 #203

Closed

OpenCode suggested changes Oct 26, 2016

View reviewed changes

OpenCode approved these changes Oct 27, 2016

View reviewed changes

andrea4ever approved these changes Nov 4, 2016

View reviewed changes

eLBati requested changes Nov 4, 2016

View reviewed changes

[MIGR] Porting Module l10n_it_fiscalcode to version 9

b0fc89b

hurrinico mentioned this pull request Nov 7, 2016

[MIGR] Porting Module l10n_it_fiscalcode to version 9 #281

Closed

hurrinico closed this Nov 7, 2016

		@@ -0,0 +1,75 @@
		.. image:: https://img.shields.io/badge/licence-AGPL--3-blue.svg
		:alt: License

Uh oh!

Uh oh!

Conversation

hurrinico commented Oct 25, 2016

Uh oh!

hurrinico commented Oct 25, 2016

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

andrea4ever Oct 25, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hurrinico commented Oct 25, 2016

Uh oh!

andrea4ever commented Oct 25, 2016

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hurrinico commented Oct 27, 2016

Uh oh!

OpenCode left a comment

Choose a reason for hiding this comment

Uh oh!

andrea4ever commented Oct 27, 2016

Uh oh!

hurrinico commented Nov 4, 2016

Uh oh!

andrea4ever commented Nov 4, 2016

Uh oh!

eLBati left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hurrinico commented Nov 7, 2016

Uh oh!

eLBati commented Nov 7, 2016

Uh oh!

hurrinico commented Nov 7, 2016

Uh oh!

hurrinico commented Nov 7, 2016

Uh oh!

Giuliano69 commented Nov 7, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

andrea4ever Oct 25, 2016 •

edited

Loading