Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16.0] bank-payment/tree/16.0/account_banking_sepa_credit_transfer error when adding bill to outgoing payment #1178

Closed
ThijsvOers opened this issue Nov 16, 2023 · 2 comments · Fixed by #1188
Assignees
Labels
bug stale PR/Issue without recent activity, it'll be soon closed automatically.

Comments

@ThijsvOers
Copy link
Contributor

Hi, I have tested module account_banking_sepa_credit_transfer with 2 vendorbills (see screenshot from the runboat db). I was able to add the first vendorbill to the outgoing payment. The second bill gave at first the error as mentioned below the moment I tried to add the vendor bill to the payment order.

Further testing led to to the conclusion that the error occurs when field 'Bill reference'is not filled in. Issue is that this field is not mandatory After adding the bill reference to the second vendor bill I could add the bill to the payment order.

So, in my opinion the reference bill should either be mandatory or the field should not mandatory for the outgoing payment order (I do not know what is mandatory for the sepa payment).

image

Error:
Traceback (most recent call last):
File "/opt/odoo/odoo/http.py", line 1589, in _serve_db
return service_model.retrying(self._serve_ir_http, self.env)
File "/opt/odoo/odoo/service/model.py", line 133, in retrying
result = func()
File "/opt/odoo/odoo/http.py", line 1616, in _serve_ir_http
response = self.dispatcher.dispatch(rule.endpoint, args)
File "/opt/odoo/odoo/http.py", line 1820, in dispatch
result = self.request.registry['ir.http']._dispatch(endpoint)
File "/opt/odoo/odoo/addons/base/models/ir_http.py", line 154, in _dispatch
result = endpoint(**request.params)
File "/opt/odoo/odoo/http.py", line 697, in route_wrapper
result = endpoint(self, *args, **params_ok)
File "/opt/odoo/addons/web/controllers/dataset.py", line 42, in call_kw
return self._call_kw(model, method, args, kwargs)
File "/opt/odoo/addons/web/controllers/dataset.py", line 33, in _call_kw
return call_kw(request.env[model], method, args, kwargs)
File "/opt/odoo/odoo/api.py", line 466, in call_kw
result = _call_kw_multi(method, model, args, kwargs)
File "/opt/odoo/odoo/api.py", line 453, in _call_kw_multi
result = method(recs, *args, **kwargs)
File "/opt/odoo/odoo/models.py", line 6538, in onchange
todo = [
File "/opt/odoo/odoo/models.py", line 6541, in
if name not in done and snapshot0.has_changed(name)
File "/opt/odoo/odoo/models.py", line 6324, in has_changed
return self[name] != record[name]
File "/opt/odoo/odoo/models.py", line 5897, in getitem
return self._fields[key].get(self, type(self))
File "/opt/odoo/odoo/fields.py", line 2797, in get
return super().get(records, owner)
File "/opt/odoo/odoo/fields.py", line 1157, in get
self.recompute(record)
File "/opt/odoo/odoo/fields.py", line 1365, in recompute
apply_except_missing(self.compute_value, recs)
File "/opt/odoo/odoo/fields.py", line 1338, in apply_except_missing
func(records)
File "/opt/odoo/odoo/fields.py", line 1387, in compute_value
records._compute_field_value(self)
File "/opt/odoo/odoo/models.py", line 4221, in _compute_field_value
fields.determine(field.compute, self)
File "/opt/odoo/odoo/fields.py", line 97, in determine
return needle(*args)
File "/mnt/data/odoo-addons-dir/account_payment_order/models/account_payment_line.py", line 185, in _compute_payment_line
communication_type, communication = move_line._get_communication()
File "/mnt/data/odoo-addons-dir/account_payment_order/models/account_move_line.py", line 25, in _get_communication
communication = self.move_id._get_payment_order_communication_full()
File "/mnt/data/odoo-addons-dir/account_payment_order/models/account_move.py", line 94, in _get_payment_order_communication_full
communication += " " + " ".join(references)
TypeError: sequence item 0: expected str instance, bool found

The above server error caused the following client error:
null

@pedrobaeza
Copy link
Member

Check #1188

angelmoya pushed a commit to SidooSL/bank-payment that referenced this issue Jan 3, 2024
…nt_order_order_communication_direct() method.

Related to: OCA#1178
ramiadavid pushed a commit to ramiadavid/bank-payment that referenced this issue Jan 13, 2024
…nt_order_order_communication_direct() method.

Related to: OCA#1178
ramiadavid pushed a commit to ramiadavid/bank-payment that referenced this issue Feb 26, 2024
…nt_order_order_communication_direct() method.

Related to: OCA#1178
Copy link

There hasn't been any activity on this issue in the past 6 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this issue to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label May 26, 2024
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Jun 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug stale PR/Issue without recent activity, it'll be soon closed automatically.
Projects
None yet
3 participants