@@ -83,7 +83,7 @@ discard block |
||
| 83 | 83 | * |
| 84 | 84 | * @param string|integer $key The kex/index of the element to remove. |
| 85 | 85 | * |
| 86 | - * @return mixed The removed element or NULL, if the collection did not contain the element. |
|
| 86 | + * @return null|DeltaInterface The removed element or NULL, if the collection did not contain the element. |
|
| 87 | 87 | */ |
| 88 | 88 | public function remove($key); |
| 89 | 89 | |
@@ -99,7 +99,7 @@ discard block |
||
| 99 | 99 | /** |
| 100 | 100 | * Gets all keys/indices of the collection. |
| 101 | 101 | * |
| 102 | - * @return array The keys/indices of the collection, in the order of the corresponding |
|
| 102 | + * @return integer[] The keys/indices of the collection, in the order of the corresponding |
|
| 103 | 103 | * elements in the collection. |
| 104 | 104 | */ |
| 105 | 105 | public function getKeys(); |
@@ -188,7 +188,7 @@ discard block |
||
| 188 | 188 | * |
| 189 | 189 | * @param Closure $func |
| 190 | 190 | * |
| 191 | - * @return array |
|
| 191 | + * @return \Baleen\Migrations\Delta\DeltaId[] |
|
| 192 | 192 | */ |
| 193 | 193 | public function map(Closure $func); |
| 194 | 194 | |
@@ -214,7 +214,7 @@ discard block |
||
| 214 | 214 | * @param int $offset The offset to start from. |
| 215 | 215 | * @param int|null $length The maximum number of elements to return, or null for no limit. |
| 216 | 216 | * |
| 217 | - * @return array |
|
| 217 | + * @return DeltaInterface[] |
|
| 218 | 218 | */ |
| 219 | 219 | public function slice($offset, $length = null); |
| 220 | 220 | |
@@ -70,7 +70,7 @@ |
||
| 70 | 70 | /** |
| 71 | 71 | * Update the current progress |
| 72 | 72 | * |
| 73 | - * @param $newProgress |
|
| 73 | + * @param integer $newProgress |
|
| 74 | 74 | * |
| 75 | 75 | * @return void |
| 76 | 76 | * |
@@ -141,7 +141,7 @@ |
||
| 141 | 141 | /** |
| 142 | 142 | * Add a version to the collection |
| 143 | 143 | * |
| 144 | - * @param mixed $version |
|
| 144 | + * @param DeltaInterface $version |
|
| 145 | 145 | * |
| 146 | 146 | * @return bool |
| 147 | 147 | * |
@@ -49,7 +49,7 @@ |
||
| 49 | 49 | * |
| 50 | 50 | * @param SingleCommand $command |
| 51 | 51 | * |
| 52 | - * @return DeltaInterface |
|
| 52 | + * @return boolean |
|
| 53 | 53 | */ |
| 54 | 54 | public function handle(SingleCommand $command) |
| 55 | 55 | { |
@@ -81,12 +81,12 @@ |
||
| 81 | 81 | return true; // nothing to do - exit early |
| 82 | 82 | } |
| 83 | 83 | |
| 84 | - $mapToIds = function(DeltaInterface $v) { |
|
| 84 | + $mapToIds = function (DeltaInterface $v) { |
|
| 85 | 85 | return $v->getId(); |
| 86 | 86 | }; |
| 87 | 87 | /** @var CollectionInterface $migrated */ |
| 88 | 88 | /** @var CollectionInterface $notMigrated */ |
| 89 | - list($migrated, $notMigrated) = $versions->partition(function($i, DeltaInterface $v) { |
|
| 89 | + list($migrated, $notMigrated) = $versions->partition(function ($i, DeltaInterface $v) { |
|
| 90 | 90 | return $v->isMigrated(); |
| 91 | 91 | }); |
| 92 | 92 | $migratedIds = $migrated->map($mapToIds); |
@@ -62,7 +62,7 @@ |
||
| 62 | 62 | } |
| 63 | 63 | // normalize namespaces |
| 64 | 64 | foreach ($namespaces as &$namespace) { |
| 65 | - $namespace = trim($namespace, '\\') . '\\'; |
|
| 65 | + $namespace = trim($namespace, '\\').'\\'; |
|
| 66 | 66 | } |
| 67 | 67 | krsort($namespaces); // we search from highest to lowest priority |
| 68 | 68 | $this->namespaces = $namespaces; |
@@ -102,7 +102,7 @@ |
||
| 102 | 102 | $str = (string) $value; |
| 103 | 103 | if (empty($str)) { |
| 104 | 104 | throw new InvalidArgumentException( |
| 105 | - 'Refusing to create a DeltaId from an empty string or any other type of value that casts into an ' . |
|
| 105 | + 'Refusing to create a DeltaId from an empty string or any other type of value that casts into an '. |
|
| 106 | 106 | 'empty string.' |
| 107 | 107 | ); |
| 108 | 108 | } |
@@ -127,7 +127,7 @@ discard block |
||
| 127 | 127 | public function remove($key) |
| 128 | 128 | { |
| 129 | 129 | $key = (string) $key; |
| 130 | - if ( ! isset($this->elements[$key]) && ! array_key_exists($key, $this->elements)) { |
|
| 130 | + if (!isset($this->elements[$key]) && !array_key_exists($key, $this->elements)) { |
|
| 131 | 131 | return null; |
| 132 | 132 | } |
| 133 | 133 | |
@@ -270,7 +270,7 @@ discard block |
||
| 270 | 270 | final public function forAll(Closure $p) |
| 271 | 271 | { |
| 272 | 272 | foreach ($this->elements as $key => $element) { |
| 273 | - if ( ! $p($key, $element)) { |
|
| 273 | + if (!$p($key, $element)) { |
|
| 274 | 274 | return false; |
| 275 | 275 | } |
| 276 | 276 | } |
@@ -303,7 +303,7 @@ discard block |
||
| 303 | 303 | */ |
| 304 | 304 | public function __toString() |
| 305 | 305 | { |
| 306 | - return __CLASS__ . '@' . spl_object_hash($this); |
|
| 306 | + return __CLASS__.'@'.spl_object_hash($this); |
|
| 307 | 307 | } |
| 308 | 308 | |
| 309 | 309 | /** |
@@ -61,7 +61,7 @@ discard block |
||
| 61 | 61 | { |
| 62 | 62 | if (!is_array($repositories) && (!is_object($repositories) || !$repositories instanceof \Traversable)) { |
| 63 | 63 | throw new InvalidArgumentException(sprintf( |
| 64 | - 'Invalid argument provided for $repositories, expecting either an array or Traversable object, but' . |
|
| 64 | + 'Invalid argument provided for $repositories, expecting either an array or Traversable object, but'. |
|
| 65 | 65 | ' "%s" given', |
| 66 | 66 | is_object($repositories) ? get_class($repositories) : gettype($repositories) |
| 67 | 67 | )); |
@@ -91,7 +91,7 @@ discard block |
||
| 91 | 91 | { |
| 92 | 92 | if (!is_array($repositories) && (!is_object($repositories) || !$repositories instanceof \Traversable)) { |
| 93 | 93 | throw new InvalidArgumentException(sprintf( |
| 94 | - 'Invalid argument provided for $repositories, expecting either an array or Traversable object, but' . |
|
| 94 | + 'Invalid argument provided for $repositories, expecting either an array or Traversable object, but'. |
|
| 95 | 95 | ' "%s" given', |
| 96 | 96 | is_object($repositories) ? get_class($repositories) : gettype($repositories) |
| 97 | 97 | )); |