| @@ -41,7 +41,7 @@ | ||
| 41 | 41 | /** | 
| 42 | 42 | * Delete the folder and all files/folders within it | 
| 43 | 43 | * | 
| 44 | - * @param $path | |
| 44 | + * @param string $path | |
| 45 | 45 | * | 
| 46 | 46 | * @return bool | 
| 47 | 47 | */ | 
| @@ -41,7 +41,7 @@ | ||
| 41 | 41 | /** | 
| 42 | 42 | * Delete the folder and all files/folders within it | 
| 43 | 43 | * | 
| 44 | - * @param $path | |
| 44 | + * @param string $path | |
| 45 | 45 | * | 
| 46 | 46 | * @return bool | 
| 47 | 47 | */ | 
| @@ -208,7 +208,7 @@ discard block | ||
| 208 | 208 |      { | 
| 209 | 209 | $file = m::mock(FileNodeInterface::class); | 
| 210 | 210 |          $file->shouldReceive('__toString') | 
| 211 | -             ->andReturn('some/file/here'); | |
| 211 | +                ->andReturn('some/file/here'); | |
| 212 | 212 | |
| 213 | 213 | $this->expectException(InvalidArgumentException::class); | 
| 214 | 214 | |
| @@ -220,7 +220,7 @@ discard block | ||
| 220 | 220 | $process = m::mock(Process::class)->makePartial(); | 
| 221 | 221 |          $process->shouldReceive('isSuccessful')->andReturn(false); | 
| 222 | 222 |          $this->processFactory->shouldReceive('createProcess') | 
| 223 | - ->andReturn($process); | |
| 223 | + ->andReturn($process); | |
| 224 | 224 | |
| 225 | 225 | $file = new LocalFile(static::$dir . 'failed_tail.test'); | 
| 226 | 226 |          $file->put('nothing interesting here'); | 
| @@ -41,7 +41,7 @@ | ||
| 41 | 41 | /** | 
| 42 | 42 | * Delete the folder and all files/folders within it | 
| 43 | 43 | * | 
| 44 | - * @param $path | |
| 44 | + * @param string $path | |
| 45 | 45 | * | 
| 46 | 46 | * @return bool | 
| 47 | 47 | */ | 
| @@ -208,7 +208,7 @@ discard block | ||
| 208 | 208 |      { | 
| 209 | 209 | $file = m::mock(FileNodeInterface::class); | 
| 210 | 210 |          $file->shouldReceive('__toString') | 
| 211 | -             ->andReturn('some/file/here'); | |
| 211 | +                ->andReturn('some/file/here'); | |
| 212 | 212 | |
| 213 | 213 | $this->expectException(InvalidArgumentException::class); | 
| 214 | 214 | |
| @@ -220,7 +220,7 @@ discard block | ||
| 220 | 220 | $process = m::mock(Process::class)->makePartial(); | 
| 221 | 221 |          $process->shouldReceive('isSuccessful')->andReturn(false); | 
| 222 | 222 |          $this->processFactory->shouldReceive('createProcess') | 
| 223 | - ->andReturn($process); | |
| 223 | + ->andReturn($process); | |
| 224 | 224 | |
| 225 | 225 | $file = new LocalFile(static::$dir . 'failed_tail.test'); | 
| 226 | 226 |          $file->put('nothing interesting here'); | 
| @@ -43,7 +43,7 @@ | ||
| 43 | 43 | * process | 
| 44 | 44 | * @param array|null $env The environment variables or null to inherit | 
| 45 | 45 | * @param string|null $input The input | 
| 46 | - * @param int|float|null $timeout The timeout in seconds or null to disable | |
| 46 | + * @param integer $timeout The timeout in seconds or null to disable | |
| 47 | 47 | * @param array $options An array of options for proc_open | 
| 48 | 48 | * | 
| 49 | 49 | * @return Process | 
| @@ -10,7 +10,7 @@ | ||
| 10 | 10 | const UNKNOWN = 'unknown'; | 
| 11 | 11 | |
| 12 | 12 | /** | 
| 13 | - * @return array | |
| 13 | + * @return string[] | |
| 14 | 14 | */ | 
| 15 | 15 | public static function getCompressionTypes() | 
| 16 | 16 |      { | 
| @@ -20,7 +20,7 @@ | ||
| 20 | 20 | } | 
| 21 | 21 | |
| 22 | 22 | /** | 
| 23 | - * @return mixed | |
| 23 | + * @return string | |
| 24 | 24 | */ | 
| 25 | 25 | public function getDirectory() | 
| 26 | 26 |      { | 
| @@ -29,7 +29,7 @@ | ||
| 29 | 29 | */ | 
| 30 | 30 | public function findFiles(FileNodeCollectionInterface $files) | 
| 31 | 31 |      { | 
| 32 | -        return $files->filter(function (FileNode $file) { | |
| 32 | +        return $files->filter(function(FileNode $file) { | |
| 33 | 33 | $metadata = $file->getMetadata(); | 
| 34 | 34 |              if ($metadata) { | 
| 35 | 35 | return ($this->filter->matches($file->getMetadata())); | 
| @@ -40,8 +40,8 @@ discard block | ||
| 40 | 40 | } | 
| 41 | 41 | |
| 42 | 42 | $outputFile = $node->getClone() | 
| 43 | - ->setPath($pathInfo['dirname'] . '/' . $pathInfo['filename'] . '.' . $this->getExtension()) | |
| 44 | - ->setCompression($this->getCompression()); | |
| 43 | + ->setPath($pathInfo['dirname'] . '/' . $pathInfo['filename'] . '.' . $this->getExtension()) | |
| 44 | + ->setCompression($this->getCompression()); | |
| 45 | 45 | |
| 46 | 46 |          $this->log(LogLevel::INFO, "Compressing file: {file} into {target} using {compression}", [ | 
| 47 | 47 | 'file' => $node, | 
| @@ -76,8 +76,8 @@ discard block | ||
| 76 | 76 | } | 
| 77 | 77 | |
| 78 | 78 | $outputFile = $node->getClone() | 
| 79 | - ->setPath($pathInfo['dirname'] . '/' . $pathInfo['filename']) | |
| 80 | - ->setCompression(CompressionType::NONE); | |
| 79 | + ->setPath($pathInfo['dirname'] . '/' . $pathInfo['filename']) | |
| 80 | + ->setCompression(CompressionType::NONE); | |
| 81 | 81 | |
| 82 | 82 |          $this->log(LogLevel::INFO, "DeCompressing file: {file} into {target} using {compression}", [ | 
| 83 | 83 | 'file' => $node, | 
| @@ -67,7 +67,7 @@ discard block | ||
| 67 | 67 | |
| 68 | 68 |          $this->log(LogLevel::INFO, "Merging files in collection $files into: {$target}"); | 
| 69 | 69 | |
| 70 | -        $filePaths = $files->map(function (LocalFile $item) { | |
| 70 | +        $filePaths = $files->map(function(LocalFile $item) { | |
| 71 | 71 | return $item->getPath(); | 
| 72 | 72 | }); | 
| 73 | 73 | |
| @@ -105,7 +105,7 @@ discard block | ||
| 105 | 105 | |
| 106 | 106 |          if (!$keepOld) { | 
| 107 | 107 | $this->log(LogLevel::DEBUG, "Deleting old files in collection $files"); | 
| 108 | -            $files->map(function (LocalFile $item) { | |
| 108 | +            $files->map(function(LocalFile $item) { | |
| 109 | 109 |                  if ($item->exists()) { | 
| 110 | 110 | $item->delete(); | 
| 111 | 111 | } |