Mòdul aeat_303: Corregit percentatge RE al mapeig i commits de la 3.8 i default no traspassats a la 3.4

5 views
Skip to first unread message

Guillem Barba Domingo

unread,
Mar 22, 2016, 9:46:24 AM3/22/16
to tryt...@googlegroups.com
Hola ZZ,
al mòdul aeat_303 hi ha alguns commits de la 3.8 i default que no estan traspassats a la brnaca 3.4:
- Fix content of vat code of the company
- Fix assignement to backruptcy field
- pep8

No sé si és correcte perquè són canvis propis d'aquetes branques o els dos "fixes" s'haurien de traspassar a la 3.4

El que si que he fet jo ara és traspassat un commit que he fet a la 3.4 a aquestes dues branques:
- Fix percentage of second level of Recargo Equivalencia

Cal actualitzar el mòdul a les diferents BD per tenir la correcció

--
Guillem Barba
http://www.guillem.alcarrer.net

Jesús Martín Jiménez

unread,
Mar 22, 2016, 11:59:58 AM3/22/16
to tryt...@googlegroups.com
Hola Guillem,

2016-03-22 14:46 GMT+01:00 Guillem Barba Domingo <guille...@gmail.com>:
Hola ZZ,
al mòdul aeat_303 hi ha alguns commits de la 3.8 i default que no estan traspassats a la brnaca 3.4:
- Fix content of vat code of the company

Aquest no s'ha de passar perquè és propi de la branca 3.8.
 
- Fix assignement to backruptcy field
- pep8

Aquests sí es podrien haver passat. El pep8 no és important i l'altre em vaig descuidar de fer el graft. Si ho poguessis fer tu, t'ho agrairia.
 

No sé si és correcte perquè són canvis propis d'aquetes branques o els dos "fixes" s'haurien de traspassar a la 3.4

El que si que he fet jo ara és traspassat un commit que he fet a la 3.4 a aquestes dues branques:
- Fix percentage of second level of Recargo Equivalencia

Cal actualitzar el mòdul a les diferents BD per tenir la correcció

--



--

Jesús Martín

Zikzakmedia SL
C/ de Sant Jaume, 9, baixos, 2ª
08720 Vilafranca del Penedès
☏ 93 890 21 08

Jordi Esteve

unread,
Mar 22, 2016, 12:03:00 PM3/22/16
to tryt...@googlegroups.com
El 22/03/16 a les 14:46, Guillem Barba Domingo ha escrit:
> Hola ZZ,
> al mòdul aeat_303 hi ha alguns commits de la 3.8 i default que no
> estan traspassats a la brnaca 3.4:
> - Fix content of vat code of the company

Aq no té sentit traspassar-ho a la branca 3.4. Precisament aq commit és
per v 3.8 i default pq a partir de la 3.8 canvia com es codifiquen el
cif dels tercers: ara tenim una llista d'identificadors de tercers i es
guarden en un únic camp, o sigui que ara el codi del vat i el vat és un
sol camp. Per això a les branques 3.8 i default de tots els mòduls aeat
hem corregit per agafar identificador[:2] com a CIF de l'empresa a
codificar en els fitxers AEAT.


> - Fix assignement to backruptcy field

Aq si tindria sentit portar-ho a v3.4 i v.3.6, em sembla. Mira-te'l i ho
fas tu mateix si et sembla correcte:

- record.bankruptcy == bool(self.auto_bankruptcy_declaration != ' ')
+ record.bankruptcy = bool(self.auto_bankruptcy_declaration != ' ')


> - pep8

Es podria, però com que ja hi ha alguna diferència entre branques potser
no sigui un canvi fàcil i demani un merge manual. Podeu viure en la
v.3.4 sense pep8, quan passeu a v.3.8 ja tindreu pep8 ;-)

