Petals SE XSLT

Unable to load the XSL using the classloader

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.3.2, 2.4
  • Fix Version/s: 2.4
  • Component/s: None
  • Security Level: Public
  • Description:

    The petals-se-xslt 2.0 was able to load the XSL using a custom classloader. This feature has been removed.
    Please, restore it.

  • Environment:
    -

Activity

Christophe DENEUX made changes - Mon, 16 Aug 2010 - 20:06:03 +0200
Field Original Value New Value
Description The petals-se-xslt was able to load the XSL using a custom classloader. This feature has been removed.
Please, restore it.
The petals-se-xslt 2.0 was able to load the XSL using a custom classloader. This feature has been removed.
Please, restore it.
Priority Blocker [ 1 ]
Hide
Vincent Zurczak added a comment - Mon, 16 Aug 2010 - 20:43:34 +0200

I've checked the tag for the version 2.0 of the component.
I don't see anything new with respect to the trunk version.
Currently, if the SU embeds JAR files which contain class files, the XSL style sheet and the JAR files are loaded in a custom class loader.

Would you have a SU for which this mechanism does not work?

Show
Vincent Zurczak added a comment - Mon, 16 Aug 2010 - 20:43:34 +0200 I've checked the tag for the version 2.0 of the component. I don't see anything new with respect to the trunk version. Currently, if the SU embeds JAR files which contain class files, the XSL style sheet and the JAR files are loaded in a custom class loader. Would you have a SU for which this mechanism does not work?
Hide
Christophe DENEUX added a comment - Tue, 17 Aug 2010 - 08:57:00 +0200

According to the source code of the trunk, the XSL is loaded only from File (see line 129,130 of the rev 15471 of XsltConfigurationHandler). The custom classloader is created, but used only to loade specific functions.
The XSL is never load from classloader as it was at line 165-171 of JBIListener of the SE XSLT 2.0

Show
Christophe DENEUX added a comment - Tue, 17 Aug 2010 - 08:57:00 +0200 According to the source code of the trunk, the XSL is loaded only from File (see line 129,130 of the rev 15471 of XsltConfigurationHandler). The custom classloader is created, but used only to loade specific functions. The XSL is never load from classloader as it was at line 165-171 of JBIListener of the SE XSLT 2.0
Hide
Vincent Zurczak added a comment - Tue, 17 Aug 2010 - 10:51:51 +0200

OK, I see.
XSL style sheets that are contained in a JAR file can not be loaded in the current version.

Show
Vincent Zurczak added a comment - Tue, 17 Aug 2010 - 10:51:51 +0200 OK, I see. XSL style sheets that are contained in a JAR file can not be loaded in the current version.
Vincent Zurczak made changes - Tue, 17 Aug 2010 - 10:51:58 +0200
Status New [ 10000 ] Open [ 10002 ]
Priority Blocker [ 1 ] Major [ 3 ]
Vincent Zurczak made changes - Tue, 17 Aug 2010 - 10:52:00 +0200
Status Open [ 10002 ] In Progress [ 10003 ]
Hide
Vincent Zurczak added a comment - Tue, 17 Aug 2010 - 11:30:02 +0200

Commit # 15553
I tested with two (almost) identical SU: in the first one, the XSL was only available as a file, while in the second, it was in a JAR file.

Show
Vincent Zurczak added a comment - Tue, 17 Aug 2010 - 11:30:02 +0200 Commit # 15553 I tested with two (almost) identical SU: in the first one, the XSL was only available as a file, while in the second, it was in a JAR file.
Vincent Zurczak made changes - Tue, 17 Aug 2010 - 11:30:02 +0200
Status In Progress [ 10003 ] Resolved [ 10004 ]
Resolution Fixed [ 1 ]
Vincent Zurczak made changes - Tue, 17 Aug 2010 - 11:30:09 +0200
Status Resolved [ 10004 ] Closed [ 10005 ]
Fix Version/s 2.4 [ 10087 ]
Transition Status Change Time Execution Times Last Executer Last Execution Date
New New Open Open
14h 46m
1
Vincent Zurczak
Tue, 17 Aug 2010 - 10:51:58 +0200
Open Open In Progress In Progress
2s
1
Vincent Zurczak
Tue, 17 Aug 2010 - 10:52:00 +0200
In Progress In Progress Resolved Resolved
38m 2s
1
Vincent Zurczak
Tue, 17 Aug 2010 - 11:30:02 +0200
Resolved Resolved Closed Closed
7s
1
Vincent Zurczak
Tue, 17 Aug 2010 - 11:30:09 +0200



People

Dates

  • Created:
    Mon, 16 Aug 2010 - 20:05:10 +0200
    Updated:
    Tue, 17 Aug 2010 - 11:30:09 +0200
    Resolved:
    Tue, 17 Aug 2010 - 11:30:02 +0200