Skip to content

[18.0] [IMP] l10n_it_ricevute_bancarie: Settle lines with date from list view - #5095

Open
odooNextev wants to merge 2 commits into
OCA:18.0from
odooNextev:18.0-imp-l10n_it_ricevute_bancarie-settle_multiple_list
Open

odooNextev wants to merge 2 commits into
OCA:18.0from
odooNextev:18.0-imp-l10n_it_ricevute_bancarie-settle_multiple_list

Conversation

@odooNextev

Copy link
Copy Markdown
Contributor

#4728

FW port: #5094

@monen17 monen17 added the is porting This pull request is porting a change from another version label Feb 20, 2026

@monen17 monen17 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Grazie della PR!
Ho fatto solo revisione del codice, vedi i commenti qui sotto.

P.S.: Se poi riesci anche ad aggiungere un test evitiamo di dover cercare questi typo a occhio la prossima volta 👀

Comment thread l10n_it_riba_oca/models/riba.py Outdated
Comment thread l10n_it_riba_oca/models/riba.py Outdated
@odooNextev

Copy link
Copy Markdown
Contributor Author

Grazie della PR! Ho fatto solo revisione del codice, vedi i commenti qui sotto.

P.S.: Se poi riesci anche ad aggiungere un test evitiamo di dover cercare questi typo a occhio la prossima volta 👀

Sì, infatti, non so come siano passati ugualmente.

@monen17

monen17 commented Feb 23, 2026

Copy link
Copy Markdown
Contributor

Grazie della PR!
Ho fatto solo revisione del codice, vedi i commenti qui sotto.

P.S.: Se poi riesci anche ad aggiungere un test evitiamo di dover cercare questi typo a occhio la prossima volta 👀

Sì, infatti, non so come siano passati ugualmente.

I test esistenti passano perché non eseguono/verificano le parti rotte del codice aggiunte da questa PR.

@stenext
stenext force-pushed the 18.0-imp-l10n_it_ricevute_bancarie-settle_multiple_list branch from 96d7954 to 736aa76 Compare February 23, 2026 13:27
@odooNextev

Copy link
Copy Markdown
Contributor Author

I test esistenti passano perché non eseguono/verificano le parti rotte del codice aggiunte da questa PR.

Sì, non c'erano test che riguardavano button_settle perciò ne ho aggiunti un paio in un nuovo commit perchè non era previsto nel FW porting

@odooNextev
odooNextev requested a review from monen17 February 23, 2026 13:39

@monen17 monen17 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Grazie mille dei test 😊
Oltre alle note qui sotto ho notato che va aggiornato anche il nome del modulo nei commit da l10n_it_ricevute_bancarie a l10n_it_riba_oca per allinearlo al nuovo nome del modulo: Image

Potresti anche mantenere l'autore del commit originale 96ceb1b nel commit fw-portato 1435c91?

Comment thread l10n_it_riba_oca/tests/test_riba.py
Comment thread l10n_it_riba_oca/readme/CONTRIBUTORS.md Outdated
Comment thread l10n_it_riba_oca/models/riba.py Outdated
Comment thread l10n_it_riba_oca/models/riba.py Outdated
@stenext
stenext force-pushed the 18.0-imp-l10n_it_ricevute_bancarie-settle_multiple_list branch from 736aa76 to e7230d5 Compare March 13, 2026 08:34
@odooNextev
odooNextev requested a review from monen17 March 13, 2026 08:37
@stenext
stenext force-pushed the 18.0-imp-l10n_it_ricevute_bancarie-settle_multiple_list branch from e7230d5 to db1ff7c Compare March 13, 2026 08:40

@monen17 monen17 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revisione del codice: per me è ok, grazie!

Comment thread l10n_it_riba_oca/tests/test_riba.py
Comment thread l10n_it_riba_oca/readme/CONTRIBUTORS.md Outdated
Comment on lines 36 to 38
- successivamente a pagamento effettivamente avvenuto selezionando la
registrazione dalla vista ed elenco ed eseguendo l'azione "Imposta data
di pagamento RiBa".

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Non è possibile modificare la data successivamente, come indicato nel readme.

