Merge pull request #4 from refactorstudio/improve-code-quality

Improve code quality (scrutinizer)

Author: Refactor Studio

src/PhpArrayToXml.php

@@ -246,7 +246,7 @@ public function getTransformTags()
     /**
      * Set the numeric tag suffix
      *
-     * @param null $value
+     * @param null|string $value
      * @return PhpArrayToXml
      */
     public function setNumericTagSuffix($value = null)
@@ -335,7 +335,7 @@ public function getCastNullValue()
     /**
      * Validate if a given value has a proper tag starting character to be used in XML
      *
-     * @param null $value
+     * @param null|string $value
      * @return bool
      */
     public static function hasValidXmlTagStartingChar($value = null)
@@ -349,7 +349,7 @@ public static function hasValidXmlTagStartingChar($value = null)
     /**
      * Validate if a given value is a valid tag character
      *
-     * @param null $value
+     * @param null|string $value
      * @return bool
      */
     public static function isValidXmlTagChar($value = null)
@@ -363,7 +363,7 @@ public static function isValidXmlTagChar($value = null)
     /**
      * Validate if a given value is a proper tag name to be used in XML
      *
-     * @param null $value
+     * @param null|string $value
      * @return bool
      */
     public static function isValidXmlTag($value = null)
@@ -552,7 +552,7 @@ protected function isNumericKey($value)
      * Creates an element for DOMDocument
      *
      * @param $name
-     * @param null $value
+     * @param null|string $value
      * @param bool $cdata
      * @param array $attributes
      * @return DOMElement
@@ -584,7 +584,7 @@ protected function createElement($name, $value = null, $cdata = false, $attribut
     /**
      * Creates a valid tag name
      *
-     * @param null $name
+     * @param null|string $name
      * @return string
      */
     protected function createValidTagName($name = null)
@@ -649,7 +649,7 @@ protected function replaceInvalidTagChars($value)
     /**
      * Creates a valid root name
      *
-     * @param null $name
+     * @param null|string $name
      * @return string
      */
     protected function createValidRootName($name = null)
@@ -666,19 +666,17 @@ protected function createValidRootName($name = null)
     /**
      * Transforms a tag name (only when specified)
      *
-     * @param null $name
+     * @param null|string $name
      * @return null|string
      */
     protected function transformTagName($name = null)
     {
         switch($this->getTransformTags()) {
             case self::LOWERCASE: {
                 return strtolower($name);
-                break;
             }
             case self::UPPERCASE: {
                 return strtoupper($name);
-                break;
             }
             default: {
                 return $name;