FloKnapp / faulancer

src/Http/Response.php

Doc Comments +1 added lines, -1 removed lines

@@ -94,7 +94,7 @@
     /**
      * Get response body and set headers
      *
-     * @return mixed
+     * @return string
      */
     public function getContent()
     {

src/Form/Validator/AbstractValidator.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
     {
         if (!$this->process($this->field->getValue())) {
 
-            $this->field->addAttribute('class',  'error');
+            $this->field->addAttribute('class', 'error');
             $this->field->setErrorMessages([$this->getMessage()]);
             return false;
 

src/Form/Type/AbstractType.php

Doc Comments +1 added lines, -1 removed lines

@@ -302,7 +302,7 @@
     }
 
     /**
-     * @param $type
+     * @param string $type
      * @return bool
      */
     private function translateType($type)

Unused Use Statements -1 removed lines

@@ -13,7 +13,6 @@
 use Faulancer\Service\SessionManagerService;
 use Faulancer\ServiceLocator\ServiceLocator;
 use Faulancer\Form\Validator\ValidatorChain;
-use Faulancer\ServiceLocator\ServiceLocatorAwareInterface;
 use Faulancer\ServiceLocator\ServiceLocatorInterface;
 use Faulancer\Session\SessionManager;
 

src/Http/XmlResponse.php

Doc Comments +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
     /**
      * @param $dom
      * @param $data
-     * @return bool|\DOMElement
+     * @return \DOMNode
      */
     protected function generateXmlElement(\DOMDocument $dom, $data )
     {

Spacing +14 added lines, -14 removed lines

@@ -17,7 +17,7 @@ 
      *
      * @param array $content
      */
-    public function __construct ($content = [])
+    public function __construct($content = [])
     {
         parent::__construct($content);
         $this->setContent($content);
@@ -31,30 +31,30 @@ 
      * @param $data
      * @return bool|\DOMElement
      */
-    protected function generateXmlElement(\DOMDocument $dom, $data )
+    protected function generateXmlElement(\DOMDocument $dom, $data)
     {
-        if ( empty( $data['name'] ) )
+        if (empty($data['name']))
             return false;
 
         // Create the element
-        $element_value = ( ! empty( $data['value'] ) ) ? $data['value'] : null;
-        $element = $dom->createElement( $data['name'], $element_value );
+        $element_value = (!empty($data['value'])) ? $data['value'] : null;
+        $element = $dom->createElement($data['name'], $element_value);
 
         // Add any attributes
-        if ( ! empty( $data['attributes'] ) && is_array( $data['attributes'] ) ) {
-            foreach ( $data['attributes'] as $attribute_key => $attribute_value ) {
-                $element->setAttribute( $attribute_key, $attribute_value );
+        if (!empty($data['attributes']) && is_array($data['attributes'])) {
+            foreach ($data['attributes'] as $attribute_key => $attribute_value) {
+                $element->setAttribute($attribute_key, $attribute_value);
             }
         }
 
         // Any other items in the data array should be child elements
-        foreach ( $data as $data_key => $child_data ) {
-            if ( ! is_numeric( $data_key ) )
+        foreach ($data as $data_key => $child_data) {
+            if (!is_numeric($data_key))
                 continue;
 
-            $child = $this->generateXmlElement( $dom, $child_data );
-            if ( $child )
-                $element->appendChild( $child );
+            $child = $this->generateXmlElement($dom, $child_data);
+            if ($child)
+                $element->appendChild($child);
         }
 
         return $element;
@@ -74,7 +74,7 @@ 
         $doc->encoding   = 'UTF-8';
         $child = $this->generateXmlElement($doc, $content);
 
-        if ( $child ) {
+        if ($child) {
             $doc->appendChild($child);
         }
 

Braces +9 added lines, -6 removed lines

@@ -33,8 +33,9 @@ 
      */
     protected function generateXmlElement(\DOMDocument $dom, $data )
     {
-        if ( empty( $data['name'] ) )
-            return false;
+        if ( empty( $data['name'] ) ) {
+                    return false;
+        }
 
         // Create the element
         $element_value = ( ! empty( $data['value'] ) ) ? $data['value'] : null;
@@ -49,12 +50,14 @@ 
 
         // Any other items in the data array should be child elements
         foreach ( $data as $data_key => $child_data ) {
-            if ( ! is_numeric( $data_key ) )
-                continue;
+            if ( ! is_numeric( $data_key ) ) {
+                            continue;
+            }
 
             $child = $this->generateXmlElement( $dom, $child_data );
-            if ( $child )
-                $element->appendChild( $child );
+            if ( $child ) {
+                            $element->appendChild( $child );
+            }
         }
 
         return $element;

src/Service/AuthenticatorService.php

Doc Comments +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
 
     /**
      * @param array $roles
-     * @return bool
+     * @return null|boolean
      */
     public function isPermitted(array $roles)
     {

src/Mail/Mailer.php

Spacing +2 added lines, -2 removed lines

@@ -188,7 +188,7 @@ 
         $headers .= 'Content-Type: multipart/mixed; charset=UTF-8; boundary=' . $boundary . $eol;
 
         if (!empty($this->from)) {
-            $headers .= 'From: ' .$this->from . $eol;
+            $headers .= 'From: ' . $this->from . $eol;
         }
 
         if (!empty($this->replyTo)) {
@@ -246,7 +246,7 @@ 
                 }
 
                 $body .= 'Content-Transfer-Encoding: base64' . $eol;
-                $body .= 'X-Attachment-Id: ' . rand(1000,99999) . $eol . $eol;
+                $body .= 'X-Attachment-Id: ' . rand(1000, 99999) . $eol . $eol;
                 $body .= $encodedContent . $eol;
 
             }

src/View/Helper/AssetList.php

Switch Indentation +6 added lines, -6 removed lines

@@ -37,13 +37,13 @@
 
         switch ($type) {
 
-        case 'js':
-            $pattern = '<script src="%s"></script>';
-            break;
+            case 'js':
+                $pattern = '<script src="%s"></script>';
+                break;
 
-        case 'css':
-            $pattern = '<link rel="stylesheet" type="text/css" href="%s">';
-            break;
+            case 'css':
+                $pattern = '<link rel="stylesheet" type="text/css" href="%s">';
+                break;
 
         }
 

src/Controller/ErrorController.php

Spacing +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
             'line'    => $this->_exception->getLine()
         ];
 
-        $trace  = $this->_exception->getTrace();
+        $trace = $this->_exception->getTrace();
 
         if (isset($trace[0]['line']) && $trace[0]['line'] !== $raiser['line']) {
             array_unshift($trace, $raiser);

src/View/ViewController.php

Unused Use Statements -4 removed lines

@@ -2,14 +2,10 @@
 
 namespace Faulancer\View;
 
-use Faulancer\Exception\ClassNotFoundException;
 use Faulancer\Exception\ConstantMissingException;
-use Faulancer\Exception\Exception;
 use Faulancer\Exception\FileNotFoundException;
 use Faulancer\Exception\ViewHelperException;
-use Faulancer\Exception\ViewHelperIncompatibleException;
 use Faulancer\Service\Config;
-use Faulancer\Service\ResponseService;
 use Faulancer\ServiceLocator\ServiceLocator;
 
 /**

Spacing +6 added lines, -6 removed lines

@@ -156,7 +156,7 @@ 
      */
     public function getVariable(string $key)
     {
-        if(isset($this->variable[$key])) {
+        if (isset($this->variable[$key])) {
             return $this->variable[$key];
         }
 
@@ -171,7 +171,7 @@ 
      */
     public function hasVariable(string $key) :bool
     {
-        if(isset($this->variable[$key])) {
+        if (isset($this->variable[$key])) {
             return true;
         }
 
@@ -186,7 +186,7 @@ 
      */
     public function setVariables(array $variables = []) :self
     {
-        foreach($variables AS $key=>$value) {
+        foreach ($variables AS $key=>$value) {
             $this->setVariable($key, $value);
         }
 
@@ -272,7 +272,7 @@ 
      */
     public function __call($name, $arguments)
     {
-        $coreViewHelper   = __NAMESPACE__ . '\Helper\\' . ucfirst($name);
+        $coreViewHelper = __NAMESPACE__ . '\Helper\\' . ucfirst($name);
 
         if (!empty($this->viewHelpers[$coreViewHelper])) {
             return $this->_callUserFuncArray($this->viewHelpers[$coreViewHelper], $arguments);
@@ -325,8 +325,8 @@ 
      */
     public function __destruct()
     {
-        unset( $this->variable );
-        unset( $this->template );
+        unset($this->variable);
+        unset($this->template);
     }
 
 }
\ No newline at end of file