Skip to content

fix: mudança no mecanismo de data de identificação para exibição na f… - #187

Merged
ncs92 merged 2 commits into
developmentfrom
fix/184-data-identificacao-undefined-ficha-tombo
Sep 5, 2025
Merged

ncs92 merged 2 commits into
developmentfrom
fix/184-data-identificacao-undefined-ficha-tombo

Conversation

@yuriBaza23

Copy link
Copy Markdown
Contributor

…ich tombo

Closes #184

O que foi feito

Somente identificado a data de identificação usando o constructor Date e mostrado usando o padrão já utilizado para datas do tombo e datas de coleta

@yuriBaza23 yuriBaza23 added this to the Ficha do tombo milestone Sep 2, 2025
@yuriBaza23 yuriBaza23 self-assigned this Sep 2, 2025
@yuriBaza23 yuriBaza23 added the bug Something isn't working label Sep 2, 2025
const dataIdentificacao = new Date(identificacao.created_at);
// eslint-disable-next-line max-len
const romanoDataIdentificacao = (`${identificacao.data_identificacao_dia}/${romanos[identificacao.data_identificacao_mes - 1]}/${identificacao.data_identificacao_ano}`);
const romanoDataIdentificacao = (`${dataIdentificacao.getDate()}/${romanos[dataIdentificacao.getMonth()]}/${dataIdentificacao.getFullYear()}`);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@yuriBaza23 acredito que o created_at não é o mesmo que data_indentificacao na tabela. Peguei como exemplo o tombo 42870, que a data de identificação é 58/2025, e o valor na coluna created_at é 26/08/2025.

Além disso, temos que lembrar que nenhuma dessas colunas é obrigatória, então pode ser que não tenha o valor de dia, ou de mes, ou de ano. O painel se comporta desta forma porque o Caxambu nem sempre sabe todos os valores que compõem a data.

Image

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@edvaldoszy Vou utilizar o tombo que fez o teste (42870) para correção. Ainda irei me preocupar com essa questão da não obrigatoriedade da data

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@edvaldoszy, segue a ficha após adaptação do código para exibição da data de identificação:

Screenshot 2025-09-03 at 10 27 12

Abrindo outras fichas para ver se tudo corre como esperado, me deparei com a ficha número 42875 que apresentava erro 500. Após deixar todos os includes como opcionais consegui chegar ao erro e tratar ele no código de exibição da ficha. Abaixo é possível ver que a causa era a inexistência do tombo.coletore.nome, que começou a mostrar undefined quando não existe.

Screenshot 2025-09-03 at 10 21 35

Aproveitei para arrumar esse detalhe com uma verificação de existência que remove esse coletor quando não informado. Veja:

Screenshot 2025-09-03 at 10 36 13

@ncs92
ncs92 merged commit 7d1d091 into development Sep 5, 2025
2 checks passed
@ncs92
ncs92 deleted the fix/184-data-identificacao-undefined-ficha-tombo branch September 5, 2025 02:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Data inválida na impressão da ficha

3 participants