// Copyright 2010 The Apache Software Foundation
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and

package org.apache.tapestry5.internal.transform;

import java.util.List;

import org.apache.tapestry5.ComponentResources;
import org.apache.tapestry5.annotations.DiscardAfter;
import org.apache.tapestry5.model.MutableComponentModel;
import org.apache.tapestry5.services.ClassTransformation;
import org.apache.tapestry5.services.ComponentClassTransformWorker;
import org.apache.tapestry5.services.ComponentMethodAdvice;
import org.apache.tapestry5.services.ComponentMethodInvocation;
import org.apache.tapestry5.services.TransformMethod;

public class DiscardAfterWorker implements ComponentClassTransformWorker
{
    
    private final ComponentMethodAdvice advice = new ComponentMethodAdvice()
    {

        public void advise(ComponentMethodInvocation invocation)
        {
            invocation.proceed();
            
            if(invocation.isFail())
                return;
            
            ComponentResources resources = invocation.getComponentResources();
            
            resources.discardPersistentFieldChanges();

        }

    };

    public void transform(final ClassTransformation transformation, final MutableComponentModel model)
    {
        final List<TransformMethod> methods = transformation.matchMethodsWithAnnotation(DiscardAfter.class);

        if (methods.isEmpty()) 
            return; 

        for (final TransformMethod metod : methods)
        {
            metod.addAdvice(advice);
        }

    }
}

@@ -1176,20 +1176,16 @@
 orker%0A%7B%0A
-    
 %0A    pri
@@ -1189,16 +1189,23 @@
 private 
+static 
 final Co
@@ -1373,36 +1373,24 @@
 .proceed();%0A
-            
 %0A           
@@ -1392,16 +1392,17 @@
       if
+ 
 (invocat
@@ -1439,28 +1439,16 @@
 return;%0A
-            
 %0A       
@@ -1519,28 +1519,16 @@
 rces();%0A
-            
 %0A       
@@ -1571,25 +1571,24 @@
 dChanges();%0A
-%0A
         %7D%0A%0A 
@@ -1843,17 +1843,16 @@
 Empty())
- 
 %0A       
@@ -1863,17 +1863,16 @@
  return;
- 
 %0A%0A      
@@ -1903,16 +1903,17 @@
 thod met
+h
 od : met
@@ -1943,16 +1943,17 @@
      met
+h
 od.addAd

Change the DiscardAfterWork's ComponentMethodAdvice to static as it does not use any members of the top-level class