Author: nzamosenchuk
Date: 2011-08-19 04:50:19 -0400 (Fri, 19 Aug 2011)
New Revision: 4780
Modified:
jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/core/NamespaceRegistryImpl.java
Log:
EXOJCR-1390 : Applying patch along with StringBuilder replacement.
Modified:
jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/core/NamespaceRegistryImpl.java
===================================================================
---
jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/core/NamespaceRegistryImpl.java 2011-08-19
08:39:14 UTC (rev 4779)
+++
jcr/trunk/exo.jcr.component.core/src/main/java/org/exoplatform/services/jcr/impl/core/NamespaceRegistryImpl.java 2011-08-19
08:50:19 UTC (rev 4780)
@@ -332,21 +332,23 @@
final Set<String> nodes = indexSearcherHolder.getNodesByUri(uri);
if (nodes.size() > 0)
{
- StringBuffer buffer = new StringBuffer();
- buffer.append("Fail to unregister namespace");
- buffer.append(prefix);
- buffer.append(" because of following nodes: ");
+ StringBuilder builder = new StringBuilder();
+ builder.append("Fail to unregister namespace '");
+ builder.append(prefix);
+ builder.append("' because of following nodes: ");
for (String uuid : nodes)
{
ItemData item = dataManager.getItemData(uuid);
if (item != null && item.isNode())
{
- buffer.append(item.getQPath().getAsString());
+ builder.append(" - ");
+ builder.append(item.getQPath().getAsString());
+ builder.append("\r\n");
}
}
- buffer.append(" contains whese prefix ");
- throw new NamespaceException(buffer.toString());
+ builder.append(" uses this prefix.");
+ throw new NamespaceException(builder.toString());
}
}
prefixes.remove(uri);
@@ -418,7 +420,9 @@
registerNamespace(prefix, uri);
}
if (log.isDebugEnabled())
+ {
log.debug("Namespace is registered " + prefix + " =
" + uri);
+ }
}
}
catch (Exception e)
Show replies by date