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

[14.0][FIX] spec_driven_model: multiple models with _stacked property #2715

Closed
wants to merge 5 commits into from

Conversation

felipezago
Copy link
Contributor

Ao implementar a MDF-e (#2603) me deparei com um problema ao instalar os módulos l10n_br_nfe e l10n_br_mdfe simultaneamente, pois ambos os módulos fazem override do modelo l10n_br_fiscal.document e definem os campos do spec, como _stacked e _spec_module, e nesse caso o valor desses atributos acaba sendo sobrescrito, ocorrendo problemas nos hooks e na exportação de XML.

Para solucionar esse problema, eu realizo a busca da própria classe do módulo onde foi feito o override, dessa forma conseguindo os atributos originais e não os sobrescritos pelo odoo.

@mileo @lfdivino @rvalyi

@OCA-git-bot
Copy link
Contributor

Hi @rvalyi,
some modules you are maintaining are being modified, check this out!

@rvalyi
Copy link
Member

rvalyi commented Sep 26, 2023

não tou muito empolgado com isso de "frame analysis", pois normalmente é lento... Entendo o hack para vcs resolver a situação de vcs, mas eu imagino que podemos fazer algo melhor. Comento sobre isso em breve. Vamos com calma com essas coisas...

Copy link
Member

@rvalyi rvalyi left a comment

Choose a reason for hiding this comment

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

Eu diria que isso é tapar o sol com a peneira, pois tem bastante outros lugares onde problemas do tipo vão aparecer (pois tem varias propriedades - herdadas pelo spec.mixin.nfe/sped.mixin.mdfe por examplo - e varias situações, como na importação). Legal se ajuda a resolver no projeto de vcs, mas vou propor algo mais genérico nesses dias. Melhor não fazer o merge disso porque vai aumentar a complexidade do codigo sem resolver de vez, com risco de poluir o code base para sempre depois. Depois que tiver testes de serializão/importação no modulo l10n_br_mdfe por examplo ai vai ficar mais tranquilo avaliar o que resolve ou não resolve no modulo spec_driven_model.

@rvalyi
Copy link
Member

rvalyi commented Jun 17, 2024

Pessoal, continuo com a mesma opiniao. Da para fazer tudo ao mesmo tempo nao, mas ja que destravei as coisas do SPED ou da importacaio de NFe, eu consigo dar um gas nesse ponto essas semanas. Migrar a NFe para a v15 e v16 me ajudou a mergulhar nisso de novo... cc @marcelsavegnago

@mileo mileo force-pushed the fix/multiple_stacked_models branch from 3115e4d to b4f9991 Compare June 19, 2024 16:57
Copy link

There hasn't been any activity on this pull request in the past 4 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 PR 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 Oct 20, 2024
@rvalyi
Copy link
Member

rvalyi commented Oct 20, 2024

Resolvido em #3431

@rvalyi rvalyi closed this Oct 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale PR/Issue without recent activity, it'll be soon closed automatically.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants