added workaroung for angularjs issue 1463 angular/angular.js#1463 ($oid can also be passed as €oid

removed useless warning message if ref field is null

Author: ujibang

src/main/java/org/restheart/hal/metadata/Relationship.java

@@ -258,14 +258,11 @@ private Object getReferenceFieldValue(String referenceField, DBObject data) thro
 
             List<Object> objs = JsonUtils.getPropsFromPath(data, referenceField);
 
-            if (objs != null && objs.size() == 1 && objs.get(0) == null)
+            if (objs == null) {
+                return null;
+            } else if (objs.size() == 1 && objs.get(0) == null)
                 return null;
 
-            if (objs == null) {
-                LOGGER.debug("cound not get the value of the reference field " + referenceField + " from " + data.toString() + "\nThe json path expression resolved to null");
-                throw new IllegalArgumentException("ref-field json path expression resolved to null");
-            }
-
             // check that objs are all 
             if (objs.stream().allMatch(obj -> {
                 if (obj == null || obj instanceof DBObject) {

src/main/java/org/restheart/handlers/injectors/BodyInjectorHandler.java

@@ -100,7 +100,7 @@ public void handleRequest(HttpServerExchange exchange, RequestContext context) t
         DBObject content;
 
         if (isNotFormData(contentTypes)) { // json or hal+json
-            final String contentString = ChannelReader.read(exchange.getRequestChannel());
+            final String contentString = workaroundAngularJSIssue1463(ChannelReader.read(exchange.getRequestChannel()));
 
             try {
                 content = (DBObject) JSON.parse(contentString);
@@ -142,11 +142,11 @@ public void handleRequest(HttpServerExchange exchange, RequestContext context) t
                 ResponseHelper.endExchangeWithMessage(exchange, HttpStatus.SC_NOT_ACCEPTABLE, errMsg);
                 return;
             }
-            
+
             File file = data.getFirst(fileFieldName).getFile();
 
             context.setFile(file);
-            
+
             injectContentTypeFromFile(content, file);
         }
 
@@ -171,6 +171,20 @@ public void handleRequest(HttpServerExchange exchange, RequestContext context) t
         getNext().handleRequest(exchange, context);
     }
 
+    /**
+     * need this to workaroung angularjs issue
+     * https://github.com/angular/angular.js/issues/1463
+     *
+     * @param content
+     * @return
+     */
+    private static String workaroundAngularJSIssue1463(String contentString) {
+        if (contentString == null)
+            return null;
+        
+        return contentString.replaceAll("\"€oid\" *:", "\"\\$oid\" :");
+    }
+
     /**
      *
      * @param contentTypes
@@ -215,22 +229,24 @@ private void filterOutReservedKeys(DBObject content, RequestContext context) {
             context.addWarning("the reserved field " + keyToRemove + " was filtered out from the request");
         });
     }
-    
+
     private void injectContentTypeFromFile(DBObject content, File file) throws IOException {
-        if (content.get("contentType") != null)
+        if (content.get("contentType") != null) {
             return;
-        
+        }
+
         String contentType;
-        
-        if (file == null)
+
+        if (file == null) {
             return;
-        else
+        } else {
             contentType = detectMediaType(file);
-        
+        }
+
         if (content == null && contentType != null) {
             content = new BasicDBObject();
-        } 
-        
+        }
+
         content.put("contentType", contentType);
     }