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

[10.0] Crash in account_payment_order #353

Closed
alexis-via opened this issue Apr 3, 2017 · 1 comment
Closed

[10.0] Crash in account_payment_order #353

alexis-via opened this issue Apr 3, 2017 · 1 comment

Comments

@alexis-via
Copy link
Contributor

When you create a payment order with a payment mode that has a variable link to a bank account, you get this crash:

Traceback (most recent call last):
  File "/home/odoo/erp/odoo/odoo/http.py", line 638, in _handle_exception
    return super(JsonRequest, self)._handle_exception(exception)
  File "/home/odoo/erp/odoo/odoo/http.py", line 675, in dispatch
    result = self._call_function(**self.params)
  File "/home/odoo/erp/odoo/odoo/http.py", line 331, in _call_function
    return checked_call(self.db, *args, **kwargs)
  File "/home/odoo/erp/odoo/odoo/service/model.py", line 119, in wrapper
    return f(dbname, *args, **kwargs)
  File "/home/odoo/erp/odoo/odoo/http.py", line 324, in checked_call
    result = self.endpoint(*a, **kw)
  File "/home/odoo/erp/odoo/odoo/http.py", line 933, in __call__
    return self.method(*args, **kw)
  File "/home/odoo/erp/odoo/odoo/http.py", line 504, in response_wrap
    response = f(*args, **kw)
  File "/home/odoo/erp/odoo/addons/web/controllers/main.py", line 878, in call_kw
    return self._call_kw(model, method, args, kwargs)
  File "/home/odoo/erp/odoo/addons/web/controllers/main.py", line 870, in _call_kw
    return call_kw(request.env[model], method, args, kwargs)
  File "/home/odoo/erp/odoo/odoo/api.py", line 679, in call_kw
    return call_kw_model(method, model, args, kwargs)
  File "/home/odoo/erp/odoo/odoo/api.py", line 664, in call_kw_model
    result = method(recs, *args, **kwargs)
  File "/home/odoo/erp/usability/account_usability/account.py", line 112, in name_search
    name=name, args=args, operator=operator, limit=limit)
  File "/home/odoo/erp/odoo/odoo/models.py", line 1591, in name_search
    return self._name_search(name, args, operator, limit=limit)
  File "/home/odoo/erp/odoo/odoo/models.py", line 1604, in _name_search
    ids = self._search(args, limit=limit, access_rights_uid=access_rights_uid)
  File "/home/odoo/erp/odoo/odoo/models.py", line 4205, in _search
    query = self._where_calc(args)
  File "/home/odoo/erp/odoo/odoo/models.py", line 4006, in _where_calc
    where_clause, where_params = e.to_sql()
  File "/home/odoo/erp/odoo/odoo/osv/expression.py", line 1287, in to_sql
    q, p = self.__leaf_to_sql(leaf)
  File "/home/odoo/erp/odoo/odoo/osv/expression.py", line 1212, in __leaf_to_sql
    raise ValueError("Invalid domain term %r" % (leaf,))
ValueError: Invalid domain term (u'id', u'in', {u'company_id': 1, u'name': u'Banque BNP', u'sequence': 10})

This is caused by the special "domain" on journal_id in the view:

domain="[('id', 'in', allowed_journal_ids and allowed_journal_ids[0] and allowed_journal_ids[0][2] or [])]"
alexis-via added a commit to akretion/banking that referenced this issue Apr 4, 2017
alexis-via pushed a commit that referenced this issue May 20, 2017
[FIX] Bug #353 crash when selecting a payment mode that has a variable link to bank accounts
@adrienpeiffer
Copy link
Contributor

Fixed on the following revision 5ce7b4c

vrenaville added a commit to vrenaville/bank-payment that referenced this issue Sep 12, 2017
…de that has a variable link to bank accounts
etobella pushed a commit to etobella/bank-payment that referenced this issue Oct 11, 2017
etobella pushed a commit to etobella/bank-payment that referenced this issue Oct 11, 2017
etobella pushed a commit to etobella/bank-payment that referenced this issue Nov 8, 2017
rven pushed a commit to rven/bank-payment that referenced this issue Oct 2, 2018
ThomasBinsfeld pushed a commit to acsone/bank-payment that referenced this issue Oct 9, 2018
sergio-teruel pushed a commit to Tecnativa/bank-payment that referenced this issue Nov 5, 2018
pedrobaeza pushed a commit to Tecnativa/bank-payment that referenced this issue Dec 13, 2018
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
sebalix pushed a commit to camptocamp/bank-payment that referenced this issue Feb 25, 2019
sebalix pushed a commit to camptocamp/bank-payment that referenced this issue Feb 25, 2019
rven pushed a commit to rven/bank-payment that referenced this issue Nov 8, 2019
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
rven pushed a commit to rven/bank-payment that referenced this issue Nov 18, 2019
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
rven pushed a commit to rven/bank-payment that referenced this issue Dec 17, 2019
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
rven pushed a commit to rven/bank-payment that referenced this issue Jan 23, 2020
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
rven pushed a commit to rven/bank-payment that referenced this issue Mar 2, 2020
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
rven pushed a commit to rven/bank-payment that referenced this issue Mar 8, 2020
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
rven pushed a commit to rven/bank-payment that referenced this issue Mar 12, 2020
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
rven pushed a commit to rven/bank-payment that referenced this issue Mar 17, 2020
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
Oscar-Valverde pushed a commit to Jarsa-dev/bank-payment that referenced this issue Oct 24, 2020
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
alan196 pushed a commit to Jarsa-dev/bank-payment that referenced this issue Nov 14, 2020
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
alan196 pushed a commit to Jarsa-dev/bank-payment that referenced this issue Nov 18, 2020
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
CarlosRoca13 pushed a commit to Tecnativa/bank-payment that referenced this issue Jan 5, 2021
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
CarlosRoca13 pushed a commit to Tecnativa/bank-payment that referenced this issue Jan 5, 2021
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
i-vyshnevska pushed a commit to camptocamp/bank-payment that referenced this issue Apr 7, 2021
Signed-off-by pedrobaeza
dnplkndll pushed a commit to Kencove/bank-payment that referenced this issue Oct 31, 2021
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
misern2 pushed a commit to QubiQ/bank-payment that referenced this issue Dec 15, 2021
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
manuelcalerosolis pushed a commit to xtendoo-corporation/bank-payment that referenced this issue Feb 3, 2022
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
misern2 pushed a commit to QubiQ/bank-payment that referenced this issue Mar 9, 2022
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
wpichler pushed a commit to wpichler/bank-payment that referenced this issue Oct 25, 2022
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
wpichler pushed a commit to wpichler/bank-payment that referenced this issue Dec 15, 2022
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
gfcapalbo pushed a commit to gfcapalbo/bank-payment that referenced this issue Jan 17, 2023
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
wpichler pushed a commit to wpichler/bank-payment that referenced this issue Jan 27, 2023
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
pedrobaeza pushed a commit to Tecnativa/bank-payment that referenced this issue Mar 4, 2023
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
angelmoya pushed a commit to SidooSL/bank-payment that referenced this issue Jan 3, 2024
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
ramiadavid pushed a commit to ramiadavid/bank-payment that referenced this issue Jan 13, 2024
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
ramiadavid pushed a commit to ramiadavid/bank-payment that referenced this issue Feb 26, 2024
* FIX: Bug OCA#353 crash when selecting a payment mode that has a variable link to a bank account
* FIX: Add 'post_move' option on account.payment.mode with default True, to keep the previous behavior
* IMP: Explicit error when bank account is missing on bank journal
* IMP: Add field default_date_prefered on payment mode.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants