openobject-italia-core-devs team mailing list archive

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

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.