Create the experiment_list context menu just once

Author: Ezequiel Torres

MLC/GUI/mlc_gui.py

@@ -86,6 +86,29 @@ def __init__(self):
         # When an ExperimentDialog is closed, update the experiment info
         self.experiment_closed.connect(self.load_and_refresh_experiment_info)
 
+        # Create the experiment list context menu
+        self._context_menu = QMenu(self)
+
+        action_open = QAction("Open", self)
+        action_open.triggered.connect(self.on_open_button_clicked)
+        self._context_menu.addAction(action_open)
+
+        action_remove = QAction("Remove", self)
+        action_remove.triggered.connect(self.on_remove_button_clicked)
+        self._context_menu.addAction(action_remove)
+
+        action_clone = QAction("Clone", self)
+        action_clone.triggered.connect(self.on_clone_button_clicked)
+        self._context_menu.addAction(action_clone)
+
+        action_export = QAction("Export", self)
+        action_export.triggered.connect(self.on_export_button_clicked)
+        self._context_menu.addAction(action_export)
+
+        action_rename = QAction("Rename", self)
+        action_rename.triggered.connect(self.on_rename_button_clicked)
+        self._context_menu.addAction(action_rename)
+
     def on_edit_workspace_clicked(self):
         logger.debug('[MLC_MANAGER] [EDIT_WORKSPACE] - On_edit_workspace_location triggered')
         new_workspace_dir = QFileDialog.getExistingDirectory(self, 'Choose the new workspace directory',
@@ -386,29 +409,11 @@ def on_rename_button_clicked(self):
     def on_experiment_list_context_menu(self, point):
         logger.debug('[MLC_MANAGER] [LIST_CONTEXT_MENU] - Context Menu displayed')
 
-        menu = QMenu(self)
-
-        action_open = QAction("Open", self)
-        action_open.triggered.connect(self.on_open_button_clicked)
-        menu.addAction(action_open)
-
-        action_remove = QAction("Remove", self)
-        action_remove.triggered.connect(self.on_remove_button_clicked)
-        menu.addAction(action_remove)
-
-        action_clone = QAction("Clone", self)
-        action_clone.triggered.connect(self.on_clone_button_clicked)
-        menu.addAction(action_clone)
-
-        action_export = QAction("Export", self)
-        action_export.triggered.connect(self.on_export_button_clicked)
-        menu.addAction(action_export)
-
-        action_rename = QAction("Rename", self)
-        action_rename.triggered.connect(self.on_rename_button_clicked)
-        menu.addAction(action_rename)
+        if self._experiment_selected is None:
+            logger.info("[MLC MANAGER] [RENAME_BUTTON] - No experiment was selected yet. Don't do anything")
+            return
 
-        menu.popup(self._autogenerated_object.experiment_list.mapToGlobal(point))
+        self._context_menu.popup(self._autogenerated_object.experiment_list.mapToGlobal(point))
 
     def on_experiment_list_clicked(self, model_index):
         list_view = self._autogenerated_object.experiment_list