[SLING-382] VersionHistoryImpl is incorrectly wrapped as a Node in server-side javascript

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Component/s: Scripting
• Assignee: Unassigned
• Reporter: Bertrand Delacretaz
• Created: 15/Apr/08 12:00
• Updated: 21/Jul/09 10:22
• Resolved: 15/Apr/08 13:02

Description

The toString() code in this ESP code fails when applied to a versionable Node:

var history = currentNode.getVersionHistory();
out.println(history);
out.write("<br/>");
out.println(history.toString());
out.write("<br/>");

IIUC the cause is that in SlingWrapFactory.wrapAsJavaObject, the jackrabbit VersionHistoryImpl is incorrectly wrapped with a ScriptableNode, as VersionHistoryImpl extends from NodeImpl and there's no specific wrapper for VersionHistoryImpl.

Issue Links

• relates to: SLING-383 Add simple JavaScript (Rhino) Wrappers for Version and VersionHistory interfaces