View GitHub Repository codeception-gherkin-param

Schedule Inspection
Subscribe
Completed
Pull Request — master (#26)
by Greg
06:41
created
Status
Category
Toggle Patches
src/GherkinParam.php 1 patch
Braces   +15 added lines, -5 removed lines patch added patch discarded remove patch 
@@ -113,8 +113,12 @@  discard block
 
 block discarded – undo
113 113 
             try {
114 114 
               $values[] = $this->getValueFromArray($variable);
115 115 
             } catch (RuntimeException $e) {
116 
-              if ($this->throwException) throw new GherkinParamException();
117 
-              if ($this->nullable) $values[] = null;
116 
+              if ($this->throwException) {
117 
+                throw new GherkinParamException();
118 
+              }
119 
+              if ($this->nullable) {
120 
+                $values[] = null;
121 
+              }
118 122 
             }
119 123 
           }
120 124 
           // normal case
 
@@ -122,13 +126,19 @@  discard block
 
 block discarded – undo
122 126 
             try {
123 127 
               $values[] = Fixtures::get($variable);
124 128 
             } catch (RuntimeException $e) {
125 
-              if ($this->throwException) throw new GherkinParamException();
126 
-              if ($this->nullable) $values[] = null;
129 
+              if ($this->throwException) {
130 
+                throw new GherkinParamException();
131 
+              }
132 
+              if ($this->nullable) {
133 
+                $values[] = null;
134 
+              }
127 135 
             }
128 136 
           }
129 137 
           // if machting value return is not found (null)
130 138 
           if (is_null(end($values))) {
131 
-            if ($this->throwException) throw new GherkinParamException();
139 
+            if ($this->throwException) {
140 
+              throw new GherkinParamException();
141 
+            }
132 142 
           }
133 143 
         }
134 144
Please login to merge, or discard this patch.