You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@roller.apache.org by Glen Mazza <gl...@gmail.com> on 2013/07/19 23:16:03 UTC
Simplify web.xml & delete velocity/deprecated folder?
Hi Team, in our web.xml[1] we still provide a RedirectServlet[2] to map
pre-3.0 Roller command URLs and a StrutsRedirectServlet[3] for mapping
Struts1 command URLs to their modern Roller equivalents -- I think we
can get rid of these two Servlets by now, any objections if I do so?
Also, the velocity/deprecated folder [4] was deprecated back in 2006, OK
if we delete it now from Roller trunk? I'd like to keep modern Roller
as streamlined as we can, give Sonar less code it can complain about. :)
Regards,
Glen
[1]
http://svn.apache.org/viewvc/roller/trunk/app/src/main/webapp/WEB-INF/web.xml?view=markup#l456
[2]
http://svn.apache.org/viewvc/roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/rendering/velocity/deprecated/RedirectServlet.java?revision=1504845&view=co
[3]
http://svn.apache.org/viewvc/roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/struts2/util/StrutsRedirectServlet.java?view=co
[4]
http://svn.apache.org/viewvc/roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/rendering/velocity/deprecated/
Re: Simplify web.xml & delete velocity/deprecated folder?
Posted by Dave <sn...@gmail.com>.
+1
- Dave
On Fri, Jul 19, 2013 at 5:16 PM, Glen Mazza <gl...@gmail.com> wrote:
> Hi Team, in our web.xml[1] we still provide a RedirectServlet[2] to map
> pre-3.0 Roller command URLs and a StrutsRedirectServlet[3] for mapping
> Struts1 command URLs to their modern Roller equivalents -- I think we can
> get rid of these two Servlets by now, any objections if I do so?
>
> Also, the velocity/deprecated folder [4] was deprecated back in 2006, OK
> if we delete it now from Roller trunk? I'd like to keep modern Roller as
> streamlined as we can, give Sonar less code it can complain about. :)
>
> Regards,
> Glen
>
> [1] http://svn.apache.org/viewvc/**roller/trunk/app/src/main/**
> webapp/WEB-INF/web.xml?view=**markup#l456<http://svn.apache.org/viewvc/roller/trunk/app/src/main/webapp/WEB-INF/web.xml?view=markup#l456>
> [2] http://svn.apache.org/viewvc/**roller/trunk/app/src/main/**
> java/org/apache/roller/**weblogger/ui/rendering/**velocity/deprecated/**
> RedirectServlet.java?revision=**1504845&view=co<http://svn.apache.org/viewvc/roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/rendering/velocity/deprecated/RedirectServlet.java?revision=1504845&view=co>
> [3] http://svn.apache.org/viewvc/**roller/trunk/app/src/main/**
> java/org/apache/roller/**weblogger/ui/struts2/util/**
> StrutsRedirectServlet.java?**view=co<http://svn.apache.org/viewvc/roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/struts2/util/StrutsRedirectServlet.java?view=co>
> [4] http://svn.apache.org/viewvc/**roller/trunk/app/src/main/**
> java/org/apache/roller/**weblogger/ui/rendering/**velocity/deprecated/<http://svn.apache.org/viewvc/roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/rendering/velocity/deprecated/>
>