Re: [Openobject-italia-core-devs] [Merge] lp:~coopenerp/openobject-italia/7.0-adding-intra-cee into lp:openobject-italia/7.0

2014-04-09 Per discussione Sergio Corato
Il 09/04/2014 12:48, Andrea Cometa ha scritto:
 il modulo è stato interamente revisionato, per una guida si può fare 
 riferimento qui: 
 http://www.coopenerp.it/openerp/invoice-intra-cee-reverse-charge-ed-extra-ue-con-openerp/

Ciao Andrea,
ottimo lavoro, ho fatto alcuni test e segnalo alcune opinioni personali:
1. il campo boolean RC così com'è lascia troppa libertà all'utente, e 
allo stesso tempo richiede una certa competenza: io lo eviterei (oppure 
renderei readonly, collegato alla presenza o meno della tassa 
auto_invoice_tax_id, per informazione) impostando che solo che righe 
che hanno una tassa auto_invoice_tax_id collegata alla tassa usata siano 
considerate. In tal modo si semplificherebbe anche il lavoro, dato che 
le imposte possono essere configurate una volta sola in fase di 
installazione.
2. il campo RC deve essere sempre compilato (r. 371)? in tal caso 
sarebbe inutile. Invece deve permettere di creare un'autofattura parziale;
3. nella vista delle tasse, il campo aggiuntivo auto_invoice_tax_id 
dovrebbe essere visibile solo in quelle di tipo acquisto, in quanto 
tutte le procedure di reverse charge partono dalla fattura di acquisto. 
Lasciare il campo anche nelle tasse di tipo vendita confonde l'utente.

-- 
Sergio Corato
Didotech.com - Icstools.it
web   : www.didotech.com
blog  : www.icstools.it
skype : sergiocorato
mail  : sergiocor...@gmail.com

https://code.launchpad.net/~coopenerp/openobject-italia/7.0-adding-intra-cee/+merge/210855
Your team OpenERP Italia core devs is subscribed to branch 
lp:openobject-italia/7.0.

___
Mailing list: https://launchpad.net/~openobject-italia-core-devs
Post to : openobject-italia-core-devs@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openobject-italia-core-devs
More help   : https://help.launchpad.net/ListHelp


Re: [Openobject-italia-core-devs] [Merge] lp:~coopenerp/openobject-italia/7.0-adding-intra-cee into lp:openobject-italia/7.0

2014-03-25 Per discussione Lorenzo Battistini - Agile BG
On 03/25/2014 04:18 PM, Francesco Apruzzese wrote:
 Il modulo ha subito un forte lavoro di re-implementazione di molte parti.

Grazie Francesco.


 Il codice ora è sotto copyright della comunità.

Per me potete aggiungere anche il vostro, se volete.


 La mini guida linkata sopra è ancora valida per chi volesse effettuare i test.

Includerei comunque la guida anche nella descrizione del modulo.



 Ci teniamo a precisare che quanto scritto nel modulo è valido solo ed 
 esclusivamente per la gestione fattura Intra CEE. Questo vuol dire che ciò 
 che manca per il Reverse Charge non è una bug o una mancanza ma è volutamente 
 tenuto fuori. Questo vuol dire che il RC dovrà o essere implementato una 
 volta che il modulo risulta utilizzabile o gestito mediante un altro modulo a 
 sè stante.

Scusa ma l'autofatturazione non è il meccanismo con cui si gestisce il 
reverse charge?
http://www.ilsole24ore.com/art/SoleOnLine4/100-parole/Diritto/R/Reverse-charge.shtml
Cosa mancherebbe a questo modulo per gestire il reverse charge?


-- 
https://code.launchpad.net/~coopenerp/openobject-italia/7.0-adding-intra-cee/+merge/210855
Your team OpenERP Italia core devs is subscribed to branch 
lp:openobject-italia/7.0.

___
Mailing list: https://launchpad.net/~openobject-italia-core-devs
Post to : openobject-italia-core-devs@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openobject-italia-core-devs
More help   : https://help.launchpad.net/ListHelp


