[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, 1.8.22
• Component/s: jcr
• Labels: candidate_oak_1_6
• Assignee: Julian Reschke
• Reporter: Julian Reschke
• Created: 30/Oct/19 14:29
• Updated: 22/Apr/20 13:24
• 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?