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

        Hide
        Jukka Zitting added a comment -

        Merged to the 1.3 branch in revision 544279.

        Show
        Jukka Zitting added a comment - Merged to the 1.3 branch in revision 544279.
        Hide
        Jukka Zitting added a comment -

        Fixed in revision 537122.

        Show
        Jukka Zitting added a comment - Fixed in revision 537122.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development