Skip to content
New issue

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

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

Already on GitHub? # to your account

fix(dynamic-view): corrige exibicao de campos com opcoes #2007

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

carlosdiego
Copy link

< dynamic-view > corrige exibição de campos com opções

Corrige um problema identificado na versão 16.4.0 do PO-UI, onde campos com opções no PoDynamicView estavam exibindo valores incorretamente, utilizando fieldLabel em vez de property. Esta correção aborda especificamente:

  • A lógica de processamento que permitia a exibição equivocada de valores baseados em fieldLabel e fieldValue.
  • A remoção da prática de definir o valor de uma propriedade baseada em fieldLabel, assegurando a independência entre a representação visual de um campo e seus dados subjacentes.

Este ajuste garante que a apresentação de campos dinâmicos no dynamic-view seja precisa e consistente.

< #2006 >


PR Checklist [Revisor]

  • Padrão de Commit (Coeso, de acordo com o que está sendo realizado)
  • Código (Boas práticas, nome de variavéis/métodos, etc.)
  • Testes unitários (Cobre a situação implementada e coverage está mantido)
  • Documentação (Clara, objetiva e com exemplos caso necessário)
  • Samples (A implementação possui exemplo no Labs/Caso de uso)
  • Rodado em navegadores suportados (Chrome, FireFox, Edge)

Qual o comportamento atual?
No componente PoDynamicView do PO-UI, campos com opções estão sendo exibidos incorretamente devido a uma lógica que utiliza fieldLabel e fieldValue para determinar o valor a ser exibido. Isso ocorre porque o sistema não utiliza apropriadamente a propriedade property para definir o valor do campo, além de definir indevidamente o valor de uma propriedade baseando-se em fieldLabel, contrariando as diretrizes do PoDynamicView e boas práticas de desenvolvimento.

Qual o novo comportamento?
Com a correção aplicada, o PoDynamicView agora respeita corretamente a propriedade property na exibição de campos com opções, eliminando o uso de fieldLabel como critério para definir o valor de uma propriedade. A lógica de exibição foi ajustada para garantir que a visualização dos dados seja precisa e consistente, tratando de forma independente a representação visual dos campos e seus dados subjacentes.

Simulação
Para reproduzir o bug relatado basta acessar o stackblitz:

Versão PO-UI 16.4.1: Exemplo do erro
Versão PO-UI 16.3.0: Exemplo com a versão anterior do PO-UI sem o erro

Copy link

There was an error handling pipeline event e13c4b1f-06ad-4904-a520-dfa27cc7b550.

@CSimoesJr
Copy link
Contributor

Boa noite @carlosdiego, conforme conversamos, sigo aguardando correção no fato que esperava-se que fosse exibido a descrição correspondente ao id, porém isso não está acontecendo

@alinelariguet alinelariguet added status: awaiting author Additional information has been requested from the author contribution labels Jan 15, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
contribution status: awaiting author Additional information has been requested from the author
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants