You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@velocity.apache.org by nb...@apache.org on 2008/10/25 01:28:06 UTC
svn commit: r707788 - in
/velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts:
SecureLinkTool.java StrutsLinkTool.java
Author: nbubna
Date: Fri Oct 24 16:28:06 2008
New Revision: 707788
URL: http://svn.apache.org/viewvc?rev=707788&view=rev
Log:
pay more attention to the comments being copied/pasted
Modified:
velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts/SecureLinkTool.java
velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts/StrutsLinkTool.java
Modified: velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts/SecureLinkTool.java
URL: http://svn.apache.org/viewvc/velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts/SecureLinkTool.java?rev=707788&r1=707787&r2=707788&view=diff
==============================================================================
--- velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts/SecureLinkTool.java (original)
+++ velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts/SecureLinkTool.java Fri Oct 24 16:28:06 2008
@@ -114,10 +114,6 @@
@Override
protected void configure(ValueParser props)
{
- // request values override configured defaults
- //NOTE: not sure this is the most intuitive way in all cases;
- // it might make sense to provide the option of whether req/res
- // values override configured ones or vice versa.
super.configure(props);
this.application = (ServletContext)props.getValue(ViewContext.SERVLET_CONTEXT_KEY);
Modified: velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts/StrutsLinkTool.java
URL: http://svn.apache.org/viewvc/velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts/StrutsLinkTool.java?rev=707788&r1=707787&r2=707788&view=diff
==============================================================================
--- velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts/StrutsLinkTool.java (original)
+++ velocity/tools/trunk/src/main/java/org/apache/velocity/tools/struts/StrutsLinkTool.java Fri Oct 24 16:28:06 2008
@@ -62,10 +62,6 @@
@Override
protected void configure(ValueParser props)
{
- // request values override configured defaults
- //NOTE: not sure this is the most intuitive way in all cases;
- // it might make sense to provide the option of whether req/res
- // values override configured ones or vice versa.
super.configure(props);
this.application = (ServletContext)props.getValue(ViewContext.SERVLET_CONTEXT_KEY);