Skip to content

Conversation

@vsnepomuceno
Copy link
Contributor

Entrega da atividade 5.

Copy link
Member

Choose a reason for hiding this comment

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

OK, isso vai rescrever o save do controlador, mas com que intuito? associado a que feature? o CK e o FM não foram modificados?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Eu estava com um problema com esse save, quando a chamada era feita no teste de controlador e sobrescrever o save retirando a chamada do método "render" funcionou, mas eu descobri depois como fazer funcionar sem o aspecto. Por isso, esse aspecto não está agregando nada. Eu criei e deixei para validar o entendimento sobre metaclasse. Acho melhor retirar, ou achar um propósito diferente pra ele.

Copy link
Member

Choose a reason for hiding this comment

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

Melhor retirar mesmo

On Jul 22, 2014, at 5:01 PM, vsnepomuceno notifications@github.com wrote:

In grails-app/controllers/rgms/researchProject/aspect/ResearchProjectControllerAspect.groovy:

@@ -0,0 +1,25 @@
+package rgms.researchProject.aspect
+
+import rgms.researchProject.ResearchProject
+import rgms.researchProject.ResearchProjectController
+
+/**

  • * Created by Vilmar Nepomuceno on 10/07/2014.
  • */
    +class ResearchProjectControllerAspect {
    +
  • def mc = ResearchProjectController.metaClass
  • def init() {
  •    mc.save = { ->
    
    Eu estava com um problema com esse save, quando a chamada era feita no teste de controlador e sobrescrever o save retirando a chamada do método "render" funcionou, mas eu descobri depois como fazer funcionar sem o aspecto. Por isso, esse aspecto não está agregando nada. Eu criei e deixei para validar o entendimento sobre metaclasse. Acho melhor retirar, ou achar um propósito diferente pra ele.


Reply to this email directly or view it on GitHub.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK.

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