[OAK-8722] dead locking related code in NodeDelegate.updateMixins

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.20.0, 1.10.7
• Component/s: jcr
• Labels: candidate_oak_1_8
• Assignee: Julian Reschke
• Reporter: Julian Reschke
• Created: 30/Oct/19 14:29
• Updated: 25/Nov/19 06:28
• Resolved: 04/Nov/19 13:46

Description

            // 3. deal with locked nodes
            boolean wasLockable = isNodeType(MIX_LOCKABLE);
            boolean isLockable = isNodeType(MIX_LOCKABLE);
            if (wasLockable && !isLockable && holdsLock(false)) {
                // TODO: This should probably be done in a commit hook
                unlock();
                sessionDelegate.refresh(true);
            }

AFAICT, this code block will never be executed. Clean up, Angela Schreiber?