Re: [Openobject-italia-core-devs] [Merge] lp:~coopenerp/openobject-italia/7.0-adding-intra-cee into lp:openobject-italia/7.0

2014-03-25 Per discussione Francesco Apruzzese

Il 25/03/2014 16:51, Lorenzo Battistini - Agile BG ha scritto:

 La mini guida linkata sopra è ancora valida per chi volesse effettuare i 
 test.
 Includerei comunque la guida anche nella descrizione del modulo.


Vediamo come poter fare la cosa al meglio

 Scusa ma l'autofatturazione non è il meccanismo con cui si gestisce il
 reverse charge?
 http://www.ilsole24ore.com/art/SoleOnLine4/100-parole/Diritto/R/Reverse-charge.shtml
 Cosa mancherebbe a questo modulo per gestire il reverse charge?



Da quello che si è potuto appurare, i due strumenti sono diversi:
http://www.mysolutionpost.it/blogs/imposte-indirette/setti/2013/10/master-iva-mysolution-quesito.aspx

Ho notato però che l'argomento è un po' particolare. Sarebbe utile 
sentire chi di queste cose ne capisce (lungi da me) per capire un attimo 
il quadro completo.
Attualmente il nostro modulo non può gestire un reverse charge poichè 
questo è pensato per riga di fattura mentre il nostro modulo genere 
autofatture per il totale del documento stesso.

-- 
https://code.launchpad.net/~coopenerp/openobject-italia/7.0-adding-intra-cee/+merge/210855
Your team OpenERP Italia core devs is subscribed to branch 
lp:openobject-italia/7.0.

___
Mailing list: https://launchpad.net/~openobject-italia-core-devs
Post to : openobject-italia-core-devs@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openobject-italia-core-devs
More help   : https://help.launchpad.net/ListHelp


Re: [Openobject-italia-core-devs] [Merge] lp:~coopenerp/openobject-italia/7.0-adding-intra-cee into lp:openobject-italia/7.0

2014-03-24 Per discussione Davide Corio @ LS
On 23 Mar 2014, at 21:51, Sergio Corato sergiocor...@gmail.com wrote:

 - non legherei la configurazione del reverse alla company ma alle 
 fiscal_position, in modo da poterne creare più di una

Mi pareva che tutti e due i branch usassero le posizioni fiscali.
Nel mio avevo messo a livello company alcuni settaggi generici, ma se c’è 
motivo per averli specifici per posizioni fiscale sono d’accordissimo nello 
spostarli.

 - terrei la procedura di creazione per riga nella fattura, in modo da poter 
 escludere delle righe

Io avevo messo un flag sulla riga fattura, siccome spesso capita che sulla 
singola fattura ci siano righe soggette a RC ed altre no.

 - legherei la verifica delle righe da duplicare sulla base della presenza o 
 meno nelle stesse dell'IVA reverse, creando quindi il collegamento dell'IVA 
 reverse all'interno dell'IVA normale dedicata (questo implica che una fattura 
 intra-cee ricevuta per essere duplicata deve avere un codice iva acquisto 
 dedicato, direi facilmente fattibile usando le posizioni fiscali nella loro 
 implementazione standard)

Yes, lo faccio fare a tutti i clienti.

Personalmente, come avevo già commentato, sarei per l’evitare la “duplicazione”.
Preferirei che i dati della fattura e delle righe venissero calcolati con un 
metodo dedicato, così che sia eventualmente possibile farne l’override per 
inserire codice custom.

Grazie come sempre per il prezioso supporto.

--
Davide Corio
davide.co...@lsweb.it




-- 
https://code.launchpad.net/~coopenerp/openobject-italia/7.0-adding-intra-cee/+merge/210855
Your team OpenERP Italia core devs is subscribed to branch 
lp:openobject-italia/7.0.

___
Mailing list: https://launchpad.net/~openobject-italia-core-devs
Post to : openobject-italia-core-devs@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openobject-italia-core-devs
More help   : https://help.launchpad.net/ListHelp


