Não está verificando a configuração de avaliação numérica ao criar recuperação por etapa

[williamsla]

MENU:

Recuperações -> Diário de recuperações de etapas -> Novo lançamento

DESCRIÇÃO:

Configurei avaliações numéricas por etapa. No ano letivo são 4 etapas e cada etapa tem 3 avaliações utilizando somatório para calcular a média. Ao clicar no botão de criar recuperação por etapa aparece a mensagem É necessário configurar uma avaliação numérica, apesar de já ter sido configurada por etapa.

OBSERVAÇÕES:

Pelo que vi em SchoolTermRecoveryDiaryRecordsController#new só está verificando se tem avaliação numérica geral, com a função current_test_setting.. Nâo deveria verificar também a função current_test_setting_step(step) ?.

AMBIENTE:

• Plataforma utilizada: instalação direta
• Sistema operacional e versão (p.ex. Ubuntu 16.04, Windows 7): ubuntu 18.04
• Navegador e versão (p.ex. Chrome 35.9.9.9): Chrome
• Outros detalhes importantes:

[msoares94]

@williamsla, que que tem um PR para resolver esse problema, porém pelo que pude acompanhar eu precisaria atualizar a versão do meu i-Diário de 1.3.6 para a 1.4.0

Se não for pedir demais, você não saberia me informar o que eu deveria ajustar na versão 1.3.6 para que essa mensagem É necessário configurar uma avaliação numérica não seja apresentada no ato de registrar uma Diário de recuperações de etapas

Segue tela de erro e configuração de avaliações.

Campo no i-Educar que informa a recuperação por etapa!

[williamsla]

Olá @marcosoliveirasoares94 , boa noite!

Tem as modificações nesse PR https://github.com/portabilis/i-diario/pull/108/files?diff=split&w=0. Para resolver esse problema rápido basta tirar a verificação de avaliação numérica da função new no arquivo app/controllers/school_term_recovery_diary_records_controller.rb.

Como conversado com o Gustavo nesse PR, o problema é que a validação de avaliação numérica não está verificando se há configuração de avaliação numérica por período escolar. E também não dá pra inserir essa verificação no local que estava.

Assim, remove a verificação da função citada e precisamos adicionar essa validação no arquivo test_setting_fetcher.rb, conforme sugerido pelo Gustavo. Ainda não fiz isso. Dessa forma, torna-se uma solução definitiva.

Se ficou confuso a explicação, dá um retorno aqui pra gente ir alinhando. rsrs

[msoares94]

@williamsla, funcionou como uma luva, muito obrigado! Por hora irei deixar comentado, pois a minha ideia é atualizar para a nova versão do i-Diário e talvez até lá você já tenha feito o outro PR e @edersoares ou outro integrante da Portabilis feito o Merge.