[JCR-2186] Unnecessary hasItemState() call in SessionItemStateManager

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.6
• Component/s: jackrabbit-core
• Assignee: Unassigned
• Reporter: Marcel Reutegger
• Created: 03/Jul/09 18:19
• Updated: 13/Aug/09 15:03
• Resolved: 05/Jul/09 14:37

Description

At the end of SessionItemStateManager.getItemState(ItemId) the underlying item state manager is first asked whether it contains the item and then it is retrieved. In case the item state manager does not know the item a NoSuchItemStateException is thrown.

The initial check is unnecessary because getItemState() on the underlying manager will also throw the exception if the item does not exist.