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

l10n_br_account_payment_order/migrations/14.0.7.0.0/post-migration.py #3408

Closed
mileo opened this issue Oct 1, 2024 · 4 comments
Closed

l10n_br_account_payment_order/migrations/14.0.7.0.0/post-migration.py #3408

mileo opened this issue Oct 1, 2024 · 4 comments
Labels

Comments

@mileo
Copy link
Member

mileo commented Oct 1, 2024

  File "/opt/odoo/auto/addons/l10n_br_account_payment_order/migrations/14.0.7.0.0/post-migration.py", line 173, in migrate
    update_payment_mode_inbound(env)
  File "/opt/odoo/auto/addons/l10n_br_account_payment_order/migrations/14.0.7.0.0/post-migration.py", line 39, in update_payment_mode_inbound
    cnab_config.sending_code_id = payment_mode.sending_code_id
AttributeError: 'account.payment.mode' object has no attribute 'sending_code_id'
Error: 'account.payment.mode' object has no attribute 'sending_code_id'
@mileo mileo added the bug label Oct 1, 2024
@antoniospneto
Copy link
Contributor

@mileo tbm peguei o erro aqui, os scripts de migração das ultimas refatorações CNAB não ficaram bom, estou refazendo eles aqui para usar SQL ao invés de ORM

@antoniospneto
Copy link
Contributor

ainda to trabalhando, mas segue pr: #3409

@mbcosta
Copy link
Contributor

mbcosta commented Oct 1, 2024

@mileo esse erro deve estar acontecendo porque acredito que vocês está atualizando um Banco de Dados da versão 14.0.6.0.0 diretamente para a versão 14.0.9.0.0, para migrar esses dados era preciso rodar a atualizações passo a passo conforme os PRs foram sendo aprovados a cada nova versão deveria rodar um update:

$ odoo -d NOME_DO_BANCO -u l10n_br_account_payment_order --workers 0 --stop-after-init

image

Para contornar o problema o que pode ser feito é voltar os campos removidos aqui 68a9318 e aqui 2bc5db9 e colocar no manifest a versão 14.0.8.0.0 comentar ou renomear o arquivo https://github.com/OCA/l10n-brazil/blob/14.0/l10n_br_account_payment_order/migrations/14.0.9.0.0/pre-migration.py rodar

$ odoo -d NOME_DO_BANCO -u l10n_br_account_payment_order --workers 0 --stop-after-init

Com isso vai rodar os scripts de migração até a versão 14.0.8.0.0 e não deve gerar nenhum erro, em seguida já deverá ser possível atualizar para a versão 14.0.9.0.0

$ odoo -d NOME_DO_BANCO -u l10n_br_account_payment_order --workers 0 --stop-after-init

Onde os campos obsoletos deverão ser apagados sem erros e com isso a atualização estará concluida.

Poderia ter deixado esses campos para evitar erro e considerei mante-los até a mudança de versão v15 ou v16 mas como a migração havia sido feita esses campos precisariam ser mantidos até a v17, acreditei que estava clara a explicação sobre a cada PR ser necessário atualizar o Banco de Dados, até por isso foram feitos PRs em separado.

@rvalyi
Copy link
Member

rvalyi commented Oct 8, 2024

Resolvido aqui #3409

@rvalyi rvalyi closed this as completed Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants