actually it was the other change that revealed a bug in
NodeTypeImpl#isNodeType which was comparing the jcr-name
exposed in getName() with the string passed to the call.
the latter can of course be an expanded name too such as
the one i used with the constant :-)

On 6/6/12 1:31 PM, Julian Reschke wrote:
> On 2012-06-06 10:43, angela@apache.org wrote:
>> ...
>> @@ -671,7 +668,7 @@ public class NodeImpl extends ItemImpl i
>>            String primaryNtName;
>>            primaryNtName = hasProperty(Property.JCR_PRIMARY_TYPE)
>>                ? getProperty(Property.JCR_PRIMARY_TYPE).getString()
>> -            : "nt:unstructured";
>> +            : NodeType.NT_UNSTRUCTURED;
>>
>>            return ntMgr.getNodeType(primaryNtName);
>>        }
>> ...
>
> I believe this one broke the JCR integration tests.
>
> Best regards, Julian