preliminar validation done

backend/app/people/inquisitor-validator.js

@@ -6,7 +6,7 @@ var areBadPetsHere = function (pets) {
     return false
   }
   var m = pets.map(function (pet) {
-    return pet == ''
+    return pet == '' || pet == undefined
   })
   return m.reduce(function (i,j) {
     return i || j
@@ -14,6 +14,7 @@ var areBadPetsHere = function (pets) {
 }
 
 var inquisitor = function (next) {
+  console.log(this)
   if(this.name == ''){
     next(erroneous_name)
   }

backend/app/people/people.js

@@ -54,10 +54,10 @@ db.once('open',function () {
   }
 
   people.modifyPet = function (id, old_pet_name, new_pet_name, callback) {
-    people.removePet(id,old_pet_name,function () {
-      people.addPet(id,new_pet_name,function () {
-        callback()
-      })
+    people.model.findById(id).then(function (person) {
+      person.pets.splice(person.pets.indexOf(old_pet_name),1)
+      person.pets.push(new_pet_name)
+      person.save(callback)
     })
   }
 

frontend/app/core/people.js

@@ -8,6 +8,12 @@ app.controller('peopleCtrl',function ($scope,ngDialog,people) {
     })
   }
 
+  $scope.badData = function () {
+    ngDialog.open({
+      template:'../error-partial.html'
+    })
+  }
+
   this.load()
 
   $scope.remove = function (item) {
@@ -16,7 +22,7 @@ app.controller('peopleCtrl',function ($scope,ngDialog,people) {
 
   $scope.add = function () {
     console.log("adding")
-    people.add($scope.name,self.load)
+    people.add($scope.name,self.load,$scope.badData)
     $scope.name = ''
   }
 
@@ -34,7 +40,7 @@ app.controller('peopleCtrl',function ($scope,ngDialog,people) {
     people.addPet(person, pet, function () {
       self.load()
       $scope.pet = ''
-    })
+    }, $scope.badData)
   }
 
   $scope.do = function (person,pet) {

frontend/app/core/peopleService.js

@@ -7,18 +7,18 @@ app.service('people', function ($http) {
     $http.delete('http://127.0.0.1:8000/people/'+item._id).then(callback)
   }
 
-  this.add = function (n,callback) {
-    $http.post('http://127.0.0.1:8000/people',{name:n}).then(callback)
+  this.add = function (n,callback,reject) {
+    $http.post('http://127.0.0.1:8000/people',{name:n}).then(callback).catch(reject)
   }
 
-  this.modify = function (item,callback) {
+  this.modify = function (item,callback,reject) {
     $http.put('http://127.0.0.1:8000/people',{person_id: item._id, name: item.name})
-    .then(callback)
+    .then(callback).catch(reject)
   }
 
-  this.addPet = function (item,pet_name,callback) {
+  this.addPet = function (item,pet_name,callback,reject) {
     $http.post('http://127.0.0.1:8000/people/pets',{person_id: item._id, petname: pet_name})
-    .then(callback)
+    .then(callback).catch(reject)
   }
 
   this.removePet = function (item, pet_name, callback) {
@@ -26,12 +26,12 @@ app.service('people', function ($http) {
     .then(callback)
   }
 
-  this.modifyPet = function (item,old_pet_name,new_petname,callback) {
+  this.modifyPet = function (item,old_pet_name,new_petname,callback,reject) {
     $http.put('http://127.0.0.1:8000/people/pets',
     {
       person_id: item._id,
       old_petname: old_pet_name,
       new_petname: new_petname
-    }).then(callback)
+    }).then(callback).catch(reject)
   }
 })

frontend/app/error-partial.html

+<div>
+  <h1>Error</h1>
+    Erroneous data.
+  <div class="row">
+    <div class="6u align-center">
+      <button ng-click='closeThisDialog()' >Oops</button>
+    </div>
+  </div>
+</div>