openobject-italia-core-devs team mailing list archive

[Lorenzo Battistini - Agile BG <lorenzo.battistini@xxxxxxxxxxx>]

Review: Needs Fixing

Se sei d'accordo, in questa MP includerei solamente le modifiche atte a far funzionare il modulo stabile (quello sul branch ufficiale 6.1) sulla 7.0. Le correzioni ai bachi o eventuali miglioramenti al modulo le introdurrei tramite apposite merge proposals.

Quindi, facendo riferimento al diff:

l 17: Lascerei la dipendenza da account_due_list. Per velocizzare il processo di merge di tale modulo sui branch community, invito tutti a fare una review su https://code.launchpad.net/~agilebg/account-payment/adding_account_due_list_7/+merge/182330

l 72-127: sono campi/modifiche che verrebbero introdotte da account_due_list, quindi, se lasciamo la dipendenza, non sono necessari

l 207: è forse una modifica che risolverebbe qualche bug conosciuto? La terrei comunque fuori per ora

l 311, 319, 328, 336, 631: come mai viene passato il self alla chiamata ad un metodo?

l 353, 728: anche questa riga risolve forse un bug esistente?

l 440: la scriverei così: creditor_province = creditor_address.province and creditor_address.province.code or ''

l 649, 658: è necessario per il porting alla 7?

l 745-750: probabilmente un enhancement da tenere fuori da questa MP


PS: non eliminare questa MP in caso ne volessi fare un'altra, al massimo la impostiamo a rejected. Ti consiglio comunque di fare le modifiche direttamente su lp:~enlightx/openobject-italia/7.0-l10n_it_ricevute_bancarie e questa MP verrà aggiornata automaticamente.

PPS: se hai bisogno di verificare il diff tra versione 6.1 e 7.0, puoi copiare nel branch 6.1 il modulo dal branch 7.0 e fare bzr diff
-- 
https://code.launchpad.net/~enlightx/openobject-italia/7.0-l10n_it_ricevute_bancarie/+merge/194577
Your team OpenERP Italia core devs is subscribed to branch lp:openobject-italia/7.0.