fix bugs

Author: lukasheinrich

recastapi/response/read.py

@@ -82,7 +82,7 @@ def basic_response(basic_response_id =  None, basic_request_id_filter = None, de
 
 def response_archive(basic_response_id = None, basic_request_id = None, filename = None):
     if basic_response_id is None:
-        basic_response_id = recastapi.response.read.basic_response(basic_request_id = basic_request_id )['id']
+        basic_response_id = recastapi.response.read.basic_response(basic_request_id_filter = basic_request_id )['id']
 
     selection_str = '?where=basic_response_id=="{}"'.format(basic_response_id)
     url = '{}{}'.format(recastapi.ENDPOINTS['RESPONSE_ARCHIVES'], selection_str)

recastapi/response/write.py

@@ -73,11 +73,12 @@ def basic_response(point_response_id,basic_request_id,result_data, description,
     existing = recastapi.response.read.basic_response(basic_request_id_filter = basic_request_id, description_filter = description)
     if existing and replace_existing:
         existing = existing[0]
-        url = '{}/{}'.format(recastapi.ENDPOINTS['BASIC_RESPONSES'],existing['id'])
+        existing_id = existing['id']
+        url = '{}/{}'.format(recastapi.ENDPOINTS['BASIC_RESPONSES'],existing_id)
         existing.update(**payload)
         existing = {k:v for k,v in existing.iteritems() if not (k.startswith('_') or k=='id')}
         recastapi.patch(url,existing)
-        return recastapi.response.read.basic_response(basic_request_id = basic_request_id)
+        return recastapi.response.read.basic_response(basic_response_id = existing_id)
     url = '{}/'.format(recastapi.ENDPOINTS['BASIC_RESPONSES'])
     return recastapi.post(url, json=payload)