@@ -57,7 +57,7 @@ |
||
57 | 57 | * * Tab/Section::name=Text Of Setting (will ensure that the correct section is navigated to, uses setting name (exact), not ID) |
58 | 58 | * * name=Text Of Setting (will not navigate, uses setting name (exact) not ID) |
59 | 59 | * |
60 | - * @param $identifier |
|
60 | + * @param string $identifier |
|
61 | 61 | * @param $value |
62 | 62 | * @throws \Magium\InvalidInstructionException |
63 | 63 | */ |
@@ -43,6 +43,9 @@ |
||
43 | 43 | return $this->messages; |
44 | 44 | } |
45 | 45 | |
46 | + /** |
|
47 | + * @param string $message |
|
48 | + */ |
|
46 | 49 | public function addMessage($message) |
47 | 50 | { |
48 | 51 | $this->messages[] = $message; |
@@ -27,7 +27,7 @@ |
||
27 | 27 | |
28 | 28 | /** |
29 | 29 | * @param $nodePath |
30 | - * @return WebDriverElement |
|
30 | + * @return null|\Facebook\WebDriver\Remote\RemoteWebElement |
|
31 | 31 | */ |
32 | 32 | |
33 | 33 | public function getNode($nodePath) |
@@ -80,6 +80,9 @@ |
||
80 | 80 | } |
81 | 81 | |
82 | 82 | |
83 | + /** |
|
84 | + * @param string $type |
|
85 | + */ |
|
83 | 86 | public function addFilterType($type) |
84 | 87 | { |
85 | 88 | $type = $this->filterTypeName($type); |
@@ -25,6 +25,10 @@ |
||
25 | 25 | return $this->products; |
26 | 26 | } |
27 | 27 | |
28 | + /** |
|
29 | + * @param integer $count |
|
30 | + * @param string $attribute |
|
31 | + */ |
|
28 | 32 | protected function getByXpath($type, $count, $attribute = null, $getElement = false) |
29 | 33 | { |
30 | 34 | $xpath = $this->getElementXpath($type, $count); |
@@ -57,7 +57,9 @@ |
||
57 | 57 | $attributes = $this->option->getOptionNames(); |
58 | 58 | foreach ($attributes as $attributeName) { |
59 | 59 | $elementOption = $this->option->getOption($attributeName); |
60 | - if (!$elementOption) continue; |
|
60 | + if (!$elementOption) { |
|
61 | + continue; |
|
62 | + } |
|
61 | 63 | // The options are sorted by their entry into the setOption() method. Need to have it ordered by the page order |
62 | 64 | |
63 | 65 | if (!$elementOption instanceof Option) { |
@@ -44,10 +44,14 @@ |
||
44 | 44 | foreach ($this->steps as $step) { |
45 | 45 | if ($step instanceof StepInterface) { |
46 | 46 | $continue = $step->execute(); |
47 | - if (!$continue) return; |
|
47 | + if (!$continue) { |
|
48 | + return; |
|
49 | + } |
|
48 | 50 | |
49 | 51 | $continue = $step->nextAction(); |
50 | - if (!$continue) return; |
|
52 | + if (!$continue) { |
|
53 | + return; |
|
54 | + } |
|
51 | 55 | } |
52 | 56 | } |
53 | 57 | } |
@@ -15,7 +15,9 @@ |
||
15 | 15 | { |
16 | 16 | $xpath = new \DOMXPath($this->document); |
17 | 17 | $xpathQuery = $this->theme->getLayeredNavigationSwatchAppliesXpath($this->title); |
18 | - if (!$xpathQuery) return false; |
|
18 | + if (!$xpathQuery) { |
|
19 | + return false; |
|
20 | + } |
|
19 | 21 | $elements = $xpath->query($xpathQuery); |
20 | 22 | return $elements->length > 0; |
21 | 23 | } |
@@ -31,7 +31,7 @@ |
||
31 | 31 | } |
32 | 32 | |
33 | 33 | /** |
34 | - * @param $label |
|
34 | + * @param string $label |
|
35 | 35 | * @return WebDriverElement |
36 | 36 | */ |
37 | 37 |