>
> No sé si és correcte perquè són canvis propis d'aquetes branques o els
> dos "fixes" s'haurien de traspassar a la 3.4
>
> El que si que he fet jo ara és traspassat un commit que he fet a la
> 3.4 a aquestes dues branques:
> - Fix percentage of second level of Recargo Equivalencia
>
> Cal actualitzar el mòdul a les diferents BD per tenir la correcció

Gràcies pel fix i per portar-ho. Porteu-ho tot a branques posteriors,
nosaltres tb fem esforç de portar el que es pugui a branques inferiors.

--
Jordi Esteve
Consultor Zikzakmedia SL
jes...@zikzakmedia.com
Mòbil 679 170 693

Zikzakmedia SL
St. Jaume, 9, baixos, 2a
08720 Vilafranca del Penedès
Tel 93 890 2108

Guillem Barba Domingo

unread,
Mar 29, 2016, 5:08:56 AM3/29/16
to tryt...@googlegroups.com
El dia 22 de març de 2016, 17:00, Jordi Esteve <jes...@zikzakmedia.com> ha escrit:
El 22/03/16 a les 14:46, Guillem Barba Domingo ha escrit:
Hola ZZ,
al mòdul aeat_303 hi ha alguns commits de la 3.8 i default que no estan traspassats a la brnaca 3.4:
- Fix content of vat code of the company

 Aq no té sentit traspassar-ho a la branca 3.4. Precisament aq commit és per v 3.8 i default pq a partir de la 3.8 canvia com es codifiquen el cif dels tercers: ara tenim una llista d'identificadors de tercers i es guarden en un únic camp, o sigui que ara el codi del vat i el vat és un sol camp. Per això a les branques 3.8 i default de tots els mòduls aeat hem corregit per agafar identificador[:2] com a CIF de l'empresa a codificar en els fitxers AEAT.

ok



- Fix assignement to backruptcy field

 Aq si tindria sentit portar-ho a v3.4 i v.3.6, em sembla. Mira-te'l i ho fas tu mateix si et sembla correcte:

-        record.bankruptcy == bool(self.auto_bankruptcy_declaration != ' ')
+        record.bankruptcy = bool(self.auto_bankruptcy_declaration != ' ')

Fet



- pep8

Es podria, però com que ja hi ha alguna diferència entre branques potser no sigui un canvi fàcil i demani un merge manual. Podeu viure en la v.3.4 sense pep8, quan passeu a v.3.8 ja tindreu pep8 ;-)

No l'he passat

 

No sé si és correcte perquè són canvis propis d'aquetes branques o els dos "fixes" s'haurien de traspassar a la 3.4

El que si que he fet jo ara és traspassat un commit que he fet a la 3.4 a aquestes dues branques:
- Fix percentage of second level of Recargo Equivalencia

Cal actualitzar el mòdul a les diferents BD per tenir la correcció

 Gràcies pel fix i per portar-ho. Porteu-ho tot a branques posteriors, nosaltres tb fem esforç de portar el que es pugui a branques inferiors.

Ja estava fet. El correu era per avisar-vos.
No he traspassat a la 3.2 i 3.0 perquè estan molt desactualitzades... potser hauriem de tenir una política de quan tancar branques antigues.

 
--
Jordi Esteve
Consultor Zikzakmedia SL
jes...@zikzakmedia.com
Mòbil 679 170 693

Zikzakmedia SL
St. Jaume, 9, baixos, 2a
08720 Vilafranca del Penedès
Tel 93 890 2108

Raimon Esteve

unread,
Mar 29, 2016, 5:54:54 AM3/29/16
to tryt...@googlegroups.com
No he traspassat a la 3.2 i 3.0 perquè estan molt desactualitzades... potser hauriem de tenir una política de quan tancar branques antigues.

Per mi, totes les < 3.8 ja es prodrien tancar ;) però no hem molesten si estàn obertes.

Una mica la política es qui l'usi, mantenir-les. Si no, tancar-les.

Per cert, ja hem publicat a la default canvis pel py3. Ara mirarem si el invoice in/out que afecta.

Salutacions.