Re: [Openobject-italia-core-devs] [Merge] lp:~coopenerp/openobject-italia/7.0-adding-intra-cee into lp:openobject-italia/7.0

2014-03-21 Per discussione Andrea Cometa
Ciao Lorenzo, la mini guida è presente qui: 
http://www.coopenerp.it/openerp/fatture-di-acquisto-intra-cee-con-openerp/

prendo in considerazione le modifiche proposte, soprattutto relativamente alla 
gestione dello stato fattura, provvederò a variarlo mediante workflow
-- 
https://code.launchpad.net/~coopenerp/openobject-italia/7.0-adding-intra-cee/+merge/210855
Your team OpenERP Italia core devs is subscribed to branch 
lp:openobject-italia/7.0.

___
Mailing list: https://launchpad.net/~openobject-italia-core-devs
Post to : openobject-italia-core-devs@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openobject-italia-core-devs
More help   : https://help.launchpad.net/ListHelp


Re: [Openobject-italia-core-devs] [Merge] lp:~coopenerp/openobject-italia/7.0-adding-intra-cee into lp:openobject-italia/7.0

2014-03-21 Per discussione Davide Corio @ LS
Se può essere utile, nel mio modulo:
lp:~enlightx/openobject-italia/7.0-reverse-charge-imp
avevo aggiunto i movimenti per riconciliare la parte iva delle fatture di 
acquisto e delle autofatture.
il codice è ovviamente un po' grezzo, l'ho pubblicato comunque nel caso in cui 
vi servisse come spunto.
-- 
https://code.launchpad.net/~coopenerp/openobject-italia/7.0-adding-intra-cee/+merge/210855
Your team OpenERP Italia core devs is subscribed to branch 
lp:openobject-italia/7.0.

___
Mailing list: https://launchpad.net/~openobject-italia-core-devs
Post to : openobject-italia-core-devs@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openobject-italia-core-devs
More help   : https://help.launchpad.net/ListHelp


Re: [Openobject-italia-core-devs] [Merge] lp:~coopenerp/openobject-italia/7.0-adding-intra-cee into lp:openobject-italia/7.0

2014-03-21 Per discussione Andrea Cometa
+1 Davide
-- 
https://code.launchpad.net/~coopenerp/openobject-italia/7.0-adding-intra-cee/+merge/210855
Your team OpenERP Italia core devs is subscribed to branch 
lp:openobject-italia/7.0.

___
Mailing list: https://launchpad.net/~openobject-italia-core-devs
Post to : openobject-italia-core-devs@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openobject-italia-core-devs
More help   : https://help.launchpad.net/ListHelp


Re: [Openobject-italia-core-devs] [Merge] lp:~coopenerp/openobject-italia/7.0-adding-intra-cee into lp:openobject-italia/7.0

2014-03-21 Per discussione Davide Corio @ LS
NB: mi pareva di aver visto da qualche parte, forse in account_voucher, un 
metodo che, date due account.move, ti restituiva gli id delle relative 
account.move.line da riconciliare.

Se non ho preso un abbaglio, potrebbe essere utile per tenere il codice pulito.
-- 
https://code.launchpad.net/~coopenerp/openobject-italia/7.0-adding-intra-cee/+merge/210855
Your team OpenERP Italia core devs is subscribed to branch 
lp:openobject-italia/7.0.

___
Mailing list: https://launchpad.net/~openobject-italia-core-devs
Post to : openobject-italia-core-devs@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openobject-italia-core-devs
More help   : https://help.launchpad.net/ListHelp


Re: [Openobject-italia-core-devs] [Merge] lp:~coopenerp/openobject-italia/7.0-adding-intra-cee into lp:openobject-italia/7.0

2014-03-20 Per discussione Lorenzo Battistini - Agile BG
Review: Needs Fixing

Ciao Andrea e grazie per il contributo.

Potresti migliorare la descrizione del modulo aggiungendo una mini guida su 
come debba essere utilizzato?

