Sling
  1. Sling
  2. SLING-383

Add simple JavaScript (Rhino) Wrappers for Version and VersionHistory interfaces

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Scripting
    • Labels:
      None

      Description

      SLING-382 reported an issue handling the JCR Version and VersionHistory objects, which were wrapped as JCR Nodes. The cause is that there are no wrappers for these interfaces but as both Version and VersionHistory extend Node, the Node wrapper was used. This prevents using the Version and VersionHistory methods.

      SLING-382 implemented a workaround for this. I think the correct solution would be to add simple wrappers for the Version and VersionHistory objects which extend from the ScriptableNode wrapper and thus provide the Version and VersionHistory methods as well as the Node methods.

      1. SLING-383.patch
        19 kB
        Bertrand Delacretaz

        Issue Links

          Activity

          Gavin made changes -
          Workflow re-open possible,doc-test-required [ 12790077 ] no-reopen-closed,doc-test-required [ 12792455 ]
          Gavin made changes -
          Workflow no-reopen-closed,doc-test-required [ 12765406 ] re-open possible,doc-test-required [ 12790077 ]
          Gavin made changes -
          Link This issue depends upon SLING-397 [ SLING-397 ]
          Gavin made changes -
          Link This issue depends on SLING-397 [ SLING-397 ]
          Gavin made changes -
          Workflow Copy of no-reopen-closed,doc-test-required [ 12763390 ] no-reopen-closed,doc-test-required [ 12765406 ]
          Gavin made changes -
          Workflow no-reopen-closed,doc-test-required [ 12475673 ] Copy of no-reopen-closed,doc-test-required [ 12763390 ]
          Felix Meschberger made changes -
          Workflow jira [ 12429089 ] no-reopen-closed,doc-test-required [ 12475673 ]
          Carsten Ziegeler made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Bertrand Delacretaz made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          Bertrand Delacretaz added a comment -

          Committed in revision 652323

          Show
          Bertrand Delacretaz added a comment - Committed in revision 652323
          Bertrand Delacretaz made changes -
          Attachment SLING-383.patch [ 12381169 ]
          Hide
          Bertrand Delacretaz added a comment -

          I'm getting connection reset by peer (https://svn.apache.org) errors ATM, will commit this patch once things are back to normal.

          Show
          Bertrand Delacretaz added a comment - I'm getting connection reset by peer ( https://svn.apache.org ) errors ATM, will commit this patch once things are back to normal.
          Bertrand Delacretaz made changes -
          Assignee Bertrand Delacretaz [ bdelacretaz ]
          Bertrand Delacretaz made changes -
          Link This issue depends on SLING-397 [ SLING-397 ]
          Bertrand Delacretaz made changes -
          Link This issue is related to SLING-154 [ SLING-154 ]
          Hide
          Felix Meschberger added a comment -

          Link to SLING-382

          Show
          Felix Meschberger added a comment - Link to SLING-382
          Felix Meschberger made changes -
          Field Original Value New Value
          Link This issue is related to SLING-382 [ SLING-382 ]
          Felix Meschberger created issue -

            People

            • Assignee:
              Bertrand Delacretaz
              Reporter:
              Felix Meschberger
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development