Changeset 507 for Dev

Timestamp:

03/11/14 22:58:35 (11 years ago)

Author:

hendrikvanantwerpen

Message:

Check that subcodes are unique.

File:

• Dev/trunk/src/server/app.js (modified) (3 diffs)

Dev/trunk/src/server/app.js

@@ -407 +407 @@
         });
     }
+    function collectFields(field,obj) {
+        if ( _.isArray(obj) ) {
+            return _.reduce(obj,function(subcodes,val,idx){
+                return subcodes.concat(collectFields(field,val));
+            },[]);
+        } else if ( _.isObject(obj) ) {
+            return _.reduce(obj,function(subcodes,val,key){
+                if ( key === field ) {
+                    subcodes.push(val);
+                }
+                return subcodes.concat(collectFields(field,val));
+            },[]);
+        } else {
+            return [];
+        }
+    }
+    function areSubcodesUnique(content) {
+        var subcodes = collectFields('subcode',content);
+        return _.uniq(subcodes).length === subcodes.length;
+    }
     app.get('/api/questions',
         ensureAuthenticated,
@@ -435 +455 @@
                     if ( others.length > 0 ) {
                         return new HTTPResult(403,{error:"Other question with this code already exists."});
+                    } else if ( !areSubcodesUnique(doc.content) ) {
+                        return new HTTPResult(400,{error:"Subcodes are not unqiue."});
                     } else {
                         return postDocument('Question',doc);
@@ -458 +480 @@
                     if ( others.length > 0 && _.some(others,function(other){ return other._id !== id; })  ) {
                         return new HTTPResult(403,{error:"Other question with this code already exists."});
+                    } else if ( !areSubcodesUnique(doc.content) ) {
+                        return new HTTPResult(400,{error:"Subcodes are not unqiue."});
                     } else {
                         return putDocument(id,rev,'Question',doc);