You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by hl...@apache.org on 2011/05/18 00:06:04 UTC

svn commit: r1104591 - /tapestry/tapestry5/trunk/plastic/src/main/java/org/apache/tapestry5/plastic/PlasticManager.java

Author: hlship
Date: Tue May 17 22:06:04 2011
New Revision: 1104591

URL: http://svn.apache.org/viewvc?rev=1104591&view=rev
Log:
TAP5-853: Move call to check() before mutation methods of PlasticManagerBuilder

Modified:
    tapestry/tapestry5/trunk/plastic/src/main/java/org/apache/tapestry5/plastic/PlasticManager.java

Modified: tapestry/tapestry5/trunk/plastic/src/main/java/org/apache/tapestry5/plastic/PlasticManager.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/plastic/src/main/java/org/apache/tapestry5/plastic/PlasticManager.java?rev=1104591&r1=1104590&r2=1104591&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/plastic/src/main/java/org/apache/tapestry5/plastic/PlasticManager.java (original)
+++ tapestry/tapestry5/trunk/plastic/src/main/java/org/apache/tapestry5/plastic/PlasticManager.java Tue May 17 22:06:04 2011
@@ -78,19 +78,19 @@ public class PlasticManager implements P
          */
         public PlasticManagerBuilder packages(Collection<String> packageNames)
         {
-            packages.addAll(packageNames);
-
             check();
 
+            packages.addAll(packageNames);
+
             return this;
         }
 
         public PlasticManagerBuilder enable(TransformationOption option)
         {
-            options.add(option);
-
             check();
 
+            options.add(option);
+
             return this;
         }