byKolev / MultilingualExtension

src/Context/MultilingualContext.php

Doc Comments +1 added lines

@@ -72,6 +72,7 @@
 
     /**
      * This function localizes the field based on Drupal standards. English language is used as a base.
+     * @return string
      */
 
     public function localizeField($field) {

Unused Use Statements -6 removed lines

@@ -4,13 +4,7 @@
 
 use Behat\Behat\Context\Context;
 use Symfony\Component\Yaml\Yaml;
-use Drupal\DrupalExtension\Context\DrupalContext;
-use Behat\MinkExtension;
-use Behat\Behat\Context\TranslatableContext;
 use Behat\Mink\Element\Element;
-use Behat\Gherkin\Node\TableNode;
-use Behat\Behat\Hook\Scope\BeforeScenarioScope;
-use Behat\MinkExtension\Context\RawMinkContext;
 
 /**
  * This is the file for Multilingual context for Drupal. The context is working based on specifications

Spacing +7 added lines, -7 removed lines

@@ -46,7 +46,7 @@ 
     }
 
     public function initializeMultilanguage() {
-        if(isset($this->multilingual_parameters['language'])&&isset($this->multilingual_parameters['translations'])) {
+        if (isset($this->multilingual_parameters['language'])&&isset($this->multilingual_parameters['translations'])) {
             $this->parseTranslationFile();
         }
     }
@@ -60,14 +60,14 @@ 
 
     public function localizeTarget($target) {
         $translations = $this->multilingual_parameters['translations'];
-        if(isset($this->translations[$target][$this->multilingual_parameters['language']])){
+        if (isset($this->translations[$target][$this->multilingual_parameters['language']])) {
             $target = $this->translations[$target][$this->multilingual_parameters['language']];
             return $target;
         }
         elseif (isset($this->translations[$target])) {
             return $target;
         }
-        else throw new Exception ("The text '$target'' is not defined in '$translations' translation file.");
+        else throw new Exception("The text '$target'' is not defined in '$translations' translation file.");
     }
 
     /**
@@ -77,7 +77,7 @@ 
     public function localizeField($field) {
         $re = "/(?:[-])(en)(?:[-])/";
         $language = "-".$this->multilingual_parameters['language']."-";
-        $field = preg_replace($re, $language,$field);
+        $field = preg_replace($re, $language, $field);
         return $field;
     }
 
@@ -166,7 +166,7 @@ 
      *
      * @When /^I wait for localized text "([^"]*)" to appear with max time "([^"]+)"(?: seconds)?$/
      */
-    public function iWaitForLocalizedTextToAppearWithMaxTime($target, $maxExecutionTime){
+    public function iWaitForLocalizedTextToAppearWithMaxTime($target, $maxExecutionTime) {
         $target = $this->localizeTarget($target);
         $this->FeatureContext->iWaitForTextToAppearWithMaxTime($target, $maxExecutionTime);
     }
@@ -190,7 +190,7 @@ 
      * @Given I click localized :link in the :rowText row
      * @Then I (should )see the localized :link in the :rowText row
      */
-    public function assertLocalizedClickInTableRow($link, $rowText){
+    public function assertLocalizedClickInTableRow($link, $rowText) {
         $link = $this->localizeTarget($link);
         $page = $this->getSession()->getPage();
         if ($link_element = $this->getTableRow($page, $rowText)->findLink($link)) {
@@ -219,7 +219,7 @@ 
      *
      * @When /^I click on the localized text "([^"]*)" in the "(?P<region>[^"]*)"(?:| region)$/
      */
-    public function iClickOnTheLocalizedTextInRegion($text, $region){
+    public function iClickOnTheLocalizedTextInRegion($text, $region) {
         $text = $this->localizeTarget($text);
         $this->FeatureContext->iClickOnTheTextInRegion($text, $region);
     }

Braces +3 added lines, -3 removed lines

@@ -63,11 +63,11 @@
         if(isset($this->translations[$target][$this->multilingual_parameters['language']])){
             $target = $this->translations[$target][$this->multilingual_parameters['language']];
             return $target;
-        }
-        elseif (isset($this->translations[$target])) {
+        } elseif (isset($this->translations[$target])) {
             return $target;
+        } else {
+            throw new Exception ("The text '$target'' is not defined in '$translations' translation file.");
         }
-        else throw new Exception ("The text '$target'' is not defined in '$translations' translation file.");
     }
 
     /**

src/Context/RawMultilingualContext.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
         $session = $this->getSession();
         $element = $session->getPage()->find(
             'xpath',
-            $session->getSelectorsHandler()->selectorToXpath('xpath', '//*[contains(text(),"' . $text . '")]'));
+            $session->getSelectorsHandler()->selectorToXpath('xpath', '//*[contains(text(),"'.$text.'")]'));
 
         if (null === $element) {
             throw new \InvalidArgumentException(sprintf('Cannot find text: "%s"', $text));