Review: Needs Fixing Ciao Alessandro, grazie per il contributo!
Solo alcune osservazioni: riga 756 del diff: userei elif type == 'debit' e poi un else finale in cui sollevare eccezione se type è diverso da credit e debit. In questo modo, se mai dovesse arrivare qualcosa di sporco in 'type', l'utente verrebbe avvisato. riga 760: come mai converti a stringa? Puoi anche usare un intero come chiave del dizionario. Inoltre, per verificare che la chiave ci sia già, puoi anche solo fare if account_id not in account_amounts riga 855: dovresti poter usare objects[0].print_page_from e non doverlo quindi mettere nel parser. riga 800: metterei l'header in una sezione con noupdate="1" , di modo che le modifiche fatte dagli utenti non vengano sovrascritte dagli aggiornamenti -- https://code.launchpad.net/~a-camilli/openobject-italia/7.0_liq_IVA_opzioni_stampa/+merge/208601 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
