Changeset 390494d in sasview for src/sas/qtgui

Timestamp:

Feb 22, 2019 9:52:14 AM (5 years ago)

Author:

GitHub <noreply@…>

Branches:

ESS_GUI, ESS_GUI_bumps_abstraction, ESS_GUI_iss1116, ESS_GUI_opencl, ESS_GUI_sync_sascalc

Children:

1078936c, 44a698c

Parents:

091eee64 (diff), afdb3b3 (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

git-author:

Piotr Rozyczko <piotr.rozyczko@…> (02/22/19 09:52:14)

git-committer:

GitHub <noreply@…> (02/22/19 09:52:14)

Message:

Merge pull request #206 from mrakitin/fix_installer

FIX: add a proper entry point

File:

• src/sas/qtgui/MainWindow/GuiManager.py (modified) (1 diff)

src/sas/qtgui/MainWindow/GuiManager.py

@@ -1019 +1019 @@
     def actionTutorial(self):
         """
-        Open the tutorial PDF file with default PDF renderer
-        """
-        # Not terribly safe here. Shell injection warning.
-        # isfile() helps but this probably needs a better solution.
-        if os.path.isfile(self._tutorialLocation):
-            result = subprocess.Popen([self._tutorialLocation], shell=True)
+        Open the page with tutorial PDF links
+        """
+        helpfile = "/user/tutorial.html"
+        self.showHelp(helpfile)
 
     def actionAcknowledge(self):