geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jacek Laskowski" <ja...@laskowski.net.pl>
Subject Re: svn commit: r631758 - in /geronimo/server/trunk/plugins/console: console-portal-driver/src/main/webapp/WEB-INF/web.xml plugin-portlets/src/main/webapp/WEB-INF/view/ajax/progressbar.jsp plugin-portlets/src/main/webapp/WEB-INF/web.xml
Date Thu, 28 Feb 2008 07:29:44 GMT
On Wed, Feb 27, 2008 at 2:30 PM,  <gawor@apache.org> wrote:
> Author: gawor
>  Date: Wed Feb 27 14:29:58 2008
>  New Revision: 631758
>
>  URL: http://svn.apache.org/viewvc?rev=631758&view=rev
>  Log:
>  forward all dwr requests through the /console context (GERONIMO-3781)
>
>  Modified:
>     geronimo/server/trunk/plugins/console/console-portal-driver/src/main/webapp/WEB-INF/web.xml
>     geronimo/server/trunk/plugins/console/plugin-portlets/src/main/webapp/WEB-INF/view/ajax/progressbar.jsp
>     geronimo/server/trunk/plugins/console/plugin-portlets/src/main/webapp/WEB-INF/web.xml
>
....
>  Modified: geronimo/server/trunk/plugins/console/plugin-portlets/src/main/webapp/WEB-INF/view/ajax/progressbar.jsp
>  URL: http://svn.apache.org/viewvc/geronimo/server/trunk/plugins/console/plugin-portlets/src/main/webapp/WEB-INF/view/ajax/progressbar.jsp?rev=631758&r1=631757&r2=631758&view=diff
>  ==============================================================================
>  --- geronimo/server/trunk/plugins/console/plugin-portlets/src/main/webapp/WEB-INF/view/ajax/progressbar.jsp
(original)
>  +++ geronimo/server/trunk/plugins/console/plugin-portlets/src/main/webapp/WEB-INF/view/ajax/progressbar.jsp
Wed Feb 27 14:29:58 2008
>  @@ -20,9 +20,9 @@
>   <fmt:setBundle basename="pluginportlets"/>
>   <fmt:message key="car.downloadStatus.processing"/>
>   <portlet:defineObjects/>
>  -<script type='text/javascript' src='/plugin/dwr/interface/ProgressMonitor.js'></script>
>  -<script type='text/javascript' src='/plugin/dwr/engine.js'></script>
>  -<script type='text/javascript' src='/plugin/dwr/util.js'></script>
>  +<script type='text/javascript' src='/console/dwr4/interface/ProgressMonitor.js'></script>
>  +<script type='text/javascript' src='/console/dwr4/engine.js'></script>
>  +<script type='text/javascript' src='/console/dwr4/util.js'></script>

Although it's assumed that the console is always bound to /console
context path, wouldn't <%= request.getContextPath() %> or
${request.contextPath} make it less error-prone rather than adding
"/console" directly?

Jacek

-- 
Jacek Laskowski
http://www.JacekLaskowski.pl

Mime
View raw message