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