storeman-io / storeman

src/Storeman.php

Doc Comments +6 added lines, -2 removed lines

@@ -154,7 +154,7 @@ 
     }
 
     /**
-     * @param Vault[] $vaults
+     * @param \Traversable $vaults
      * @param string $lockName
      */
     protected function acquireLocks(\Traversable $vaults, string $lockName)
@@ -169,7 +169,7 @@ 
     }
 
     /**
-     * @param Vault[] $vaults
+     * @param \Traversable $vaults
      * @param string $lockName
      */
     protected function releaseLocks(\Traversable $vaults, string $lockName)
@@ -183,6 +183,10 @@ 
         }
     }
 
+    /**
+     * @param null|integer $revision
+     * @param null|string $vaultTitle
+     */
     protected function getVaultForDownload(?int $revision, ?string $vaultTitle): ?Vault
     {
         $revision = $revision ?: $this->getLastRevision();

Spacing +2 added lines, -2 removed lines

@@ -116,7 +116,7 @@ 
 
     public function restore(int $toRevision = null, string $fromVault = null, SynchronizationProgressListenerInterface $progressListener = null): OperationResultList
     {
-        $this->getLogger()->notice(sprintf("Restoring from %s...", $toRevision ? "r{$toRevision}": 'latest revision'));
+        $this->getLogger()->notice(sprintf("Restoring from %s...", $toRevision ? "r{$toRevision}" : 'latest revision'));
 
         $vault = $this->getVaultForDownload($toRevision, $fromVault);
 
@@ -132,7 +132,7 @@ 
 
     public function dump(string $targetPath, int $revision = null, string $fromVault = null, SynchronizationProgressListenerInterface $progressListener = null): OperationResultList
     {
-        $this->getLogger()->notice(sprintf("Dumping from %s to {$targetPath}...", $revision ? "r{$revision}": 'latest revision'));
+        $this->getLogger()->notice(sprintf("Dumping from %s to {$targetPath}...", $revision ? "r{$revision}" : 'latest revision'));
 
         $vault = $this->getVaultForDownload($revision, $fromVault);
 

Braces +2 added lines, -4 removed lines

@@ -166,8 +166,7 @@ 
                 $this->getLogger()->debug("Vault {$vault->getIdentifier()} is at r{$lastSynchronization->getRevision()}");
 
                 $max = max($max, $lastSynchronization->getRevision());
-            }
-            else
+            } else
             {
                 $this->getLogger()->debug("Vault {$vault->getIdentifier()} has no synchronizations yet");
             }
@@ -226,8 +225,7 @@ 
         if ($vaultTitle)
         {
             $vault = $vaultContainer->getVaultByTitle($vaultTitle);
-        }
-        else
+        } else
         {
             $vaults = $vaultContainer->getVaultsHavingRevision($revision);
             $vault = $vaultContainer->getPrioritizedVault($vaults);

src/Index/Index.php

Braces +1 added lines, -2 removed lines

@@ -41,8 +41,7 @@
             if ($parentNode === null)
             {
                 throw new Exception("Trying to add object {$indexObject->getRelativePath()} without existing parent node");
-            }
-            elseif (!$parentNode->getIndexObject()->isDirectory())
+            } elseif (!$parentNode->getIndexObject()->isDirectory())
             {
                 throw new Exception("Trying to add object {$indexObject->getRelativePath()} under parent node which is not a directory");
             }

src/Cli/Application.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 
 class Application extends \Symfony\Component\Console\Application
 {
-    public const LOGO =  <<<TXT
+    public const LOGO = <<<TXT
   ____  _                                       
  / ___|| |_ ___  _ __ ___ _ __ ___   __ _ _ __  
  \___ \| __/ _ \| '__/ _ \ '_ ` _ \ / _` | '_ \ 

src/StorageAdapter/FlysystemStorageAdapter.php

Braces +4 added lines, -8 removed lines

@@ -46,8 +46,7 @@ 
             {
                 throw new Exception(sprintf('writeStream() failed for %s.', $relativePath));
             }
-        }
-        catch (\Exception $exception)
+        } catch (\Exception $exception)
         {
             throw new Exception($exception->getMessage(), 0, $exception);
         }
@@ -58,8 +57,7 @@ 
         try
         {
             return $this->filesystem->has($relativePath);
-        }
-        catch (\Exception $exception)
+        } catch (\Exception $exception)
         {
             throw new Exception($exception->getMessage(), 0, $exception);
         }
