siriusphp / input

src/Element/Factory.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 
 use Sirius\Input\Element\Input;
 use Sirius\Input\Element;
-use Sirius\Input\Specs;
 
 class Factory
 {

Spacing +4 added lines, -4 removed lines

@@ -30,13 +30,13 @@ 
 
             return $this;
         }
-        if ( ! is_string($classOrClosure)) {
+        if (!is_string($classOrClosure)) {
             throw new \RuntimeException('Input type must be a class or a closure');
         }
-        if ( ! class_exists($classOrClosure)) {
+        if (!class_exists($classOrClosure)) {
             throw new \RuntimeException(sprintf('Class %s does not exist', $classOrClosure));
         }
-        if ( ! is_subclass_of($classOrClosure, '\Sirius\Input\Element')) {
+        if (!is_subclass_of($classOrClosure, '\Sirius\Input\Element')) {
             throw new \RuntimeException(
                 sprintf('Class %s must extend the \Sirius\Input\Element class', $classOrClosure)
             );
@@ -70,7 +70,7 @@ 
             $element = new $class($name, $options);
         }
 
-        if ( ! $element instanceof Element) {
+        if (!$element instanceof Element) {
             throw new \RuntimeException('Cannot create a valid element based on the data provided');
         }
 

src/Element/Input/MultiSelect.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 namespace Sirius\Input\Element\Input;
 
 use Sirius\Input\Specs;
-use Sirius\Filtration\Filter\Callback;
 
 class MultiSelect extends Select
 {

Spacing +2 added lines, -2 removed lines

@@ -27,10 +27,10 @@
      */
     function filterValue($value, $valueIdentifier = null)
     {
-        if ( ! $value) {
+        if (!$value) {
             return null;
         }
-        if ( ! is_array($value)) {
+        if (!is_array($value)) {
             $value = (array) $value;
         }
         $allowedValues = array_keys($this->getOptions());

src/InputFilter.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 
 use Sirius\Filtration\Filtrator;
 use Sirius\Filtration\FiltratorInterface;
-use Sirius\Input\Element\AbstractElement;
 use Sirius\Input\Element\Factory as ElementFactory;
 use Sirius\Input\Traits\HasChildrenTrait;
 use Sirius\Input\Traits\HasAttributesTrait;

Spacing +11 added lines, -11 removed lines

@@ -77,17 +77,17 @@ 
         ValidatorInterface $validator = null,
         FiltratorInterface $filtrator = null
     ) {
-        if ( ! $elementFactory) {
+        if (!$elementFactory) {
             $elementFactory = new ElementFactory();
         }
         $this->elementFactory = $elementFactory;
 
-        if ( ! $validator) {
+        if (!$validator) {
             $validator = new Validator();
         }
         $this->validator = $validator;
 
-        if ( ! $filtrator) {
+        if (!$filtrator) {
             $filtrator = new Filtrator();
         }
         $this->filtrator = $filtrator;
@@ -150,11 +150,11 @@ 
      */
     function prepare($force = false)
     {
-        if ($this->isPrepared && ! $force) {
+        if ($this->isPrepared && !$force) {
             return $this;
         }
         $this->init();
-        if ( ! $this->isInitialized()) {
+        if (!$this->isInitialized()) {
             throw new \LogicException('Input was not properly initialized');
         }
 
@@ -197,12 +197,12 @@ 
     protected function prepareFiltration()
     {
         $filters = $this->getFilters();
-        if ( ! $filters || ! is_array($filters)) {
+        if (!$filters || !is_array($filters)) {
             return;
         }
         $filtrator = $this->getFiltrator();
         foreach ($filters as $filter) {
-            $params = is_array($filter) ? $filter : array( $filter );
+            $params = is_array($filter) ? $filter : array($filter);
             if (isset($params[0])) {
                 $filtrator->add(Filtrator::SELECTOR_ROOT, $params[0], @$params[1], @$params[2], @$params[3]);
             }
@@ -247,7 +247,7 @@ 
      */
     function getUploadHandlers()
     {
-        if ( ! $this->uploadHandlers) {
+        if (!$this->uploadHandlers) {
             $this->uploadHandlers = new UploadHandlerAggregate;
         }
 
@@ -282,7 +282,7 @@ 
     function populate($values = array())
     {
         $this->prepare();
-        if ( ! $this->isPrepared()) {
+        if (!$this->isPrepared()) {
             throw new \LogicException('Input was not prepared and cannot receive data');
         }
 
@@ -331,7 +331,7 @@ 
      */
     function isValid($skipDataProcessing = false)
     {
-        if ( ! $skipDataProcessing) {
+        if (!$skipDataProcessing) {
             $this->values = $this->getFiltrator()->filter($this->rawValues);
         }
 
@@ -339,7 +339,7 @@ 
         $validator = $this->getValidator();
         $validator->validate($this->values);
 
-        if ( ! $skipDataProcessing) {
+        if (!$skipDataProcessing) {
             $this->processUploads();
         }
 

src/AttributesContainer.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 
     function removeClass($className)
     {
-        if ( ! $this->hasClass($className)) {
+        if (!$this->hasClass($className)) {
             return;
         }
         $classes = preg_replace('/\b' . $className . '\b/i', '', $this->get('class'));

src/DataContainer.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
 
             return;
         }
-        if ( ! is_string($nameOrArray)) {
+        if (!is_string($nameOrArray)) {
             throw new \InvalidArgumentException('Only strings or arrays are accepted as first argument of the set() method of the DataContainer class');
         }
         if ($value !== null) {

src/Element/Input/File.php

Spacing +2 added lines, -2 removed lines

@@ -43,8 +43,8 @@
         );
         if (is_array($this->getUploadRules())) {
             foreach ($this->getUploadRules() as $rule) {
-                if ( ! is_array($rule)) {
-                    $rule = array( $rule );
+                if (!is_array($rule)) {
+                    $rule = array($rule);
                 }
                 $name    = $rule[0];
                 $options = isset($rule[1]) ? $rule[1] : null;

src/Element/Input/Select.php

Spacing +2 added lines, -2 removed lines

@@ -71,7 +71,7 @@ 
             $this->getName(),
             'callback',
             array(
-                Callback::OPTION_CALLBACK => array( $this, 'filterValue' )
+                Callback::OPTION_CALLBACK => array($this, 'filterValue')
             )
         );
     }
@@ -86,7 +86,7 @@ 
      */
     function filterValue($value, $valueIdentifier = null)
     {
-        if ( ! $value) {
+        if (!$value) {
             return null;
         }
         $allowedValues = array_keys($this->getOptions());

src/Traits/HasAttributesTrait.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 
     protected function ensureAttributes()
     {
-        if ( ! isset($this[Specs::ATTRIBUTES])) {
+        if (!isset($this[Specs::ATTRIBUTES])) {
             $this[Specs::ATTRIBUTES] = new AttributesContainer();
         }
     }

src/Traits/HasChildrenTrait.php

Spacing +7 added lines, -7 removed lines

@@ -61,8 +61,8 @@ 
             throw new \RuntimeException(sprintf('Variable $nameorElement must be a string or an instance of the Element class'));
         }
         // add the index for sorting
-        if ( ! isset($element['__index'])) {
-            $element['__index'] = ($this->elementsIndex --);
+        if (!isset($element['__index'])) {
+            $element['__index'] = ($this->elementsIndex--);
         }
         $this->elements[$name] = $element;
 
@@ -135,20 +135,20 @@ 
     protected function childComparator($childA, $childB)
     {
         if ($childA->getPosition() < $childB->getPosition()) {
-            return - 1;
+            return -1;
         }
         if ($childA->getPosition() > $childB->getPosition()) {
             return 1;
         }
         if ($childA->get('__index') > $childB->get('__index')) {
-            return - 1;
+            return -1;
         }
         if ($childA->get('__index') < $childB->get('__index')) {
             return 1;
         }
 
         // if the priority is the same, childB is first
-        return - 1;
+        return -1;
     }
 
     /**
@@ -159,7 +159,7 @@ 
     function getElements()
     {
         // first sort the children so they are retrieved by priority
-        uasort($this->elements, array( $this, 'childComparator' ));
+        uasort($this->elements, array($this, 'childComparator'));
 
         return $this->elements;
     }
@@ -172,7 +172,7 @@ 
     {
         foreach ($this->elements as $element) {
             $group = $element->getGroup();
-            if ($group && ! isset($this->elements[$group])) {
+            if ($group && !isset($this->elements[$group])) {
                 $element->setGroup(null);
             }
         }