Merge pull request #1 from jjsearle/master

Bug Fix : Invalid json added to payload when not specifying in_MAC or out_MAC

Author: ashsabet

move_by_attribute/process_move.py

@@ -25,12 +25,14 @@
 LOGOUT_MAC = args.out_MAC
 
 to_seat_id = 0
-if (len(table_macs_to_seats.search(Query().mac == LOGIN_MAC)) > 0):
-    to_seat_id = table_macs_to_seats.search(Query().mac == LOGIN_MAC)[0]['seat']
+if LOGIN_MAC:
+	if (len(table_macs_to_seats.search(Query().mac == LOGIN_MAC)) > 0):
+	    to_seat_id = table_macs_to_seats.search(Query().mac == LOGIN_MAC)[0]['seat']
 
 from_seat_id = 0
-if (len(table_macs_to_seats.search(Query().mac == LOGOUT_MAC)) > 0):
-    from_seat_id = table_macs_to_seats.search(Query().mac == LOGOUT_MAC)[0]['seat']
+if LOGOUT_MAC:
+	if (len(table_macs_to_seats.search(Query().mac == LOGOUT_MAC)) > 0):
+	    from_seat_id = table_macs_to_seats.search(Query().mac == LOGOUT_MAC)[0]['seat']
 
 move_body = {'employee_id': EMPLOYEE_EMAIL.split('@', 1)[0],
              'move_time': str(datetime.datetime.now().replace(microsecond=0)),
@@ -55,4 +57,4 @@
 
 response = requests.put(conf.URL_BASE + conf.URL_MOVES + '/' + str(move_id) + '/complete', headers=conf.HEADERS)
 
-print(str(response.status_code) + ': ' + response.reason)
+print(str(response.status_code) + ': ' + response.reason)