@@ -77,8 +75,7 @@ 
             {
                 throw new Exception(sprintf('unlink() failed for %s', $relativePath));
             }
-        }
-        catch (\Exception $exception)
+        } catch (\Exception $exception)
         {
             throw new Exception($exception->getMessage(), 0, $exception);
         }
@@ -98,8 +95,7 @@ 
             }
 
             return $stream;
-        }
-        catch (\Exception $exception)
+        } catch (\Exception $exception)
         {
             throw new Exception($exception->getMessage(), 0, $exception);
         }

src/LockAdapter/AbstractLockAdapter.php

Braces +1 added lines, -2 removed lines

@@ -96,8 +96,7 @@
 
                 unset($this->lockDepthMap[$name]);
             }
-        }
-        else
+        } else
         {
             $this->logger->info("Requested lock release for non-hold lock '{$name}'. Doing nothing.");
         }

src/IndexBuilder/StandardIndexBuilder.php

Braces +2 added lines, -4 removed lines

@@ -17,12 +17,10 @@
         if (!file_exists($path))
         {
             throw new \InvalidArgumentException("Given path '{$path}' does not exist.");
-        }
-        elseif (!is_dir($path))
+        } elseif (!is_dir($path))
         {
             throw new \InvalidArgumentException("Given path '{$path}' is not a directory.");
-        }
-        elseif (!is_readable($path))
+        } elseif (!is_readable($path))
         {
             throw new \InvalidArgumentException("Given directory '{$path}' is not readable.'");
         }

doc/encryptionPerformanceTest.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
     $contexts[$algorithm] = hash_init($algorithm);
 }
 
-while(!feof($fileHandle))
+while (!feof($fileHandle))
 {
     $buffer = fread($fileHandle, 65536);
 

src/OperationListBuilder/OperationListBuilderInterface.php

Doc Comments -1 removed lines

@@ -17,7 +17,6 @@
      *
      * @param Index $mergedIndex
      * @param Index $localIndex
-     * @param Index|null $remoteIndex
      * @return OperationList
      */
     public function buildOperationList(Index $mergedIndex, Index $localIndex): OperationList;

src/IndexMerger/StandardIndexMerger.php

Doc Comments +12 added lines

@@ -100,6 +100,10 @@ 
         return $mergedIndex;
     }
 
+    /**
+     * @param IndexObject|null $localObject
+     * @param IndexObject|null $lastLocalObject
+     */
     protected function isLocalObjectModified(?IndexObject $localObject, ?IndexObject $lastLocalObject, int $options): bool
     {
         if (!$lastLocalObject)
@@ -132,6 +136,10 @@ 
         return $localObjectModified;
     }
 
+    /**
+     * @param IndexObject|null $remoteObject
+     * @param IndexObject|null $lastLocalObject
+     */
     protected function isRemoteObjectModified(?IndexObject $remoteObject, ?IndexObject $lastLocalObject): bool
     {
         if ($lastLocalObject)
@@ -144,6 +152,10 @@ 
         }
     }
 
+    /**
+     * @param IndexObject|null $localObject
+     * @param IndexObject|null $lastLocalObject
+     */
     protected function resolveConflict(ConflictHandlerInterface $conflictHandler, IndexObject $remoteObject, ?IndexObject $localObject, ?IndexObject $lastLocalObject): IndexObject
     {
         $this->logger->notice("Resolving conflict at {$remoteObject->getRelativePath()}...");

Braces +3 added lines, -6 removed lines

@@ -75,12 +75,10 @@ 
             if ($localObjectModified && $remoteObjectModified)
             {
                 $mergedIndex->addObject($this->resolveConflict($conflictHandler, $remoteObject, $localObject, $lastLocalObject));
-            }
-            elseif ($localObjectModified && $localObject !== null)
+            } elseif ($localObjectModified && $localObject !== null)
             {
                 $mergedIndex->addObject($localObject);
-            }
-            elseif ($remoteObjectModified && $remoteObject !== null)
+            } elseif ($remoteObjectModified && $remoteObject !== null)
             {
                 $mergedIndex->addObject($remoteObject);
             }
@@ -137,8 +135,7 @@ 
         if ($lastLocalObject)
         {
             return !$lastLocalObject->equals($remoteObject, IndexObject::CMP_IGNORE_BLOBID | IndexObject::CMP_IGNORE_INODE);
-        }
-        else
+        } else
         {
             return $remoteObject !== null;
         }