Jackrabbit Content Repository
  1. Jackrabbit Content Repository
  2. JCR-911

Remove sanityCheck() from ItemImpl.getSession()

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.9, 1.0, 1.0.1, 1.1, 1.1.1, 1.2.1, 1.2.2, 1.2.3, 1.3
    • Fix Version/s: 1.3.1
    • Component/s: None
    • Labels:
      None

      Description

      The following code causes an InvalidItemStateException to be thrown for no good reason:

      Property property = ...;
      property.setValue((Value) null);
      property.getSession();

      There are cases (I'm looking at one right now) where it's good to be able to access the session of an Item even if it has already been invalidated.

      The simple fix is to remove the sanityCheck() call from ItemImpl.getSession(). I'll do that unless someone has a good reason why the sanity check should be kept.

        Activity

        Jukka Zitting created issue -
        Jukka Zitting made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Fix Version/s 1.4 [ 12312447 ]
        Jukka Zitting made changes -
        Fix Version/s 1.4 [ 12312447 ]
        Fix Version/s 1.3.1 [ 12312512 ]
        Jukka Zitting made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Jukka Zitting made changes -
        Workflow jira [ 12403625 ] no-reopen-closed, patch-avail [ 12468929 ]

          People

          • Assignee:
            Jukka Zitting
            Reporter:
            Jukka Zitting
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development