@@ -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 | )); |