Potresti aggiungere anche il copyright dell'associazione
#Copyright (C) 2012 Associazione OpenERP Italia
#(http://www.openerp-italia.org).
?

Sarebbe ottimo se riuscissi ad aggiungere anche i test YAML, che aiuterebbero 
anche a capire le specifiche del modulo.

Per quanto riguarda la scrittura diretta di 'paid' nello stato della fattura, 
anch'io sono contrario. Facendo così si avrebbe il campo 'stato' incoerente con 
lo stato del workflow.
Una fattura va nello stato pagato quando tutti i suoi debiti/crediti sono 
riconciliati.

In questo caso, prendendo l'esempio descritto qui
http://gestionaleamica.com/Blog/fatturazione-in-reverse-charge-ecco-come-farla/
nella terza registrazione contabile,
'Fornitore in Dare €200 (valore dell’IVA al 20%)'
viene riconciliato parzialmente con i 200 del debito della fattura fornitore, 
lasciando la fattura aperta per 1000. Quando poi verrano versati quei 1000€ al 
fornitore, la fattura verrà automaticamente chiusa.
Mentre la fattura cliente si riconcilia subito totalmente con
'Cliente (se stessi per autofattura) in Avere €1200 (il valore della fattura 
con IVA)'
-- 
https://code.launchpad.net/~coopenerp/openobject-italia/7.0-adding-intra-cee/+merge/210855
Your team OpenERP Italia core devs is subscribed to branch 
lp:openobject-italia/7.0.

___
Mailing list: https://launchpad.net/~openobject-italia-core-devs
Post to : openobject-italia-core-devs@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openobject-italia-core-devs
More help   : https://help.launchpad.net/ListHelp


Re: [Openobject-italia-core-devs] [Merge] lp:~coopenerp/openobject-italia/7.0-adding-intra-cee into lp:openobject-italia/7.0

2014-03-13 Per discussione Andrea Cometa
Review: Approve

ora il branch dovrebbe essere quello giusto :)
-- 
https://code.launchpad.net/~coopenerp/openobject-italia/7.0-adding-intra-cee/+merge/210855
Your team OpenERP Italia core devs is subscribed to branch 
lp:openobject-italia/7.0.

___
Mailing list: https://launchpad.net/~openobject-italia-core-devs
Post to : openobject-italia-core-devs@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openobject-italia-core-devs
More help   : https://help.launchpad.net/ListHelp


Re: [Openobject-italia-core-devs] [Merge] lp:~coopenerp/openobject-italia/7.0-adding-intra-cee into lp:openobject-italia/7.0

2014-03-13 Per discussione Davide Corio @ LS
Review: Disapprove

Personalmente non sono a favore dello scrivere manualmente paid.
Una fattura dovrebbe risultare in tale stato perchè è openerp a deciderlo, dopo 
aver verificato la riconciliazione completa.

Il deepcopy è pericoloso.

L'autofattura non dovrebbe essere intestata all'azienda stessa? dal vostro 
codice mi pare che venga intestata al fornitore.

Inoltre penso che se dobbiamo includere nuovi moduli, soprattutto in vista di 
OpenERP 8.0, dovrebbero rispettare le linee guida OCA, (nuove API OpenERP, 
PEP8, test automatici, ...)
-- 
https://code.launchpad.net/~coopenerp/openobject-italia/7.0-adding-intra-cee/+merge/210855
Your team OpenERP Italia core devs is subscribed to branch 
lp:openobject-italia/7.0.

___
Mailing list: https://launchpad.net/~openobject-italia-core-devs
Post to : openobject-italia-core-devs@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openobject-italia-core-devs
More help   : https://help.launchpad.net/ListHelp


Re: [Openobject-italia-core-devs] [Merge] lp:~coopenerp/openobject-italia/7.0-adding-intra-cee into lp:openobject-italia/7.0

2014-03-13 Per discussione Francesco Apruzzese

