Bug 114739 - PatternStylesheetResolver fails on Windows
PatternStylesheetResolver fails on Windows
Status: CLOSED RAWHIDE
Product: Red Hat Web Application Framework
Classification: Retired
Component: other (Show other bugs)
nightly
All Linux
medium Severity medium
: ---
: ---
Assigned To: Vadim Nasardinov
:
Depends On:
Blocks: 113496
  Show dependency treegraph
 
Reported: 2004-02-02 04:00 EST by Daniel Berrange
Modified: 2007-04-18 13:02 EDT (History)
0 users

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2004-03-17 18:00:09 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description Daniel Berrange 2004-02-02 04:00:26 EST
Description of problem:
See
https://www.redhat.com/archives/redhat-ccm-list/2004-February/msg00000.html

<quote>
   I have tried to install WAF 6.1 in Windows. However I run into
problems serving the stylesheet. I found out later, the problem is using
java.net.URL(). I have changed the file com.arsdigita.templating.
PatternStylesheetResolver to accommodate for any platform (not just
Windows)

I have used
String realFile = new File(pathname).toURL().toExternalForm();
return new java.net.URL(realFile);

instead of
return new java.net.URL("file://" + pathname);
</quote>

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:
Comment 1 Vadim Nasardinov 2004-02-02 17:33:57 EST
Looks like we may want to also apply this fix to the following files:

$ find /var/vadim/p4checkout/core-platform/dev/src/ -name \*.java \ |
  xargs grep '"file://' | sed 's/java:/java:\n/'

/var/vadim/p4checkout/core-platform/dev/src/com/arsdigita/packaging/ConfigRegistry.java:
            return new URL(new URL("file://" + home), "conf/");
/var/vadim/p4checkout/core-platform/dev/src/com/arsdigita/runtime/RuntimeClassLoader.java:
            return super.unmarshal("file://" + literal + "/conf/", errors);
/var/vadim/p4checkout/core-platform/dev/src/com/arsdigita/sitenode/BasePresentationManager.java:
                    java.net.URL sheet = new java.net.URL("file://" + sctx.getRealPath(path));
/var/vadim/p4checkout/core-platform/dev/src/com/arsdigita/templating/Templating.java:
                        URL newURL = new URL("file://" + filename);
Comment 2 Vadim Nasardinov 2004-02-02 17:36:53 EST
Looks like bug 112560 may need to be re-fixed along the same lines.
Comment 3 Vadim Nasardinov 2004-02-03 15:21:44 EST
Fixed on the trunk in change 40018.
Comment 4 Jon Orris 2004-02-11 09:47:00 EST
Dan, do you have any verification that this is working properly on the
Windows end? If so, I'm going to go ahead & close this.



Note You need to log in before you can comment on or make changes to this bug.