Merge pull request #15 from bstoney/master

Changed Windows paths to use environment variables

Author: castarco

src/Installer/WindowsInstaller.php

@@ -27,7 +27,7 @@ protected function getInstallPath()
         $currentUser = $this->system->getCurrentUser();
 
         if ('Administrator' === $currentUser) {
-            return 'C:\ProgramData\jupyter-php';
+            return $this->getProgramDataPath() . '\jupyter-php';
         } else {
             return $this->system->getCurrentUserHome() . '\.jupyter-php';
         }
@@ -48,8 +48,8 @@ protected function installKernel()
         $currentUser = $this->system->getCurrentUser();
 
         $kernelSpecPath = ('Administrator' === $currentUser) ?
-            'C:\ProgramData\jupyter\kernels\jupyter-php' :
-            $this->system->getCurrentUserHome() . '\AppData\Roaming\jupyter\kernels\jupyter-php';
+            $this->getProgramDataPath() . '\jupyter\kernels\jupyter-php' :
+            $this->getAppDataPath() . '\jupyter\kernels\jupyter-php';
 
         $this->system->ensurePath($kernelSpecPath);
         file_put_contents($kernelSpecPath . '\kernel.json', $kernelDef);
@@ -73,4 +73,22 @@ protected function executeSilentComposerCommand($installPath)
 
         return $composerStatus;
     }
+
+    private function getProgramDataPath()
+    {
+        if (function_exists('getenv') && false !== getenv('PROGRAMDATA')) {
+            return getenv('PROGRAMDATA');
+        } else {
+            throw new \RuntimeException('Unable to obtain the program data directory.');
+        }
+    }
+
+    private function getAppDataPath()
+    {
+        if (function_exists('getenv') && false !== getenv('APPDATA')) {
+            return getenv('APPDATA');
+        } else {
+            throw new \RuntimeException('Unable to obtain the app data directory.');
+        }
+    }
 }