Merge pull request #16 from viswa-swami/check_for_error_before_processing

Check if get was successful before trying to set some config.

Author: sinchb

foscam/foscam.py

@@ -613,20 +613,25 @@ def set_motion_detection(self, enabled=1):
         Get the current config and set the motion detection on or off
         '''
         result, current_config = self.get_motion_detect_config()
+        if result != FOSCAM_SUCCESS:
+            return result
         current_config['isEnable'] = enabled
         self.set_motion_detect_config(current_config)
+        return FOSCAM_SUCCESS
 
     def enable_motion_detection(self):
         '''
         Enable motion detection
         '''
-        self.set_motion_detection(1)
+        result = self.set_motion_detection(1)
+        return result
 
     def disable_motion_detection(self):
         '''
         disable motion detection
         '''
-        self.set_motion_detection(0)
+        result = self.set_motion_detection(0)
+        return result
 
     def get_alarm_record_config(self, callback=None):
         '''