On Wed, Feb 27, 2008 at 2:30 PM,  <[EMAIL PROTECTED]> 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

Reply via email to