You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jackrabbit.apache.org by "Dave Brosius (JIRA)" <ji...@apache.org> on 2009/02/05 05:19:59 UTC

[jira] Updated: (JCR-1966) [PATCH] Remove Stutter in NodeState

     [ https://issues.apache.org/jira/browse/JCR-1966?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Dave Brosius updated JCR-1966:
------------------------------

    Attachment: remove_stutter.diff

> [PATCH] Remove Stutter in NodeState
> -----------------------------------
>
>                 Key: JCR-1966
>                 URL: https://issues.apache.org/jira/browse/JCR-1966
>             Project: Jackrabbit Content Repository
>          Issue Type: Improvement
>          Components: jackrabbit-jcr-rmi
>    Affects Versions: 1.6.0
>            Reporter: Dave Brosius
>             Fix For: 1.6.0
>
>         Attachments: remove_stutter.diff
>
>
> Code duplicates code for no reason
> Index: src/main/java/org/apache/jackrabbit/jcr2spi/state/NodeState.java
> ===================================================================
> --- src/main/java/org/apache/jackrabbit/jcr2spi/state/NodeState.java	(revision 740824)
> +++ src/main/java/org/apache/jackrabbit/jcr2spi/state/NodeState.java	(working copy)
> @@ -449,7 +449,7 @@
>               */
>              NodeState parent = getParent();
>              NodeId wspId = (NodeId) getWorkspaceId();
> -            def = definitionProvider.getQNodeDefinition(getParent().getNodeTypeNames(), getName(), getNodeTypeName(), wspId);
> +            def = definitionProvider.getQNodeDefinition(parent.getNodeTypeNames(), getName(), getNodeTypeName(), wspId);
>          }
>          return def;
>      }

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.