addressing @josenavas comments

Author: antgonza

qiita_pet/handlers/study_handlers/description_handlers.py

@@ -29,6 +29,8 @@
                                  QiitaDBDuplicateHeaderError, QiitaDBError)
 from qiita_pet.handlers.base_handlers import BaseHandler
 from qiita_pet.handlers.util import check_access
+from qiita_pet.handlers.study_handlers.listing_handlers import (
+    ListStudiesHandler)
 
 html_error_message = "<b>An error occurred %s %s</b></br>%s"
 
@@ -654,10 +656,18 @@ def delete_study(self, study, user, callback):
             is done and it fails
         """
         study_id = study.id
+        study_title = study.title
 
         try:
             Study.delete(study_id)
-            self.redirect('/study/list/')
+
+            # redirecting to list but also passing messages
+            # we need to change the request.method to GET
+            self.request.method = 'GET'
+            ListStudiesHandler(self.application, self.request)._execute(
+                [t(self.request) for t in self.application.transforms],
+                message=('Study "%s" has been deleted' % study_title),
+                msg_level='success')
         except Exception as e:
             msg = "Couldn't remove study %d: %s" % (study_id, str(e))
             msg_level = "danger"

qiita_pet/handlers/study_handlers/listing_handlers.py

@@ -123,13 +123,16 @@ def _check_owner(user, study):
 class ListStudiesHandler(BaseHandler):
     @authenticated
     @coroutine
-    def get(self):
+    def get(self, message="", msg_level=None):
         all_emails_except_current = yield Task(self._get_all_emails)
         all_emails_except_current.remove(self.current_user.id)
         avail_meta = SampleTemplate.metadata_headers() +\
             get_table_cols("study")
-        self.render('list_studies.html', availmeta=avail_meta,
-                    all_emails_except_current=all_emails_except_current)
+        self.render('list_studies.html',
+                    availmeta=avail_meta,
+                    all_emails_except_current=all_emails_except_current,
+                    message=message,
+                    msg_level=msg_level)
 
     def _get_all_emails(self, callback):
         callback(list(User.iter()))