Il 13/03/2014 18:08, Davide Corio @ LS ha scritto:
 Review: Disapprove

 Personalmente non sono a favore dello scrivere manualmente paid.
 Una fattura dovrebbe risultare in tale stato perchè è openerp a deciderlo, 
 dopo aver verificato la riconciliazione completa.

Ovviamente ogni tipo di soluzione è ben accetta se possiamo migliorare 
il codice. La decisione riguardo lo stato di pagamento è dovuta al fatto 
che per far risultare una fattura pagata dovremmo registrare un 
pagamento che in realtà non sarebbe mai stato emesso e pertanto 
eventuali riepiloghi risulterebbero sfalsati.

 Il deepcopy è pericoloso.

Non trovo contro-documentazione a riguardo. Sarebbe utile (anche al di 
fuori del contesto) capire perchè.


 L'autofattura non dovrebbe essere intestata all'azienda stessa? dal vostro 
 codice mi pare che venga intestata al fornitore.

Voci discordanti. Ma lo si può cambiare in un niente.


 Inoltre penso che se dobbiamo includere nuovi moduli, soprattutto in vista di 
 OpenERP 8.0, dovrebbero rispettare le linee guida OCA, (nuove API OpenERP, 
 PEP8, test automatici, ...)

Per le nuove API ok. Per il PEP8 abbiamo cercato di attenerci il più 
possibile ma su questo credo che ci sarebbe da aprire un enorme 
discussione in piazza poiché neanche i moduli di OpenERP basilari 
rispettano la minima regola. I moduli scritti bene si contano davvero 
sulle dita di una mano. Sfido chiunque a smentirmi. Se mi dite dove 
abbiamo fallato provvediamo a correggere subito. Questi, in fin dei 
conti, sono dettagli secondari.

-- 
https://code.launchpad.net/~coopenerp/openobject-italia/7.0-adding-intra-cee/+merge/210855
Your team OpenERP Italia core devs is subscribed to branch 
lp:openobject-italia/7.0.

___
Mailing list: https://launchpad.net/~openobject-italia-core-devs
Post to : openobject-italia-core-devs@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openobject-italia-core-devs
More help   : https://help.launchpad.net/ListHelp


Re: [Openobject-italia-core-devs] [Merge] lp:~coopenerp/openobject-italia/7.0-adding-intra-cee into lp:openobject-italia/7.0

2014-03-13 Per discussione Davide Corio @ LS

On 13 Mar 2014, at 18:51, Francesco Apruzzese cesc...@gmail.com wrote:
 Ovviamente ogni tipo di soluzione è ben accetta se possiamo migliorare 
 il codice. La decisione riguardo lo stato di pagamento è dovuta al fatto 
 che per far risultare una fattura pagata dovremmo registrare un 
 pagamento che in realtà non sarebbe mai stato emesso e pertanto 
 eventuali riepiloghi risulterebbero sfalsati.

Temo il contrario.
La fattura fornitore viene riconciliata dal pagamento al fornitore + scrittura 
integrativa, siccome l'iva deve essere esplicitata sulla fattura intra comunque.
L'autofattura quindi rimarrebbe scoperta.

ma forse ho compreso male il vostro codice?

 Non trovo contro-documentazione a riguardo. Sarebbe utile (anche al di 
 fuori del contesto) capire perchè.

è pericoloso tanto quanto la funzione duplica di openerp :-)
Penso sarebbe più opportuno avere una funzione tipo _prepare_auto_invoice e 
_prepare_auto_invoice_lines, così da dare la possibilità a chiunque di 
ereditare il modulo per cambiare determinati comportamenti

 L'autofattura non dovrebbe essere intestata all'azienda stessa? dal vostro 
 codice mi pare che venga intestata al fornitore.
 
 Voci discordanti. Ma lo si può cambiare in un niente.

Quindi sarebbe necessario avere un riscontro da parte dell'agenzia delle 
entrate presumo.

 Per le nuove API ok. Per il PEP8 abbiamo cercato di attenerci il più 
 possibile ma su questo credo che ci sarebbe da aprire un enorme 
 discussione in piazza poiché neanche i moduli di OpenERP basilari 
 rispettano la minima regola. I moduli scritti bene si contano davvero 
 sulle dita di una mano. Sfido chiunque a smentirmi.

