reapack

commit 7feb68c30610e907d0377e715a8ace35abcccc92
parent 445912aa5afbbcb2df34a2f1e39ed79ba74dae13
Author: cfillion <cfillion@users.noreply.github.com>
Date:   Fri, 14 Oct 2016 20:16:06 -0700

about pkg: fix locate in browser/finder feature on windows

Diffstat:
M
src/about.cpp
 | 
14
+++++++++-----

1 file changed, 9 insertions(+), 5 deletions(-)
diff --git a/src/about.cpp b/src/about.cpp
@@ -20,6 +20,7 @@
 #include "browser.hpp"
 #include "encoding.hpp"
 #include "errors.hpp"
+#include "filesystem.hpp"
 #include "index.hpp"
 #include "listview.hpp"
 #include "menu.hpp"
@@ -548,8 +549,10 @@ bool AboutPackageDelegate::fillContextMenu(Menu &menu, const int index) const
   if(index < 0)
     return false;
 
+  const Source *src = m_sources->at(index);
+
   menu.addAction(AUTO_STR("Copy source URL"), ACTION_COPY_URL);
-  menu.setEnabled(m_current.size() > 0,
+  menu.setEnabled(m_current.size() > 0 && FS::exists(src->targetPath()),
     menu.addAction(AUTO_STR("Locate in explorer/finder"), ACTION_LOCATE));
 
   return true;
@@ -589,10 +592,11 @@ void AboutPackageDelegate::copySourceUrl()
 void AboutPackageDelegate::locate()
 {
   if(const Source *src = currentSource()) {
-    stringstream arg;
-    arg << "/select," << quoted(Path::prefixRoot(src->targetPath()).join());
+    string arg = "/select,\"";
+    arg += Path::prefixRoot(src->targetPath()).join();
+    arg += '"';
 
-    ShellExecute(nullptr, nullptr, AUTO_STR("explorer.exe"),
-      make_autostring(arg.str()).c_str(), nullptr, SW_SHOW);
+    ShellExecute(nullptr, AUTO_STR("open"), AUTO_STR("explorer.exe"),
+      make_autostring(arg).c_str(), nullptr, SW_SHOW);
   }
 }