Completed
Pull Request — master (#15)
by
unknown
09:11 queued 06:06
created
src/Common/Collection/CollectionInterface.php 1 patch
Doc Comments   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 
Please login to merge, or discard this patch.
src/Common/Event/Progress.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -70,7 +70,7 @@
 block discarded – undo
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
      *
Please login to merge, or discard this patch.
src/Delta/Collection/Collection.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -141,7 +141,7 @@
 block discarded – undo
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
      *
Please login to merge, or discard this patch.
src/Service/DomainBus/Migrate/Single/SingleHandler.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@
 block discarded – undo
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
     {
Please login to merge, or discard this patch.
src/Delta/Repository/VersionRepository.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -81,12 +81,12 @@
 block discarded – undo
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);
Please login to merge, or discard this patch.
src/Delta/Comparator/NamespacesAwareComparator.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -62,7 +62,7 @@
 block discarded – undo
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;
Please login to merge, or discard this patch.
src/Delta/DeltaId.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -102,7 +102,7 @@
 block discarded – undo
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
         }
Please login to merge, or discard this patch.
src/Common/Collection/AbstractCollection.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
     /**
Please login to merge, or discard this patch.
src/Migration/Repository/AggregateMigrationRepository.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
             ));
Please login to merge, or discard this patch.