ciandt-dev / behat-placeholders-extension

src/Ciandt/Behat/PlaceholdersExtension/Exception/MissingSectionException.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
  *
  * @author bwowk
  */
-class MissingSectionException extends RuntimeException{
+class MissingSectionException extends RuntimeException {
     
         
     public function __construct($configKey, $configPath, $section) {

src/Ciandt/Behat/PlaceholdersExtension/Exception/UndefinedPlaceholderException.php

Spacing +1 added lines, -1 removed lines

@@ -8,6 +8,6 @@
  *
  * @author bwowk
  */
-class UndefinedPlaceholderException extends RuntimeException{
+class UndefinedPlaceholderException extends RuntimeException {
     //put your code here
 }

src/Ciandt/Behat/PlaceholdersExtension/Exception/RedundantConfigTagException.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
  *
  * @author bwowk
  */
-class RedundantConfigTagException extends RuntimeException{
+class RedundantConfigTagException extends RuntimeException {
     public function __construct($tags) {
         $message = "Multiple config tags found. Should have only one: " . implode(', ', $tags);
         parent::__construct($message);

src/Ciandt/Behat/PlaceholdersExtension/Transformer/PlaceholdersTransformer.php

Indentation -1 removed lines

@@ -10,7 +10,6 @@
 
 /**
  * Transforms a single argument value.
-
  */
 class PlaceholdersTransformer implements ArgumentTransformer
 {

Spacing +2 added lines, -2 removed lines

@@ -27,8 +27,8 @@
     }
 
     public function transformArgument(DefinitionCall $definitionCall, $argumentIndex, $argumentValue) {
-        if (RuntimePlaceholdersTransformation::supportsArgument($argumentValue)){
-            return RuntimePlaceholdersTransformation::transformArgument($argumentValue,$this->repository);
+        if (RuntimePlaceholdersTransformation::supportsArgument($argumentValue)) {
+            return RuntimePlaceholdersTransformation::transformArgument($argumentValue, $this->repository);
         }
         return $argumentValue;
     }

src/Ciandt/Behat/PlaceholdersExtension/Utils/PlaceholderUtils.php

Spacing +9 added lines, -9 removed lines

@@ -25,9 +25,9 @@ 
         self::$configKeys = $configKeys;
     }
     
-    public static  function getSectionKey($tag){
+    public static  function getSectionKey($tag) {
         $tagParts = explode(':', $tag);
-        if (count($tagParts) == 2){
+        if (count($tagParts) == 2) {
             return $tagParts[1];
         } else {
             return 'default';
@@ -35,8 +35,8 @@ 
         
     }
     
-    public static function getConfigTag($tags){
-        $configTags = self::filterConfigTags($tags,false);
+    public static function getConfigTag($tags) {
+        $configTags = self::filterConfigTags($tags, false);
         if (empty($configTags)) {
             return false;
         }
@@ -47,25 +47,25 @@ 
     }
 
 
-    public static function getConfigKey($tag){
+    public static function getConfigKey($tag) {
         return explode(':', $tag)[0];
     }
     
     
     public static function filterVariantTags($tags, $exclude)
     {   
-        return array_filter($tags, function ($tag) use ($exclude) {
+        return array_filter($tags, function($tag) use ($exclude) {
             return (in_array($tag, self::$variantTags) xor $exclude);
         });
     }
     
-    public static function getVariant($tags){
+    public static function getVariant($tags) {
         $variantTags = self::filterVariantTags($tags, false);
         if (count($variantTags) > 1) {
             throw new \RuntimeException("Scenario should only have one variant tag."
                     . " Multiple found: " . implode(', ', $variantTags));
         }
-        if (empty($variantTags)){
+        if (empty($variantTags)) {
             return 'default';
         }
         return end($variantTags);
@@ -74,7 +74,7 @@ 
 
     public static function filterConfigTags($tags, $exclude)
     {
-        return array_filter($tags, function ($tag) use ($exclude){
+        return array_filter($tags, function($tag) use ($exclude){
             return (in_array(self::getConfigKey($tag), self::$configKeys) xor $exclude);
         });
     }

src/Ciandt/Behat/PlaceholdersExtension/Subscriber/BeforeScenarioSubscriber.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
             );
     }
     
-    public function onBeforeScenarioTested($beforeScenario){
+    public function onBeforeScenarioTested($beforeScenario) {
         $featureTags = $beforeScenario->getFeature()->getTags();
         $scenarioTags = $beforeScenario->getNode()->getTags();
         $this->scenarioTags = array_merge($featureTags, $scenarioTags);

src/Ciandt/Behat/PlaceholdersExtension/Gherkin/ScenarioBranchingFileLoader.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
         $filename = $this->findRelativePath($path);
         $content = file_get_contents($path);
         $feature = $this->parser->parse($content, $filename);
-        return null === $feature? $feature : $this->branchFeatureVariants($feature);
+        return null === $feature ? $feature : $this->branchFeatureVariants($feature);
     }
     
     /**

src/Ciandt/Behat/PlaceholdersExtension/Config/PlaceholdersRepository.php

Doc Comments -1 removed lines

@@ -47,7 +47,6 @@
     /**
      * reads the YAML placeholder definitions and returns an associative array
      *
-     * @param type $config_files
      * @todo use %paths.base% value
      * @return array
      */

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 
 use Symfony\Component\Yaml\Yaml;
 use Ciandt\Behat\PlaceholdersExtension\Utils\PlaceholderUtils;
-use Ciandt\Behat\PlaceholdersExtension\PlaceholderContainer\PlaceholderContainer;
 use Ciandt\Behat\PlaceholdersExtension\Exception\MissingSectionException;
 use Ciandt\Behat\PlaceholdersExtension\Exception\UndefinedPlaceholderException;
 use Exception;

Indentation +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
 
     private function getFilePath($key)
     {
-         if (key_exists($key, $this->configs)) {
+          if (key_exists($key, $this->configs)) {
             return $this->configs[$key]['path'];
         }
         return null;

Spacing +4 added lines, -4 removed lines

@@ -87,14 +87,14 @@ 
         $this->environment = $environment;
     }
     
-    private function getScenarioTags(){
+    private function getScenarioTags() {
         return $this->beforeScenarioSubscriber->getScenarioTags();
     }
 
     private function replaceRecursively($placeholder, $replaced = array())
     {
         // if the current placeholder was already replaced before, this is a cyclic dependecy
-        if (in_array($placeholder, $replaced)){
+        if (in_array($placeholder, $replaced)) {
             $tree = implode('>', $replaced);
             throw new Exception("Cyclic placeholder dependecy detected. Trying to replace $placeholder again when already replaced: $tree");
         }
@@ -116,13 +116,13 @@ 
 
 
         // Then look in the placeholder files
-        if ($replacement === null && $configTag !== false ) {
+        if ($replacement === null && $configTag !== false) {
             $section = PlaceholderUtils::getSectionKey($configTag);
             $placeholders = $this->getSectionPlaceholders($configKey, $section);
             $replacement = $this->recursiveReplacementSearch($keys, $placeholders);
         }
 
-        if ($replacement === null && $configTag !== false){
+        if ($replacement === null && $configTag !== false) {
             $configPath = $this->getFilePath($configKey);
             $treePosition = "$configPath>$section>placeholders";
             throw new UndefinedPlaceholderException("No $placeholder replacement was defined on runtime or on $treePosition>$placeholder for variant $variant and environment $environment");

src/Ciandt/Behat/PlaceholdersExtension/Transformation/RuntimePlaceholdersTransformation.php

Doc Comments +3 added lines

@@ -34,6 +34,9 @@
         return $string;
     }
 
+    /**
+     * @param PlaceholdersRepository $repository
+     */
     public static function transformArgument($argument, $repository) {
         if (is_string($argument)) {
             return self::replaceStringPlaceholders($argument, $repository);

Indentation +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 /**
  *
  */
- class RuntimePlaceholdersTransformation extends RuntimeCallee implements PlaceholdersTransformation {
+  class RuntimePlaceholdersTransformation extends RuntimeCallee implements PlaceholdersTransformation {
 
     public function __toString() {
         return 'UserDefinedPlaceholderTransform';

Spacing +2 added lines, -2 removed lines

@@ -41,7 +41,7 @@ 
 
         if ($argument instanceof PyStringNode) {
             $strings = $argument->getStrings();
-            foreach ($strings as &$string){
+            foreach ($strings as &$string) {
                 self::replaceStringPlaceholders($string, $repository);
             }
             return new PyStringNode($strings, $argument->getLine());
@@ -54,7 +54,7 @@ 
         }
     }
     
-    private static function replaceTablePlaceholders(&$item, $key, $repository){
+    private static function replaceTablePlaceholders(&$item, $key, $repository) {
         $item = self::replaceStringPlaceholders($item, $repository);
     }
     

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 
 namespace Ciandt\Behat\PlaceholdersExtension\Transformation;
 
-use Behat\Behat\Definition\Call\DefinitionCall;
 use Behat\Gherkin\Node\PyStringNode;
 use Behat\Gherkin\Node\TableNode;
 use Behat\Testwork\Call\RuntimeCallee;