Image

Le altre due modalità di modifica della data pagamento funzionano correttamente.
Non darei la possibilità di modificare la data di una registrazione confermata dalla vista ad elenco.
Piuttosto indichiamo che a pagamento effettuato, è possibile rimettere in bozza la registrazione e modificare la data, (perdendo le riconciliazioni, che possono essere rifatte a mano) come da standard Odoo

@MarcoCalcagni

Copy link
Copy Markdown

/ocabot rebase

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, PR rebased to 18.0.

@OCA-git-bot
OCA-git-bot force-pushed the 18.0-imp-l10n_it_ricevute_bancarie-settle_multiple_list branch from db1ff7c to dd39de1 Compare July 10, 2026 09:32
@OCA-git-bot OCA-git-bot added the mod:l10n_it_riba_oca Module l10n_it_riba_oca label Jul 10, 2026
@MaurizioPellegrinet

Copy link
Copy Markdown
  • non ho possibilità di confermare ricevute (slip detail) su distinte diverse (dà errore, vedi sotto)

  • non permette di confermare su singola distinta un numero diricevute diverso dal totale di quelle presentate

  • se ho segnato una ricevuta come insoluta non permette all'interno della distinta di confermare le altre come pagate (in questo caso devo andare su slip detail e confermare tutte quelle rimaste inerenti quella distinta)

  • dettaglio errore Odoo Server Error
    RPC_ERROR
    Odoo Server Error
    Occured on oca-l10n-italy-18-0-pr5095-dd39de132342.runboat.odoo-community.org on model riba.payment.multiple on 2026-07-10 09:42:44 GMT
    Traceback (most recent call last):
    File "/opt/odoo/odoo/http.py", line 2167, in _transactioning
    return service_model.retrying(func, env=self.env)
    File "/opt/odoo/odoo/service/model.py", line 157, in retrying
    result = func()
    File "/opt/odoo/odoo/http.py", line 2134, in _serve_ir_http
    response = self.dispatcher.dispatch(rule.endpoint, args)
    File "/opt/odoo/odoo/http.py", line 2382, in dispatch
    result = self.request.registry['ir.http']._dispatch(endpoint)
    File "/opt/odoo/odoo/addons/base/models/ir_http.py", line 333, in _dispatch
    result = endpoint(**request.params)
    File "/opt/odoo/odoo/http.py", line 754, in route_wrapper
    result = endpoint(self, *args, **params_ok)
    File "/opt/odoo/addons/web/controllers/dataset.py", line 42, in call_button
    action = call_kw(request.env[model], method, args, kwargs)
    File "/opt/odoo/odoo/api.py", line 535, in call_kw
    result = getattr(recs, name)(*args, **kwargs)
    File "/mnt/data/odoo-addons-dir/l10n_it_riba_oca/wizard/wizard_riba_multiple_payment.py", line 92, in pay
    sbf_lines.riba_line_settlement(
    File "/mnt/data/odoo-addons-dir/l10n_it_riba_oca/models/riba.py", line 509, in riba_line_settlement
    ("move_id", "=", self.credit_move_id.id),
    File "/opt/odoo/odoo/fields.py", line 5381, in get
    raise ValueError("Expected singleton: %s" % record)
    ValueError: Expected singleton: account.move(54, 60)

The above server error caused the following client error:
RPC_ERROR: Odoo Server Error
RPC_ERROR
at makeErrorFromResponse (http://oca-l10n-italy-18-0-pr5095-dd39de132342.runboat.odoo-community.org/web/assets/df82606/web.assets_web.min.js:3165:165)
at XMLHttpRequest. (http://oca-l10n-italy-18-0-pr5095-dd39de132342.runboat.odoo-community.org/web/assets/df82606/web.assets_web.min.js:3171:13)

@monen17

monen17 commented Aug 28, 2026

Copy link
Copy Markdown
Contributor
  • non ho possibilità di confermare ricevute (slip detail) su distinte diverse (dà errore, vedi sotto)

    • non permette di confermare su singola distinta un numero diricevute diverso dal totale di quelle presentate

    • se ho segnato una ricevuta come insoluta non permette all'interno della distinta di confermare le altre come pagate (in questo caso devo andare su slip detail e confermare tutte quelle rimaste inerenti quella distinta)

    • dettaglio errore Odoo Server Error
      RPC_ERROR
      Odoo Server Error
      Occured on oca-l10n-italy-18-0-pr5095-dd39de132342.runboat.odoo-community.org on model riba.payment.multiple on 2026-07-10 09:42:44 GMT
      Traceback (most recent call last):
      File "/opt/odoo/odoo/http.py", line 2167, in _transactioning
      return service_model.retrying(func, env=self.env)
      File "/opt/odoo/odoo/service/model.py", line 157, in retrying
      result = func()
      File "/opt/odoo/odoo/http.py", line 2134, in _serve_ir_http
      response = self.dispatcher.dispatch(rule.endpoint, args)
      File "/opt/odoo/odoo/http.py", line 2382, in dispatch
      result = self.request.registry['ir.http']._dispatch(endpoint)
      File "/opt/odoo/odoo/addons/base/models/ir_http.py", line 333, in _dispatch
      result = endpoint(**request.params)
      File "/opt/odoo/odoo/http.py", line 754, in route_wrapper
      result = endpoint(self, *args, **params_ok)
      File "/opt/odoo/addons/web/controllers/dataset.py", line 42, in call_button
      action = call_kw(request.env[model], method, args, kwargs)
      File "/opt/odoo/odoo/api.py", line 535, in call_kw
      result = getattr(recs, name)(*args, **kwargs)
      File "/mnt/data/odoo-addons-dir/l10n_it_riba_oca/wizard/wizard_riba_multiple_payment.py", line 92, in pay
      sbf_lines.riba_line_settlement(
      File "/mnt/data/odoo-addons-dir/l10n_it_riba_oca/models/riba.py", line 509, in riba_line_settlement
      ("move_id", "=", self.credit_move_id.id),
      File "/opt/odoo/odoo/fields.py", line 5381, in get
      raise ValueError("Expected singleton: %s" % record)
      ValueError: Expected singleton: account.move(54, 60)

The above server error caused the following client error: RPC_ERROR: Odoo Server Error RPC_ERROR at makeErrorFromResponse (http://oca-l10n-italy-18-0-pr5095-dd39de132342.runboat.odoo-community.org/web/assets/df82606/web.assets_web.min.js:3165:165) at XMLHttpRequest. (http://oca-l10n-italy-18-0-pr5095-dd39de132342.runboat.odoo-community.org/web/assets/df82606/web.assets_web.min.js:3171:13)

Grazie @MaurizioPellegrinet delle prove, credo che alcune delle cose che hai segnalato non dipendano da questa PR: ho trovato la segnalazione #5247 che è molto simile, potresti verificare se #5286 risolve?
Quando sarà mergiata e inclusa in questa PR vedremo se manca ancora qualcosa.

@eLBati

eLBati commented Sep 11, 2026

Copy link
Copy Markdown
Member

Segnalo che sto lavorando su #5308 , che rende questa obsoleta

@eLBati

eLBati commented Sep 14, 2026

Copy link
Copy Markdown
Member

#5308 ready, chiudo questa

@monen17

monen17 commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

#5308 ready, chiudo questa

Quando #5308 sarà mergiata si potrà chiudere, fino ad allora questa potrebbe servire

@monen17 monen17 reopened this Sep 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

is porting This pull request is porting a change from another version mod:l10n_it_riba_oca Module l10n_it_riba_oca series:18.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

l10n_it_ricevute_bancarie - Registrazione riba a scadenza in dettaglio distinte riba

10 participants