make sure creating objects with fcr:new creates the object at the rig…

…ht path level.
fcrepo · May 8, 2013 · 87ab40a · 87ab40a
1 parent d14cb8a
commit 87ab40a
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 4 deletions.
diff --git a/fcrepo-http-api/src/main/java/org/fcrepo/api/FedoraUnnamedObjects.java b/fcrepo-http-api/src/main/java/org/fcrepo/api/FedoraUnnamedObjects.java
@@ -58,9 +58,11 @@ public MultivaluedMap<String, String> getMatrixParameters() {
             }
 
         };
-        ImmutableList.Builder<PathSegment> segments = ImmutableList.builder();
-        segments.addAll(pathList.subList(0, pathList.size() - 1));
+
+		ImmutableList.Builder<PathSegment> segments = ImmutableList.builder();
+        segments.addAll(pathList);
         segments.add(path);
+
         try {
             return objectsResource.createObject(
                     segments.build(), "test label",

diff --git a/fcrepo-http-api/src/test/java/org/fcrepo/integration/api/FedoraNodesIT.java b/fcrepo-http-api/src/test/java/org/fcrepo/integration/api/FedoraNodesIT.java
@@ -41,6 +41,9 @@ public void testIngestWithNew() throws Exception {
         assertTrue("Response wasn't a PID", compile("[a-z]+").matcher(content)
                 .find());
         assertTrue("new object did not mint a PID", !content.endsWith("/fcr:new"));
+
+		assertEquals("Object wasn't created!", 200,
+							getStatus(new HttpGet(serverAddress + content)));
     }
 
     @Test

diff --git a/fcrepo-http-commons/src/main/java/org/fcrepo/AbstractResource.java b/fcrepo-http-commons/src/main/java/org/fcrepo/AbstractResource.java
@@ -149,8 +149,12 @@ public void setHttpServletRequest(final HttpServletRequest servletRequest) {
     public static final String toPath(List<PathSegment> paths) {
         StringBuffer result = new StringBuffer();
         for (PathSegment path: paths) {
-            result.append('/');
-            result.append(path.getPath());
+			final String p = path.getPath();
+
+			if(!p.equals("")) {
+            	result.append('/');
+            	result.append(p);
+			}
         }
         return result.toString();
     }