jkuchar / BigFileTools

src/File.php

Doc Comments +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
 	/**
 	 * Moves file to new location / rename
 	 * @param string $destination path
-	 * @return bool
+	 * @return boolean|null
 	 * @throws Exception
 	 */
 	function move($destination)

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
 		// $readable = is_readable($file); // does not always return correct value for directories
 
 		$fp = @fopen($this->path, "r"); // must be file and must be readable
-		if($fp) {
+		if ($fp) {
 			fclose($fp);
 			return true;
 		}

examples/example-deprecated.php

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
 <?php
 
 require '../vendor/autoload.php';
-echo $f = BigFileTools::fromPath(__FILE__)->getSize()." bytes";
\ No newline at end of file
+echo $f = BigFileTools::fromPath(__FILE__)->getSize() . " bytes";
\ No newline at end of file

src/Driver/ComDriver.php

Braces +3 added lines, -2 removed lines

@@ -22,8 +22,9 @@
 	{
 		// Use the Windows COM interface
 		$fsobj = new \COM('Scripting.FileSystemObject');
-		if (dirname($path) == '.')
-			$this->path = ((substr(getcwd(), -1) == DIRECTORY_SEPARATOR) ? getcwd() . basename($path) : getcwd() . DIRECTORY_SEPARATOR . basename($path));
+		if (dirname($path) == '.') {
+					$this->path = ((substr(getcwd(), -1) == DIRECTORY_SEPARATOR) ? getcwd() . basename($path) : getcwd() . DIRECTORY_SEPARATOR . basename($path));
+		}
 		$f = $fsobj->GetFile($path);
 		return BigInteger::of($f->Size);
 	}

src/Driver/NativeSeekDriver.php

Spacing +3 added lines, -3 removed lines

@@ -32,15 +32,15 @@
 		// TODO: There is *hope* that ftell() or fseek() fails when file is over 4GB
 		// TODO: This really needs tests, any ideas how to test this in CI? (please let me know)
 
-		if($flockResult === false) {
+		if ($flockResult === false) {
 			throw new Exception("Couldn't get file lock. Operation abandoned.");
 		}
 
-		if($seekResult !== 0) {
+		if ($seekResult !== 0) {
 			throw new Exception("Seeking to end of file failed");
 		}
 
-		if($position === false) {
+		if ($position === false) {
 			throw new Exception("Cannot determine position in file. ftell() failed.");
 		}
 

src/Utils.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 	{
 
 		$path = realpath($path);
-		if(!$path) {
+		if (!$path) {
 			// TODO: use hack like http://stackoverflow.com/questions/4049856/replace-phps-realpath or http://www.php.net/manual/en/function.realpath.php#84012
 			//       probably as optional feature that can be turned on when you know, what are you doing
 

src/Driver/SizeDriverAggregator.php

Spacing +3 added lines, -3 removed lines

@@ -31,7 +31,7 @@ 
 	 */
 	public function __construct(array $drivers)
 	{
-		foreach($drivers as $driver) {
+		foreach ($drivers as $driver) {
 			$this->addDriver($driver);
 		}
 	}
@@ -60,8 +60,8 @@ 
 	public function getFileSize($path)
 	{
 		$this->lastExceptions = [];
-		foreach($this->drivers as $driver) {
-			try{
+		foreach ($this->drivers as $driver) {
+			try {
 				$result = $driver->getFileSize($path);
 				$this->lastUsedDriver = $driver;
 				return $result;

src/Driver/SizeDriverFactory.php

Spacing +4 added lines, -4 removed lines

@@ -19,12 +19,12 @@
 	 */
 	public function __construct(array $drivers)
 	{
-		foreach($drivers as $driver) {
-			try{
-				if(is_callable($driver)) {
+		foreach ($drivers as $driver) {
+			try {
+				if (is_callable($driver)) {
 					$this->addDriver($driver());
 
-				} else if(class_exists($driver)) {
+				} else if (class_exists($driver)) {
 					$this->addDriver(new $driver());
 
 				} else {

src/Driver/ExecDriver.php

Doc Comments +10 added lines, -1 removed lines

@@ -46,7 +46,7 @@ 
 	 * Convert string into integer
 	 * Must be precise number, otherwise you will see and exception.
 	 *
-	 * @param $valueAsString
+	 * @param string $valueAsString
 	 * @return BigInteger
 	 * @throws Exception
 	 */
@@ -65,6 +65,9 @@ 
 
 	}
 
+	/**
+	 * @param string $path
+	 */
 	private function getFileSizeWindows($path)
 	{
 		$escapedPath = escapeshellarg($path);
@@ -73,6 +76,9 @@ 
 		);
 	}
 
+	/**
+	 * @param string $path
+	 */
 	private function getFileSizeLinux($path)
 	{
 		$escapedPath = escapeshellarg($path);
@@ -81,6 +87,9 @@ 
 		);
 	}
 
+	/**
+	 * @param string $path
+	 */
 	private function getFileSizeMac($path)
 	{
 		$escapedPath = escapeshellarg($path);

Spacing +5 added lines, -5 removed lines

@@ -17,7 +17,7 @@ 
 			throw new PrerequisiteException("Exec function is disabled");
 		}
 
-		if(strtoupper(substr(PHP_OS, 0, 3)) == 'WIN') {
+		if (strtoupper(substr(PHP_OS, 0, 3)) == 'WIN') {
 			$this->os = self::OS_WINDOWS;
 		} elseif (strtoupper(PHP_OS) == "DARWIN") {
 			$this->os = self::OS_MAC;
@@ -34,7 +34,7 @@ 
 	 */
 	public function getFileSize($path)
 	{
-		switch($this->os) {
+		switch ($this->os) {
 			case self::OS_WINDOWS: return $this->getFileSizeWindows($path); break;
 			case self::OS_LINUX:   return $this->getFileSizeLinux($path); break;
 			case self::OS_MAC:     return $this->getFileSizeMac($path); break;
@@ -51,8 +51,8 @@ 
 	 * @throws Exception
 	 */
 	private function convertToInteger($valueAsString) {
-		if(!is_string($valueAsString)) {
-			throw new Exception("Cannot convert to integer. Expected string, but got " . gettype($valueAsString). ".");
+		if (!is_string($valueAsString)) {
+			throw new Exception("Cannot convert to integer. Expected string, but got " . gettype($valueAsString) . ".");
 		}
 		$trimmedInput = trim($valueAsString);
 
@@ -60,7 +60,7 @@ 
 			return BigInteger::of($trimmedInput);
 
 		} catch (ArithmeticException $e) {
-			throw new Exception("Returned value cannot be converted to an integer.",0, $e);
+			throw new Exception("Returned value cannot be converted to an integer.", 0, $e);
 		}
 
 	}