thecodingmachine / nodejs-installer

src/NodeJsInstaller.php

Spacing +5 added lines, -5 removed lines

@@ -81,9 +81,9 @@ 
             // We want to get output from stdout, not from stderr.
             // Therefore, we use proc_open.
             $descriptorspec = array(
-                0 => array("pipe", "r"),  // stdin
-                1 => array("pipe", "w"),  // stdout
-                2 => array("pipe", "w"),  // stderr
+                0 => array("pipe", "r"), // stdin
+                1 => array("pipe", "w"), // stdout
+                2 => array("pipe", "w"), // stderr
             );
             $pipes = array();
 
@@ -151,9 +151,9 @@ 
             }
         } elseif (Environment::isWindows() && Environment::getArchitecture() == 64) {
             if (version_compare($version, '4.0.0') >= 0) {
-                return "https://nodejs.org/dist/v" . $version . "/win-x64/node.exe";
+                return "https://nodejs.org/dist/v".$version."/win-x64/node.exe";
             } else {
-                return "https://nodejs.org/dist/v" . $version . "/x64/node.exe";
+                return "https://nodejs.org/dist/v".$version."/x64/node.exe";
             }
         } elseif (Environment::isMacOS() && Environment::getArchitecture() == 32) {
             return "https://nodejs.org/dist/v".$version."/node-v".$version."-darwin-x86.tar.gz";

Doc Comments +10 added lines

@@ -110,6 +110,7 @@ 
      *
      * Note: trailing "v" will be removed from version string.
      *
+     * @param string $binDir
      * @return null|string
      */
     public function getNodeJsLocalInstallVersion($binDir)
@@ -289,6 +290,10 @@ 
         }
     }
 
+    /**
+     * @param string $targetDir
+     * @param boolean $isLocal
+     */
     public function createBinScripts($binDir, $targetDir, $isLocal)
     {
         if (!file_exists($binDir)) {
@@ -316,6 +321,7 @@ 
      * @param string $fullTargetDir
      * @param string $scriptName
      * @param bool   $isLocal
+     * @param string $target
      */
     private function createBinScript($binDir, $fullTargetDir, $scriptName, $target, $isLocal)
     {
@@ -376,6 +382,10 @@ 
         return (strlen($relativePath) === 0) ? './' : $relativePath;
     }
 
+    /**
+     * @param string $zipFileName
+     * @param string $targetDir
+     */
     private function unzip($zipFileName, $targetDir)
     {
         $zip = new \ZipArchive();

src/NodeJsPlugin.php

Doc Comments +1 added lines

@@ -243,6 +243,7 @@
     /**
      * Uninstalls NodeJS.
      * Note: other classes cannot be loaded here since the package has already been removed.
+     * @param string $targetDir
      */
     private function onUninstall($binDir, $targetDir)
     {