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. _______________________________________________ Mailing list: https://launchpad.net/~openobject-italia-core-devs Post to : [email protected] Unsubscribe : https://launchpad.net/~openobject-italia-core-devs More help : https://help.launchpad.net/ListHelp
