Merge pull request #2151 from antgonza/fix-delete-sample

fixing error in sample delete
qiita-spots · Jun 21, 2017 · 02bff6c · 02bff6c
2 parents 9bb1fd5 + 2626da1
commit 02bff6c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/qiita_pet/handlers/api_proxy/sample_template.py b/qiita_pet/handlers/api_proxy/sample_template.py
@@ -521,7 +521,9 @@ def sample_template_patch_request(user_id, req_op, req_path, req_value=None,
     if req_op == 'remove':
         req_path = [v for v in req_path.split('/') if v]
 
-        # format: study_id/row_id/column|sample/attribute_id
+        # format
+        # column: study_id/row_id/columns/column_name
+        # sample: study_id/row_id/samples/sample_id
         if len(req_path) != 4:
             return {'status': 'error',
                     'message': 'Incorrect path parameter'}

diff --git a/qiita_pet/templates/study_ajax/sample_prep_summary.html b/qiita_pet/templates/study_ajax/sample_prep_summary.html
@@ -65,7 +65,7 @@
       $.ajax({
         url: '{% raw qiita_config.portal_dir %}/study/description/sample_template/',
         type: 'PATCH',
-        data: {'op': 'remove', 'path': '/{{study_id}}/samples/' + sample_name},
+        data: {'op': 'remove', 'path': '/{{study_id}}/' + row_id + '/samples/' + column_name},
         success: function(data) {
           if(data.status == 'error') {
             bootstrapAlert(data.message, "danger");