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

Corrigido atestado de matricula e histórico escolar para buscar corretamente nomes dos pais. #65

Conversation

lucasdavila
Copy link
Contributor

Olá segue atualização no atestado de matricula e no histórico escolar, para funcionar corretamente com o novo cadastro de alunos. Atualmente o relatório não exibe o nome dos pais informados pelo novo cadastro.

O relatório tentará primeiro exibir o nome dos pais nesta ordem:

  • nome da pessoa fisica vinculada ao cadastro do aluno.
  • nome informado manualmente no antigo cadastro de aluno.
  • por fim caso não encontre nenhum nume, exibirá 'Não informado'.

@lucasdavila lucasdavila changed the title Corrigido atestado de matricula para buscar corretamente nomes dos pais. Corrigido atestado de matricula e histórico escolar para buscar corretamente nomes dos pais. Aug 5, 2014
@lucasdavila lucasdavila force-pushed the correcao-nome-dos-pais-atestado-de-frequencia branch from 31f45b4 to 567fe55 Compare October 7, 2014 02:43
Lucas D'Avila added 2 commits October 6, 2014 23:43
* O relatório tentará primeiro exibir o nome dos pais nesta ordem:

  - nome da pessoa fisica vinculada ao cadastro do aluno.

  - nome informado  manualmente no antigo cadastro de aluno.

  - por fim caso não encontre nenhum nume, exibirá 'Não informado'.
* O relatório tentará primeiro exibir o nome dos pais nesta ordem:

  - nome da pessoa fisica vinculada ao cadastro do aluno.

  - nome informado  manualmente no antigo cadastro de aluno.

  - por fim caso não encontre nenhum nume, exibirá 'Não informado'.
@gabrielms
Copy link
Contributor

Feito!

Muito obrigado pela contribuição.

@gabrielms gabrielms closed this Oct 8, 2014
@lucasdavila lucasdavila deleted the correcao-nome-dos-pais-atestado-de-frequencia branch November 16, 2014 18:28
@lucasdavila
Copy link
Contributor Author

@gabrielms eu estava vendo este relatório na versão master e minha última correção no SQL não está lá [1], continua com aquele erro no Trim, porem nos commits deste PR está certo.

Eu acredito que você fez merge antes de eu enviar a correção e então você fechou a issue. Para corrigir o SQL definitivamente vou enviar outro pull request corrigindo novamente.

[1] https://github.com/portabilis/ieducar/blob/master/ieducar/modules/Reports/ReportSources/portabilis_atestado_matricula.jrxml#L50

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

Successfully merging this pull request may close these issues.

2 participants