Completed
Pull Request — master (#26)
by Greg
01:13
created
src/GherkinParam.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
           elseif (preg_match(self::$regEx['array'], $variable)) {
113 113
             try {
114 114
               $values[] = $this->getValueFromArray($variable);
115
-            } catch(RuntimeException $e) {
115
+            } catch (RuntimeException $e) {
116 116
               if ($this->throwException) throw new GherkinParamException();
117 117
               if ($this->nullable) $values[] = null;
118 118
             }
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
           else {
122 122
             try {
123 123
               $values[] = Fixtures::get($variable);
124
-            } catch(RuntimeException $e) {
124
+            } catch (RuntimeException $e) {
125 125
               if ($this->throwException) throw new GherkinParamException();
126 126
               if ($this->nullable) $values[] = null;
127 127
             }
@@ -136,7 +136,7 @@  discard block
 block discarded – undo
136 136
         // due to the default behavior when `search` and `replace` arrays size mismatch
137 137
         $param = $this->mapParametersToValues($matches, $values, $param);
138 138
 
139
-      } catch(GherkinParamException $e) {
139
+      } catch (GherkinParamException $e) {
140 140
         // only active if throwException setting is true
141 141
         throw new ExtensionException($this, "Incorrect parameter name ${param}, or not initialized");
142 142
       }
@@ -159,7 +159,7 @@  discard block
 block discarded – undo
159 159
   final private function mapParametersToValues(array $matches, array $values, string $param)
160 160
   {
161 161
     //TODO: move count() into separate variable [performance]
162
-    for ($i=0; $i<count($matches); $i++) {
162
+    for ($i = 0; $i < count($matches); $i++) {
163 163
       $search = $matches[$i];
164 164
       if (\is_string($search)) { // if null then skip
165 165
         if (isset($values[$i])) {
@@ -272,7 +272,7 @@  discard block
 block discarded – undo
272 272
         $prop = new ReflectionProperty(get_class($arg), 'table');
273 273
         $prop->setAccessible(true);
274 274
         $table = $prop->getValue($arg);
275
-        foreach($table as $i => $row) {
275
+        foreach ($table as $i => $row) {
276 276
           foreach ($row as $j => $cell) {
277 277
             $val = $this->getValueFromParam($cell);
278 278
             $table[$i][$j] = $val ? $val : null; // issue TableNode does not support `null` values in table
Please login to merge, or discard this patch.