You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@geronimo.apache.org by Vamsavardhana Reddy <c1...@gmail.com> on 2007/01/04 21:06:07 UTC

Re: svn commit: r492105 - in /geronimo/server: branches/1.1/applications/console-standard/src/webapp/WEB-INF/view/repository/ branches/1.2/applications/console/geronimo-console-standard/src/main/webapp/WEB-INF/view/repository/ trunk/applications/cons

Joe,

You are right.  Thanks for pointing this out.  I have tested the javascript
separately before merging and testing it in console jsps.  I missed this
during the reorg.  Will fix it right away.

Vamsi

On 1/5/07, Joe Bohn <jo...@earthlink.net> wrote:
>
> Vamsi,
>
> Shouldn't the localFile.split in the if below use "/" for *nix instead
> of "\\"?   If so, then it needs to be corrected in each of the jsps that
> were changed.
>
> Joe
>
>
> vamsic007@apache.org wrote:
> > +    // Split the path
> > +    var pathParts = localFile.split("\\"); // Assuming windows file
> delim
> > +    if(localFile.indexOf("/") > 0) // May be *nix delim
> > +        pathParts = localFile.split("\\");
>