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

8.0 Cannot even generate a SEPA credit transfer file ! #111

Closed
alexis-via opened this issue Jan 24, 2015 · 2 comments
Closed

8.0 Cannot even generate a SEPA credit transfer file ! #111

alexis-via opened this issue Jan 24, 2015 · 2 comments
Milestone

Comments

@alexis-via
Copy link
Contributor

Here is the scenario on v8:

  1. install the module account_banking_sepa_credit_transfer and make sure that the module account_banking_sepa_direct_debit is NOT INSTALLED
  2. create a new payment order, select a payment mode for SEPA credit transfer, select an invoice, confirm
  3. in the wizard pop-up, leave the default parameters and generate the XML file: here you get a crash:
2015-01-24 22:45:13,713 8553 ERROR o8_test2 openerp.http: Exception during JSON request handling.
Traceback (most recent call last):
  File "/home/alexis/new_boite/dev/odoo/openerp/http.py", line 525, in _handle_exception
    return super(JsonRequest, self)._handle_exception(exception)
  File "/home/alexis/new_boite/dev/odoo/openerp/http.py", line 562, in dispatch
    result = self._call_function(**self.params)
  File "/home/alexis/new_boite/dev/odoo/openerp/http.py", line 302, in _call_function
    return checked_call(self.db, *args, **kwargs)
  File "/home/alexis/new_boite/dev/odoo/openerp/service/model.py", line 113, in wrapper
    return f(dbname, *args, **kwargs)
  File "/home/alexis/new_boite/dev/odoo/openerp/http.py", line 299, in checked_call
    return self.endpoint(*a, **kw)
  File "/home/alexis/new_boite/dev/odoo/openerp/http.py", line 791, in __call__
    return self.method(*args, **kw)
  File "/home/alexis/new_boite/dev/odoo/openerp/http.py", line 395, in response_wrap
    response = f(*args, **kw)
  File "/home/alexis/new_boite/dev/odoo/addons/web/controllers/main.py", line 953, in call_button
    action = self._call_kw(model, method, args, {})
  File "/home/alexis/new_boite/dev/odoo/addons/web/controllers/main.py", line 941, in _call_kw
    return getattr(request.registry.get(model), method)(request.cr, request.uid, *args, **kwargs)
  File "/home/alexis/new_boite/dev/odoo/openerp/api.py", line 241, in wrapper
    return old_api(self, *args, **kwargs)
  File "/home/alexis/new_boite/dev/banking-80/account_banking_sepa_credit_transfer/wizard/export_sepa.py", line 153, in create_sepa
    cr, uid, pain_root, gen_args, context=context)
  File "/home/alexis/new_boite/dev/odoo/openerp/api.py", line 241, in wrapper
    return old_api(self, *args, **kwargs)
  File "/home/alexis/new_boite/dev/banking-80/account_banking_pain_base/models/banking_export_pain.py", line 196, in generate_group_header_block
    context=context)
  File "/home/alexis/new_boite/dev/odoo/openerp/api.py", line 241, in wrapper
    return old_api(self, *args, **kwargs)
  File "/home/alexis/new_boite/dev/banking-80/account_banking_pain_base/models/banking_export_pain.py", line 269, in generate_initiating_party_block
    context=context)
  File "/home/alexis/new_boite/dev/odoo/openerp/api.py", line 241, in wrapper
    return old_api(self, *args, **kwargs)
  File "/home/alexis/new_boite/dev/banking-80/account_banking_pain_base/models/res_company.py", line 47, in _get_initiating_party_identifier
    party_identifier = company.sepa_creditor_identifier
AttributeError: 'res.company' object has no attribute 'sepa_creditor_identifier'

Code analysis:
In account_banking_pain_base line 47, we have:

    party_identifier = company.sepa_creditor_identifier

but this field is declared in the module account_banking_sepa_direct_debit, which is not installed. The change was introduced by @pedrobaeza 24 days https://github.com/OCA/bank-payment/blame/8.0/account_banking_pain_base/models/res_company.py#L47

@alexis-via
Copy link
Contributor Author

OK, let's make sure that this horrible bug doesn't survive the OCA code sprint.

alexis-via pushed a commit to akretion/banking that referenced this issue Jun 1, 2015
pedrobaeza added a commit that referenced this issue Jun 4, 2015
…party_identifier

Fix bug #111 New field initiating_party_identifier on res.companyi
@yvaucher
Copy link
Member

I'm closing this as #172 was merge

cc @alexis-via @pedrobaeza

etobella pushed a commit to etobella/bank-payment that referenced this issue Oct 16, 2017
etobella pushed a commit to etobella/bank-payment that referenced this issue Nov 13, 2017
etobella pushed a commit to etobella/bank-payment that referenced this issue Nov 16, 2017
sergio-teruel pushed a commit to Tecnativa/bank-payment that referenced this issue Nov 9, 2018
pedrobaeza pushed a commit to Tecnativa/bank-payment that referenced this issue Dec 13, 2018
axelpriem pushed a commit to axelpriem/bank-payment that referenced this issue Jan 21, 2020
ValentinVinagre pushed a commit to sygel-technology/bank-payment that referenced this issue Apr 21, 2020
CarlosRoca13 pushed a commit to Tecnativa/bank-payment that referenced this issue Dec 24, 2020
alexis-via pushed a commit to akretion/banking that referenced this issue Dec 26, 2020
CarlosRoca13 pushed a commit to Tecnativa/bank-payment that referenced this issue Jan 5, 2021
alexis-via pushed a commit to akretion/banking that referenced this issue Jan 5, 2022
manuelregidor pushed a commit to sygel-technology/bank-payment that referenced this issue Mar 29, 2022
wpichler pushed a commit to wpichler/bank-payment that referenced this issue Oct 25, 2022
gfcapalbo pushed a commit to gfcapalbo/bank-payment that referenced this issue Jan 13, 2023
gfcapalbo pushed a commit to gfcapalbo/bank-payment that referenced this issue Jan 17, 2023
gfcapalbo pushed a commit to gfcapalbo/bank-payment that referenced this issue Jan 17, 2023
gfcapalbo pushed a commit to gfcapalbo/bank-payment that referenced this issue Jan 17, 2023
wpichler pushed a commit to wpichler/bank-payment that referenced this issue Jan 27, 2023
Reyes4711-S73 pushed a commit to Studio73/bank-payment that referenced this issue Mar 21, 2023
pedrobaeza pushed a commit to Tecnativa/bank-payment that referenced this issue May 27, 2023
trisdoan pushed a commit to trisdoan/bank-payment that referenced this issue Mar 4, 2024
ramiadavid pushed a commit to ramiadavid/bank-payment that referenced this issue Mar 5, 2024
trisdoan pushed a commit to trisdoan/bank-payment that referenced this issue Mar 6, 2024
trisdoan pushed a commit to trisdoan/bank-payment that referenced this issue Mar 22, 2024
This issue was closed.
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

3 participants