Fix GladysAssistant#249 : If user has only one house and say "I'm bac…

…k home", Gladys should suppose it's his only house
R6n0 · Nov 14, 2017 · 58ecc27 · 58ecc27
1 parent 633dd25
commit 58ecc27
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 2 deletions.
diff --git a/api/core/brain/parser/parser.house.js b/api/core/brain/parser/parser.house.js
@@ -23,7 +23,8 @@ module.exports = function(originalText){
 
         return {
             text, 
-            houses: result
+            houses: result,
+            allHouses: houses
         };
     });
 };

diff --git a/api/core/brain/parser/parser.js b/api/core/brain/parser/parser.js
@@ -34,8 +34,9 @@ module.exports.parse = function parse(text) {
         .then((result) => {
 
             houses = result.houses;
+            allHouses = result.allHouses;
             replacedText = result.text;
 
-            return Promise.resolve({deviceTypes, rooms, houses, times, replacedText});
+            return Promise.resolve({deviceTypes, rooms, houses, times, replacedText, allHouses});
         });
 };
diff --git a/api/core/event/event.command.js b/api/core/event/event.command.js
@@ -15,6 +15,11 @@ module.exports = function command(scope) {
     if(scope.rooms && scope.rooms.length == 1){
         event.room = scope.rooms[0].id;
     }
+
+    // if there is only one house and you say "I'm back at home" "I left home", Gladys suppose it's your only house
+    if((scope.label === 'back-at-home' || scope.label === 'left-home') && !event.house && scope.allHouses.length === 1) {
+        event.house = scope.allHouses[0];
+    }
 
     return gladys.event.create(event)
         .then(() => {