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

Reordena apresentação de Revista-Fascículo #931

Merged
merged 2 commits into from Oct 9, 2019

Conversation

@rfdeoliveira
Copy link
Contributor

commented Oct 7, 2019

Issue relacionada: #806

@viniciusandrade

This comment has been minimized.

Copy link
Member

commented Oct 8, 2019

@rfdeoliveira
Apliquei esse PR no ambiente de testes e fiz a busca por "Trab. educ. saúde" e selecionei o filtro tipo de registro "Revista - Fascículo" mas aparentemente a listagem continua sem o ordenamento proposto, dei um olhada rápida no código e me parece OK.

Poderia verificar no ambiente de testes se estou esquecendo de algum detalhe.

@rfdeoliveira

This comment has been minimized.

Copy link
Contributor Author

commented Oct 8, 2019

@viniciusandrade
encontrei o problema, faltou passar o valor da primeira posição da string como parâmetro da função Substr, só havia passado o comprimento dela. Não havia percebido isso nos testes do meu ambiente, pois estava usando datas completas e em sequência, no ambiente de testes, haviam datas no seguinte formato 20191000, que gera a substring 91000, ao invés de 2019. Fiz a correção e o push do commit.

@viniciusandrade viniciusandrade merged commit bac298d into bireme:master Oct 9, 2019
@viniciusandrade

This comment has been minimized.

Copy link
Member

commented Oct 9, 2019

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.