Andreipt2 - Criado funcao SearchEvaluationbyData, porem ainda nao fun… #22
Conversation
rcaa
left a comment
rcaa
left a comment
There was a problem hiding this comment.
O código realmente possui vários erros. Comentei os mais relevantes que encontrei pra que você possa evitar nas próximas atividades.
| @Transactional(readOnly = true) | ||
| class EvaluationController { | ||
|
|
||
| static allowedMethods = [update: "PUT"] |
There was a problem hiding this comment.
por que todo esse código foi removido?
| } | ||
| '*'{ respond evaluationInstance, [status: OK] } | ||
| static allowedMethods = [save: "POST", update: "PUT", delete: "DELETE"] | ||
|
|
There was a problem hiding this comment.
também não entendi o porquê adicionar todo esse código
| /*if (evaluationInstance == null) { | ||
| notFound() | ||
| return | ||
| }*/ |
There was a problem hiding this comment.
talvez você tenha alterado a identação do código. Isso deve ser evitado pois aparecem várias alterações que não tem relação com a tarefa
| <label for="criterion"> | ||
| <g:message code="evaluation.criterion.label" default="Criterion" /> | ||
| <span class="required-indicator">*</span> | ||
| </label> |
There was a problem hiding this comment.
por que esse optionValue foi removido?
|
|
||
| </div> | ||
|
|
||
| <div id="list-student" class="content scaffold-list" role="main"> |
There was a problem hiding this comment.
Aqui também não entendi o motivo da remoção do código.
| </g:eachError> | ||
| </ul> | ||
| </g:hasErrors> | ||
| <g:form url="[action:'saveAll']" > |
There was a problem hiding this comment.
muito provavelmente essa mudança quebra a funcionalidade "create" de Evaluation
| <%@ page import="ta.Evaluation" %> | ||
| <!DOCTYPE html> | ||
| <html> | ||
| <head> |
There was a problem hiding this comment.
aparentemente aqui ocorre o mesmo erro: alterar identação. Isso dificulta muito a revisão do código.
|
|
||
| <td>${fieldValue(bean: evaluationInstance, field: "value")}</td> | ||
|
|
||
| <td>${fieldValue(bean: evaluationInstance, field: "criterion.description")}</td> |
There was a problem hiding this comment.
algo vai deixar de funcionar com essa alteração.
| <table> | ||
| <thead> | ||
| <tr> | ||
|
|
There was a problem hiding this comment.
evitar pular linha desnecessariamente.
| <li class="fieldcontain"> | ||
| <span id="criterion-label" class="property-label"><g:message code="evaluation.criterion.label" default="Criterion" /></span> | ||
|
|
||
| <span class="property-value" aria-labelledby="criterion-label"><g:link controller="criterion" action="show" id="${evaluationInstance?.criterion?.id}">${evaluationInstance?.criterion?.description?.encodeAsHTML()}</g:link></span> |
There was a problem hiding this comment.
essa alteração tambémdeve quebrar alguma funcionalidade.
2 participants
…cional.