Il fatto che il core di openerp sia scritto male non penso sia un 
giustificativo valido per continuare a scrivere male il codice.

se osv.osv è deprecato in favore di orm.Model, perchè continuare ad usare il 
primo?
se è risaputo che spezzare le righe ad 80 colonne aumenta la leggibilità, 
perchè non farlo?

BTW, questa è ovviamente la mia opinione. Spero che arrivino altri contributi 
alla revisione di questo modulo.

--
Davide Corio
davide.co...@lsweb.it


-- 
https://code.launchpad.net/~coopenerp/openobject-italia/7.0-adding-intra-cee/+merge/210855
Your team OpenERP Italia core devs is subscribed to branch 
lp:openobject-italia/7.0.

___
Mailing list: https://launchpad.net/~openobject-italia-core-devs
Post to : openobject-italia-core-devs@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openobject-italia-core-devs
More help   : https://help.launchpad.net/ListHelp


Re: [Openobject-italia-core-devs] [Merge] lp:~coopenerp/openobject-italia/7.0-adding-intra-cee into lp:openobject-italia/7.0

2014-03-13 Per discussione Davide Corio @ LS

On 13 Mar 2014, at 19:12, Davide Corio @ LS davide.co...@lsweb.it wrote:
 La fattura fornitore viene riconciliata dal pagamento al fornitore + 
 scrittura integrativa, siccome l'iva deve essere esplicitata sulla fattura 
 intra comunque.

o meglio... questo è quello che avevo in mente :)

--
Davide Corio
davide.co...@lsweb.it

-- 
https://code.launchpad.net/~coopenerp/openobject-italia/7.0-adding-intra-cee/+merge/210855
Your team OpenERP Italia core devs is subscribed to branch 
lp:openobject-italia/7.0.

___
Mailing list: https://launchpad.net/~openobject-italia-core-devs
Post to : openobject-italia-core-devs@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openobject-italia-core-devs
More help   : https://help.launchpad.net/ListHelp


Re: [Openobject-italia-core-devs] [Merge] lp:~coopenerp/openobject-italia/7.0-adding-intra-cee into lp:openobject-italia/7.0

2014-03-13 Per discussione Davide Corio @ LS
Review: Abstain


-- 
https://code.launchpad.net/~coopenerp/openobject-italia/7.0-adding-intra-cee/+merge/210855
Your team OpenERP Italia core devs is subscribed to branch 
lp:openobject-italia/7.0.

___
Mailing list: https://launchpad.net/~openobject-italia-core-devs
Post to : openobject-italia-core-devs@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openobject-italia-core-devs
More help   : https://help.launchpad.net/ListHelp


Re: [Openobject-italia-core-devs] [Merge] lp:~coopenerp/openobject-italia/7.0-adding-intra-cee into lp:openobject-italia/7.0

2014-03-13 Per discussione Francesco Apruzzese

Il 13/03/2014 19:12, Davide Corio @ LS ha scritto:
 Temo il contrario.
 La fattura fornitore viene riconciliata dal pagamento al fornitore + 
 scrittura integrativa, siccome l'iva deve essere esplicitata sulla fattura 
 intra comunque.
 L'autofattura quindi rimarrebbe scoperta.

 ma forse ho compreso male il vostro codice?

Se hai una fattura fornitore ed una fattura intra e le paghi entrambe 
risulta che hai emesso due volte lo stesso importo come pagamento. Il 
succo è questo. OpenERP richiede due pagamenti legati alle due fatture 
per farle risultare pagate. O forse mi sfugge qualche automatismo?

 è pericoloso tanto quanto la funzione duplica di openerp :-)

