Skip to content

[11.0][MIG] l10n it ricevute bancarie: Migration to 11.0#526

Closed
jackjack82 wants to merge 45 commits into
OCA:11.0from
jackjack82:11.0-mig-l10n_it_ricevute_bancarie
Closed

[11.0][MIG] l10n it ricevute bancarie: Migration to 11.0#526
jackjack82 wants to merge 45 commits into
OCA:11.0from
jackjack82:11.0-mig-l10n_it_ricevute_bancarie

Conversation

@jackjack82

Copy link
Copy Markdown
Member

No description provided.

@jackjack82 jackjack82 changed the title 11.0 mig l10n it ricevute bancarie [11.0][MIG] l10n it ricevute bancarie: Migration to 11.0 Jun 28, 2018
@jackjack82 jackjack82 force-pushed the 11.0-mig-l10n_it_ricevute_bancarie branch from b63f94a to 8629524 Compare July 2, 2018 09:12
@eLBati

eLBati commented Jul 3, 2018

Copy link
Copy Markdown
Member

Reminder: include #507

@jackjack82 jackjack82 force-pushed the 11.0-mig-l10n_it_ricevute_bancarie branch from 8629524 to b2ae533 Compare July 3, 2018 07:44
@eLBati

eLBati commented Jul 6, 2018

Copy link
Copy Markdown
Member

@jackjack82 controlli travis?

Il commit 8e7a5d0 idealmente era da aggiungere a parte , così il commit di migrazione alla 11 conteneva solo le modifiche di porting

@eLBati eLBati added needs fixing Has conflicts or is failing mandatory CI checks and removed 11.0 labels Jul 13, 2018
@eLBati eLBati added this to the 11.0 milestone Jul 13, 2018
eLBati and others added 21 commits July 16, 2018 12:31
Changes for 6.0 should be done in a branch for 6.0.
For the reason to use 'fields.date' instead of 'time', see
https://bugs.launchpad.net/openobject-server/+bug/925361
https://bugs.launchpad.net/openerp-web/+bug/918257
Moving to lp:~agilebg/openobject-italia/riba_v7_style
eLBati and others added 15 commits July 16, 2018 12:31
after odoo/odoo@efa4f85
otherwise:

2017-05-20 03:21:02,605 6389 ERROR openerp_test odoo.addons.l10n_it_ricevute_bancarie.tests.test_riba: ERROR: test_delete_due_cost_line (odoo.addons.l10n_it_ricevute_bancarie.tests.test_riba.TestInvoiceDueCost)
2017-05-20 03:21:02,605 6389 ERROR openerp_test odoo.addons.l10n_it_ricevute_bancarie.tests.test_riba: Traceback (most recent call last):
2017-05-20 03:21:02,605 6389 ERROR openerp_test odoo.addons.l10n_it_ricevute_bancarie.tests.test_riba: `   File "/home/travis/build/OCA/l10n-italy/l10n_it_ricevute_bancarie/tests/test_riba.py", line 205, in test_delete_due_cost_line
2017-05-20 03:21:02,605 6389 ERROR openerp_test odoo.addons.l10n_it_ricevute_bancarie.tests.test_riba: `     self.invoice.action_invoice_draft()
2017-05-20 03:21:02,606 6389 ERROR openerp_test odoo.addons.l10n_it_ricevute_bancarie.tests.test_riba: `   File "/home/travis/build/OCA/l10n-italy/l10n_it_ricevute_bancarie/models/account/account.py", line 213, in action_invoice_draft
2017-05-20 03:21:02,606 6389 ERROR openerp_test odoo.addons.l10n_it_ricevute_bancarie.tests.test_riba: `     line.unlink()
2017-05-20 03:21:02,606 6389 ERROR openerp_test odoo.addons.l10n_it_ricevute_bancarie.tests.test_riba: `   File "/home/travis/odoo-10.0/addons/account/models/account_invoice.py", line 1360, in unlink
2017-05-20 03:21:02,606 6389 ERROR openerp_test odoo.addons.l10n_it_ricevute_bancarie.tests.test_riba: `     raise UserError(_('You can only delete an invoice line if the invoice is in draft state.'))
2017-05-20 03:21:02,606 6389 ERROR openerp_test odoo.addons.l10n_it_ricevute_bancarie.tests.test_riba: ` UserError: (u'You can only delete an invoice line if the invoice is in draft state.', None)
File "odoo/lib/python2.7/site-packages/odoo/models.py", line 4369, in <lambda>
    @api.returns('self', lambda value: value.id)
AttributeError: 'NoneType' object has no attribute 'id'

while duplicating
…on riba line, and button 'Settle all' that allow to settle all the riba lines that are in state 'Accredited' (OCA#359)
…zione, viene usato sempre lo stesso conto. (OCA#472)

Allora per la riconciliazione prendiamo comunque la riga che ci interessa, quella con 'debit'
…form view 2) autopost settle and unsolved moves 3) bank expenses in move only if existing, does not make sense to have 0 value accounting lines (OCA#484)
prevent to use a bank accoun tfor "Ri.Ba. bank account", otherwise, if a bank account is used, its move lines will appear as blu lines in bank statement reconciliation
@jackjack82 jackjack82 force-pushed the 11.0-mig-l10n_it_ricevute_bancarie branch 2 times, most recently from 7fd7426 to efc9b91 Compare July 16, 2018 12:20
@jackjack82 jackjack82 force-pushed the 11.0-mig-l10n_it_ricevute_bancarie branch from efc9b91 to ba76d87 Compare July 16, 2018 12:24
@jackjack82

jackjack82 commented Jul 16, 2018

Copy link
Copy Markdown
Member Author

hi everybody, does someone have an example of automatic test of reports?
I have this error and can not solve it. ImportError:
No module named 'odoo.report'

thanks!

@SimoRubi

Copy link
Copy Markdown
Member

@jackjack82 look at https://github.com/odoo/odoo/blob/11.0/addons/hr_payroll/tests/test_payslip_flow.py#L6
, in v10 it had your same line from odoo.report import render_report

@pedrobaeza pedrobaeza mentioned this pull request Oct 9, 2018
35 tasks
@labaggio

Copy link
Copy Markdown

@jackjack82 cosa manca per riuscire a terminare e mergiare questa PR?

@eLBati

eLBati commented Oct 27, 2018

Copy link
Copy Markdown
Member

@jackjack82 vedi anche #622

Pensi di riuscire a sistemare questa PR?

@andreampiovesana

Copy link
Copy Markdown
Contributor

Se installo questa PR la riconciliazione bancaria si rompe!

@eLBati

eLBati commented Nov 22, 2018

Copy link
Copy Markdown
Member

Questa PR ha bisogno di amore.
Se qualcuno volesse dargliene, può partire dal branch jackjack82:11.0-mig-l10n_it_ricevute_bancarie e aggiungere le correzioni su un proprio branch

@ruben-tonetto

Copy link
Copy Markdown

La piglio io!

@ruben-tonetto

Copy link
Copy Markdown

ho rilevato 2 errori:

  1. il metodo get_move_lines_for_reconciliation() in l10n_it_ricevute_bancarie non riceve il parametro partner_id
  2. questo metodo chiama domain_move_lines_for_reconciliation() passando 2 parametri: str e excluded_ids; ma questo metodo riceve solo str

sistemando non si inchioda più

@eLBati

eLBati commented Nov 28, 2018

Copy link
Copy Markdown
Member

Chiudo in favore di #680

@eLBati eLBati closed this Nov 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs fixing Has conflicts or is failing mandatory CI checks

Projects

None yet

Development

Successfully merging this pull request may close these issues.