Jordi Esteve (Zikzakmedia)

unread,
Mar 29, 2016, 6:03:03 AM3/29/16
to tryt...@googlegroups.com
El 29/03/16 a les 11:08, Guillem Barba Domingo ha escrit:
>
> Ja estava fet. El correu era per avisar-vos.
> No he traspassat a la 3.2 i 3.0 perquè estan molt desactualitzades...
> potser hauriem de tenir una política de quan tancar branques antigues.
>

Si, es podria anar tancant així que tanquin les branques dels mòduls
oficials. A l'abril, quan es publiqui la 4.0, es tancarà la 3.0. El
problema serà pensar-hi pq no ho tenim automatitzat.

Sergi Almacellas Abellana

unread,
Mar 29, 2016, 8:53:47 AM3/29/16
to tryt...@googlegroups.com
El 29/03/16 a les 11:54, Raimon Esteve ha escrit:
>
> Per cert, ja hem publicat a la default canvis pel py3. Ara mirarem si el
> invoice in/out que afecta.
Si heu fet un script ja ho podeu repassar, perquè us heu carregat alguna
cosa.

Per exemple:

https://drone.io/bitbucket.org/trytonspain/trytond-country_zip_es/16

Amb la conseqüent correcció:

https://bitbucket.org/trytonspain/trytond-country_zip_es/commits/b892587d66e2af74f71194a99cdeef2fe83cc382

A més a més, també s'ha de fer:

https://bitbucket.org/trytonspain/trytond-country_zip_es/commits/3a6818c53a8f2c2a50e3dd2f000595af0c841531

Gràcies :)

Podeu probar que el mòdul funciona amb py27 y py34 amb executant la
següent comanda:

tox -e py27-sqlite,py34-sqlite

Si no teniu el tox instal·lat el podeu instal·lar al virtualenv amb:

pip install tox

P.D: heu de tenir el py34 instal·lat a la màquina (si feu servir ubuntu
ja be instal·lat per defecte).

Salutacions.

--
Sergi Almacellas Abellana
www.koolpi.com
Twitter: @pokoli_srk

Raimon Esteve

unread,
Mar 29, 2016, 9:16:28 AM3/29/16
to tryt...@googlegroups.com
El dia 29 de març de 2016, 14:53, Sergi Almacellas Abellana <se...@koolpi.com> ha escrit:
El 29/03/16 a les 11:54, Raimon Esteve ha escrit:

Per cert, ja hem publicat a la default canvis pel py3. Ara mirarem si el
invoice in/out que afecta.
Si heu fet un script ja ho podeu repassar, perquè us heu carregat alguna cosa.

Per exemple:

https://drone.io/bitbucket.org/trytonspain/trytond-country_zip_es/16

Amb la conseqüent correcció:

https://bitbucket.org/trytonspain/trytond-country_zip_es/commits/b892587d66e2af74f71194a99cdeef2fe83cc382

si els mòduls hi han fitxers com csv o altres, jrxml, etc, cal afegir-los en el setup. Faré un grep; si en veus que manca, afegeix-los.

 
Ok. Per ara, a tests, no s'ha aplicat res/revisa't. En TODO.
 


Gràcies :)

Podeu probar que el mòdul funciona amb py27 y py34 amb executant la següent comanda:

tox -e py27-sqlite,py34-sqlite

Si no teniu el tox instal·lat el podeu instal·lar al virtualenv amb:

pip install tox

P.D: heu de tenir el py34 instal·lat a la màquina (si  feu servir ubuntu ja be instal·lat per defecte).

Ok, no hem mirat res sobre el tox encara.
 

Salutacions.

--
Sergi Almacellas Abellana
www.koolpi.com
Twitter: @pokoli_srk



--
Raimon Esteve - @raimonesteve
// Cloud Tryton - OpenERP www.zzsaas.com
// DOC: http://doc.zzsaas.com
www.zikzakmedia.com
Reply all
Reply to author
Forward
0 new messages