Merge lp:~scigghia/openobject-italia/riba_v7_scigghia into lp:~openobject-italia-core-devs/openobject-italia/adding_riba_v7

[Andrea Cometa]

fix contenente controllo sulla presenza banca partner, con relativo messaggio di errore

[Andrea Cometa (scigghia)]

testato

review: Approve

[giudalo (g-dalo)]

adesso non sono + convinto

review: Approve

[Francesco Apruzzese (opencode)]

Approvo il commento scritto sopra

[Andrea Cometa (scigghia)]

ho aggiunto anche un controllo sulla lunghezza della città della banca, il tracciato riporta posizione da 46 a 70 (comune e sigla prov.)
non c'era controllo, ora viene troncata la stringa

[Francesco Apruzzese (opencode)]

Good Job, Community! Good Job!

review: Approve

[giudalo (g-dalo)]

Bene

review: Approve

[andrea_cg (a-gallina)]

Bravo!

review: Approve

[Davide Corio (enlightx-deactivatedaccount)]

Mi astengo nel senso che in questo modulo ci sarebbero talmente tante cose da sistemare che sarebbe inutile controllare questo MP :-)

Per me quindi potete procedere, anche all'inclusione del modulo in openobject-italia/7.0

review: Abstain

[Andrea Cometa (scigghia)]

ok allora faccio un resubmit e lo propongo su openobject-italia

[Andrea Cometa (scigghia)]

https://code.launchpad.net/~scigghia/openobject-italia/riba_v7_scigghia/+merge/207478

[Andrea Cometa (scigghia)]

come non detto, torno qui a chiedere il merge

=== modified file 'l10n_it_ricevute_bancarie/wizard/riba_file_export.py'
--- l10n_it_ricevute_bancarie/wizard/riba_file_export.py	2013-12-16 15:22:58 +0000
+++ l10n_it_ricevute_bancarie/wizard/riba_file_export.py	2014-02-20 15:19:44 +0000
@@ -179,7 +179,7 @@
                raise orm.except_orm('Error', _('No IBAN specified for ') + line.partner_id.name)
             debit_abi = debit_bank.iban[5:10]
             debit_cab = debit_bank.iban[10:15]
-            debitor_city = debitor_address.city or ''
+            debitor_city = debitor_address.city and debitor_address.city.ljust(23)[0:23] or ''
             debitor_province = debitor_address.province.code or ''
             if not line.due_date: # ??? VERIFICARE
                 due_date =  '000000'
@@ -188,6 +188,8 @@
 
             if not line.partner_id.vat and not line.partner_id.fiscalcode:
                 raise orm.except_orm('Error', _('No VAT or Fiscal code specified for ') + line.partner_id.name)
+            if not (debit_bank.bank and debit_bank.bank.name or debit_bank.bank_name):
+                raise orm.except_orm('Error', _('No debit_bank specified for ') + line.partner_id.name)
             Riba = [
                         line.sequence,
                         due_date,