You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@jackrabbit.apache.org by "Julian Reschke (Jira)" <ji...@apache.org> on 2023/02/10 18:01:00 UTC

[jira] [Created] (JCRVLT-684) Mixins recovered using stashing set on incorrect node

Julian Reschke created JCRVLT-684:
-------------------------------------

             Summary: Mixins recovered using stashing set on incorrect node
                 Key: JCRVLT-684
                 URL: https://issues.apache.org/jira/browse/JCRVLT-684
             Project: Jackrabbit FileVault
          Issue Type: Bug
          Components: vlt
    Affects Versions: 3.5.4
            Reporter: Julian Reschke


This:

https://github.com/apache/jackrabbit-filevault/blob/master/vault-core/src/main/java/org/apache/jackrabbit/vault/fs/impl/io/NodeStash.java#L215

{noformat}
        // restore mixins
        Property mixinProperty = tmpNode.hasProperty(JcrConstants.JCR_MIXINTYPES + PROTECTED_PROPERTIES_SUFFIX) ? tmpNode.getProperty(JcrConstants.JCR_MIXINTYPES + PROTECTED_PROPERTIES_SUFFIX) : null;
        if (mixinProperty != null) {
            for (Value value : mixinProperty.getValues()) {
                tmpNode.addMixin(value.getString());
            }
        }
{noformat}

looks fishy to me. Shouldn't it be:

{noformat}
        // restore mixins
        Property mixinProperty = tmpNode.hasProperty(JcrConstants.JCR_MIXINTYPES + PROTECTED_PROPERTIES_SUFFIX) ? tmpNode.getProperty(JcrConstants.JCR_MIXINTYPES + PROTECTED_PROPERTIES_SUFFIX) : null;
        if (mixinProperty != null) {
            for (Value value : mixinProperty.getValues()) {
                destNode.addMixin(value.getString());
            }
        }
{noformat}





--
This message was sent by Atlassian Jira
(v8.20.10#820010)