byKolev / MultilingualExtension

src/Context/MultilingualContext.php

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

Braces +6 added lines, -6 removed lines

@@ -84,11 +84,11 @@ 
         $not_clean_url_language_code = substr($current_url,$base_url_length+3,2);
         if(in_array($clean_url_language_code,$this->languages_iso_codes) && substr($current_url,$base_url_length+2,1) == "/") {
             return $clean_url_language_code;
-        }
-        else if (in_array($not_clean_url_language_code,$this->languages_iso_codes) && substr($current_url,$base_url_length+5,1) == "/"){
+        } else if (in_array($not_clean_url_language_code,$this->languages_iso_codes) && substr($current_url,$base_url_length+5,1) == "/"){
             return $not_clean_url_language_code;
+        } else {
+            return $this->multilingual_parameters['default_language'];
         }
-        else return $this->multilingual_parameters['default_language'];
     }
 
     /**
@@ -103,11 +103,11 @@ 
         if(isset($this->translations[$target][$this->multilingual_parameters['default_language']])){
             $target = $this->translations[$target][$this->languageDetection()];
             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

Braces +3 added lines, -6 removed lines

@@ -70,15 +70,13 @@ 
 
         if(isset($element)) {
             $text = $element->getValue();
-        }
-        else {
+        } else {
             throw new \Exception(sprintf("Element is null"));
         }
 
         if($text === $value) {
             return true;
-        }
-        else {
+        } else {
             throw new \Exception(sprintf('Value of input : "%s" does not match the text "%s"', $text, $value));
         }
     }
@@ -92,8 +90,7 @@ 
                 $this->iShouldSeeInTheSourceOfThePage($text);
                 $isTextFound = true;
                 break;
-            }
-            catch (\Exception $e) {
+            } catch (\Exception $e) {
                 sleep(1);
             }
         }