In pratica va a finire che non abbiamo scritto un modulo ma una bomba a 
mano per quante cose pericolose ci sono dentro. :)
Continuo però a non capire perché deepcopy sia pericoloso ed ora non 
capisco neanche perchè la funzione di duplicazione di OpenERP sia 
pericolosa. Va bene. C'è pericolo. Ma vorrei sapere perchè. Anche per 
conoscenza personale. Sai com'è

 Quindi sarebbe necessario avere un riscontro da parte dell'agenzia delle 
 entrate presumo.

Il primo che finisce in carcere ci chiama e noi correggiamo il modulo. :)

 Il fatto che il core di openerp sia scritto male non penso sia un 
 giustificativo valido per continuare a scrivere male il codice.

Non mi riferivo al core infatti. Mi riferivo ai moduli. Compresi i 
nostri. Basta prenderne qualcuno a campione. Non me ne voglia chi ha 
scritto i codici, eh. Sia chiaro. Ma continuo a ripetere che abbiamo 
PROVATO a sottostare alle regole PEP8. Se mi dite dove abbiamo sbagliato 
correggo. Altrimenti non riesco a seguirvi.


 se osv.osv è deprecato in favore di orm.Model, perchè continuare ad usare il 
 primo?

Hai ragione e correggo.

 se è risaputo che spezzare le righe ad 80 colonne aumenta la leggibilità, 
 perchè non farlo?

C'è una sola riga che supera gli 80 caratteri. L'altra è un commento. 
Personalmente non ho dato importanza alla cosa poichè la riga più lunga 
è di 86 caratteri. Correggo e la prossima volta presterò più attenzione.

 BTW, questa è ovviamente la mia opinione. Spero che arrivino altri contributi 
 alla revisione di questo modulo.

Siamo una community. È giusto che sia così.

-- 
https://code.launchpad.net/~coopenerp/openobject-italia/7.0-adding-intra-cee/+merge/210855
Your team OpenERP Italia core devs is subscribed to branch 
lp:openobject-italia/7.0.

___
Mailing list: https://launchpad.net/~openobject-italia-core-devs
Post to : openobject-italia-core-devs@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openobject-italia-core-devs
More help   : https://help.launchpad.net/ListHelp


Re: [Openobject-italia-core-devs] [Merge] lp:~coopenerp/openobject-italia/7.0-adding-intra-cee into lp:openobject-italia/7.0

2014-03-13 Per discussione Davide Corio @ LS

On 13 Mar 2014, at 19:30, Francesco Apruzzese cesc...@gmail.com wrote:
 
 Se hai una fattura fornitore ed una fattura intra e le paghi entrambe 

umh... la fattura fornitore è la fattura intra :)

 C'è pericolo. Ma vorrei sapere perchè. Anche per 
 conoscenza personale. Sai com'è

Semplicemente perchè devi fare attenzione a non copiare cose sbagliate.
Ma qui sta alla bontà di chi scrive il codice.

Ma principalmente ciò che la copia inibisce è la possibilità di intervenire 
nella creazione del documento.

 C'è una sola riga che supera gli 80 caratteri. L'altra è un commento. 
 Personalmente non ho dato importanza alla cosa poichè la riga più lunga 
 è di 86 caratteri. Correggo e la prossima volta presterò più attenzione.

Non volevo sindacare sulla qualità del tuo codice, era un riferimento generale, 
alla scarsa qualità del codice della localizzazione italiana.
Non sono uno sviluppatore, quindi non penso di poter scrivere codice migliore 
del tuo, ma apprezzo le dinamiche e gli sforzi della comunità internazionale e 
mi piacerebbe vedere lo stesso nella community italiana.

--
Davide Corio
davide.co...@lsweb.it


-- 
https://code.launchpad.net/~coopenerp/openobject-italia/7.0-adding-intra-cee/+merge/210855
Your team OpenERP Italia core devs is subscribed to branch 
lp:openobject-italia/7.0.

___
Mailing list: https://launchpad.net/~openobject-italia-core-devs
Post to : openobject-italia-core-devs@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openobject-italia-core-devs
More help   : https://help.launchpad.net/ListHelp