Handle spaces in path names

- IT demonstrating failure when creating an object with a space in its path and then trying to perform a SPARQL update on it

Resolves: https://www.pivotaltracker.com/story/show/72296804

fcrepo-http-api/src/test/java/org/fcrepo/integration/http/api/FedoraNodesIT.java

@@ -1689,4 +1689,19 @@ public void testLastModifiedUpdatedAfterUpdates() throws Exception {
         assertFalse("Last-Modified headers should have changed", lastmod1 == lastmod2);
     }
 
+    @Test
+    public void testUpdateObjectWithSpaces() throws Exception {
+        final String id = getRandomUniquePid() + " 2";
+        final HttpResponse createResponse = createObject(id);
+        final String subjectURI = createResponse.getFirstHeader("Location").getValue();
+        final HttpPatch updateObjectGraphMethod = new HttpPatch(subjectURI);
+        updateObjectGraphMethod.addHeader("Content-Type", "application/sparql-update");
+        final BasicHttpEntity e = new BasicHttpEntity();
+        e.setContent(new ByteArrayInputStream(
+            "INSERT { <> <http://purl.org/dc/elements/1.1/title> \"test\" } WHERE {}".getBytes()));
+        updateObjectGraphMethod.setEntity(e);
+        final HttpResponse response = client.execute(updateObjectGraphMethod);
+        assertEquals(NO_CONTENT.getStatusCode(), response.getStatusLine().getStatusCode());
+    }
+
 }

fcrepo-http-commons/src/main/java/org/fcrepo/http/commons/api/rdf/HttpIdentifierTranslator.java

@@ -152,8 +152,13 @@ public boolean isCanonical() {
     @Override
     public Resource getSubject(final String absPath) throws RepositoryException {
         resetTranslationChain();
-        LOGGER.debug("Creating RDF subject from identifier: {}", absPath);
-        return doForward(absPath);
+        if ( absPath != null && absPath.indexOf("%20") != -1 ) {
+            LOGGER.debug("Creating RDF subject from identifier with spaces: {}", absPath);
+            return doForward(absPath.replaceAll("%20"," "));
+        } else {
+            LOGGER.debug("Creating RDF subject from identifier: {}", absPath);
+            return doForward(absPath);
+        }
     }
 
     @Override

fcrepo-kernel-impl/src/main/java/org/fcrepo/kernel/impl/utils/JcrPropertyStatementListener.java

@@ -123,7 +123,12 @@ public void addedStatement(final Statement s) {
                     skolemizedBnodeMap.put(subject.getId(), subjectNode);
                 }
             } else {
-                subjectNode = session.getNode(subjects.getPathFromSubject(subject));
+                final String path = subjects.getPathFromSubject(subject);
+                if ( path != null && path.indexOf("%20") != -1 ) {
+                    subjectNode = session.getNode(path.replaceAll("%20"," "));
+                } else {
+                    subjectNode = session.getNode(path);
+                }
             }
 
             // special logic for handling rdf:type updates.