Skip to content

Conversation

wsteixeira
Copy link
Contributor

Foi incluído o $filter para filtros complexos no GET Request quando o atributo range está com true

Fixes #1211

Page Dynamic Table

1211


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?
Quando é usado o atributo range a URL da requisição está sendo gerada com um objeto e não funciona conforme esperado.

Qual o novo comportamento?
Quando é usado o atributo range a URL da requisição está sendo gerado o atributo $filter para filtro complexo conforme o Guia de APIs da Totvs - Filtros complexos

Simulação

A simulação pode ser feita no App.

  • Para uso completo do App, será necessário configurar o proxy ou copiar a requisição e colar no navegador.
  • Será necessário logar no Datasul e o usuário se senha esta descrito em TDN, use a senha que tem a letra 'z'.

Exemplo de requisição: GET

payload

view

@wsteixeira wsteixeira force-pushed the page-dynamic-table/1211 branch from a564fd8 to 21fcdcc Compare March 3, 2022 22:24
@alinelariguet alinelariguet added the DTHFUI-11668 DTHFUI-11668 label Jul 15, 2025
@wsteixeira wsteixeira force-pushed the page-dynamic-table/1211 branch 2 times, most recently from 6b57318 to 939724e Compare July 25, 2025 14:32
Foi incluído o `$filter` para filtros complexos no GET Request
quando o atributo `range` está com `true`

Fixes po-ui#1211
@wsteixeira wsteixeira force-pushed the page-dynamic-table/1211 branch from 939724e to 749a46f Compare July 25, 2025 16:06
@arthur-polidorio
Copy link
Contributor

Olá @wsteixeira, agradecemos imensamente por sua contribuição!

Durante a análise, identificamos alguns pontos que necessitam de ajustes para que possamos prosseguir com a aprovação. São eles:

  1. Ajustar o commit para o page-dynamic-search ao invés do page-dynamic-table.

  2. No trecho:
    complexFilter = !complexFilter ? '' : (complexFilter += ' and ');
    Realizar a adequação do uso do operador de atribuição composta, a fim de evitar a geração de code smell no código.
    Sugestão de implementação com efeito análogo:

image
  1. No trecho:
    complexFilter += ${property} ge ${filter[property].start} and ${property} le ${filter[property].end};
    Adequar a concatenação dos valores, garantindo o uso de aspas simples conforme as diretrizes estabelecidas no Guia de APIs da Totvs - Filtros complexos
    Sugestão de implementação:
image image
  1. Ajustar a exibição dos valores do disclaimer e tooltip para cenários em que o filtro avançado contenha o atributo range.
image

Ficamos à disposição para esclarecer qualquer dúvida e apoiar no que for necessário. Mais uma vez, obrigado por sua colaboração!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PoPageDynamicTableComponent - URL inadequada
4 participants