silverstripe / sspak

src/Webroot.php

Braces +12 added lines, -4 removed lines

@@ -16,7 +16,9 @@ 
 	 * Calls sniff once and then caches
 	 */
 	public function details() {
-		if(!$this->details) $this->details = $this->sniff();
+		if(!$this->details) {
+			$this->details = $this->sniff();
+		}
 		return $this->details;
 	}
 
@@ -26,7 +28,9 @@ 
 	public function sniff() {
 		global $snifferFileContent;
 
-		if(!$snifferFileContent) $snifferFileContent = file_get_contents(PACKAGE_ROOT . 'src/sspak-sniffer.php');
+		if(!$snifferFileContent) {
+			$snifferFileContent = file_get_contents(PACKAGE_ROOT . 'src/sspak-sniffer.php');
+		}
 
 		$remoteSniffer = '/tmp/sspak-sniffer-' . rand(100000,999999) . '.php';
 		$this->uploadContent($snifferFileContent, $remoteSniffer);
@@ -35,7 +39,9 @@ 
 		$this->unlink($remoteSniffer);
 
 		$parsed = @unserialize($result['output']);
-		if(!$parsed) throw new Exception("Could not parse sspak-sniffer content:\n{$result['output']}\n");
+		if(!$parsed) {
+			throw new Exception("Could not parse sspak-sniffer content:\n{$result['output']}\n");
+		}
 		return $parsed;
 	}
 
@@ -45,7 +51,9 @@ 
 	 */
 	public function execSudo($command, $options = array()) {
 		if($this->sudo) {
-			if(is_array($command)) $command = $this->executor->commandArrayToString($command);
+			if(is_array($command)) {
+				$command = $this->executor->commandArrayToString($command);
+			}
 			// Try running sudo without asking for a password
 			try {
 				return $this->exec("sudo -n -u " . escapeshellarg($this->sudo) . " " . $command, $options);

Doc Comments +2 added lines, -3 removed lines

@@ -68,9 +68,8 @@ 
 
 	/**
 	 * Put the database from the given sspak file into this webroot.
-	 * @param array $details The previously sniffed details of this webroot
 	 * @param bool $dropdb Drop the DB prior to install
-	 * @param string $sspakFile Filename
+	 * @param SSPakFile $sspak Filename
 	 */
 	public function putdb($sspak, $dropdb) {
 		$details = $this->details();
@@ -138,7 +137,7 @@ 
 	}
 
 	/**
-	 * @param $sspak SSPakFile SSPak file to extract assets from
+	 * @param SSPakFile $sspak SSPakFile SSPak file to extract assets from
 	 * @todo There should be a return value or exception thrown to indicate success or failure to put assets
 	 */
 	public function putassets($sspak) {