Review: Needs Fixing Ciao Alessandro, grazie ancora per il modulo :-)
Negli header potresti aggiungere il copyright dell'associazione? # Copyright (C) 2014 # Associazione OpenERP Italia (<http://www.openerp-italia.org>) Ovviamente se vuoi puoi anche aggiungere il tuo riga 64: come website, piuttosto che niente, potresti mettere http://www.openerp-italia.org ? righe 163, 168 e forse altre: vedo un carattere probabilmente non unicode Campi che usano il metodo _tot_operation_number: hai usato store=True. Visto che il ricalcolo dipende anche dal campo 'line_BL_ids', è sufficiente mettere store=True, o è necessario specificare il dizionario con almeno le due chiavi 'spesometro.comunicazione' e 'spesometro.comunicazione.line.bl'? In altre parole, modificando campi di 'spesometro.comunicazione.line.bl', i campi 'totale_FA' etc vengono ricalcolati? riga 320 e siccessive: il parse a int è necessario? riga 1375: _default è necessario? riga 1508: pdb (a cosa serve wizard.spesometro.default ? ) riga 1604 : serve? -- https://code.launchpad.net/~a-camilli/openobject-italia/6.1-spesometro/+merge/210439 Your team OpenERP Italia core devs is subscribed to branch lp:openobject-italia/6.1. _______________________________________________ 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
