delboy1978uk / form

src/AbstractForm.php

Doc Comments +2 added lines, -2 removed lines

@@ -175,7 +175,7 @@ 
     }

 

     /**

-     * @param array $dynamicForms

+     * @param FormInterface[] $dynamicForms

      * @param array $data

      */

     private function populateDynamicForms(array $dynamicForms, array $data): void

@@ -348,7 +348,7 @@ 
     }

 

     /**

-     * @param boolean $displayError

+     * @param boolean $displayErrors

      */

     public function setDisplayErrors(bool $displayErrors): void

     {

Unused Use Statements +6 added lines, -6 removed lines

@@ -2,12 +2,12 @@
 

 namespace Del\Form;

 

-use Del\Form\Collection\FieldCollection;

-use Del\Form\Field\CheckBox;

-use Del\Form\Field\FieldInterface;

-use Del\Form\Field\FileUpload;

-use Del\Form\Renderer\FormRenderer;

-use Del\Form\Renderer\FormRendererInterface;

+use Del\Form\Collection\FieldCollection;
+use Del\Form\Field\CheckBox;
+use Del\Form\Field\FieldInterface;
+use Del\Form\Field\FileUpload;
+use Del\Form\Renderer\FormRenderer;
+use Del\Form\Renderer\FormRendererInterface;
 use Del\Form\Traits\HasAttributesTrait;

 

 abstract class AbstractForm implements FormInterface

src/Field/FieldAbstract.php

Doc Comments +5 added lines, -1 removed lines

@@ -54,6 +54,10 @@ 
 

     abstract public function init();

 

+    /**
+     * @param string $name
+     * @param string $value
+     */
     public function __construct($name, $value = null)

     {

         $this->required = false;

@@ -158,7 +162,7 @@ 
     }

 

     /**

-     * @param FilterInterface $transformer

+     * @param TransformerInterface $transformer

      */

     public function setTransformer(TransformerInterface $transformer): void

     {

Unused Use Statements +9 added lines, -11 removed lines

@@ -2,18 +2,16 @@
 

 namespace Del\Form\Field;

 

-use Del\Form\Collection\FilterCollection;

-use Del\Form\Collection\ValidatorCollection;

-use Del\Form\Filter\Adapter\FilterAdapterZf;

-use Del\Form\Filter\FilterInterface;

-use Del\Form\FormInterface;

-use Del\Form\Renderer\Field\FieldRendererInterface;

-use Del\Form\Renderer\Field\TextRender;

-use Del\Form\Traits\HasAttributesTrait;

-use Del\Form\Validator\NotEmpty;

-use Del\Form\Validator\ValidatorInterface;

+use Del\Form\Collection\FilterCollection;
+use Del\Form\Collection\ValidatorCollection;
+use Del\Form\Filter\FilterInterface;
+use Del\Form\FormInterface;
+use Del\Form\Renderer\Field\FieldRendererInterface;
+use Del\Form\Renderer\Field\TextRender;
+use Del\Form\Traits\HasAttributesTrait;
+use Del\Form\Validator\NotEmpty;
+use Del\Form\Validator\ValidatorInterface;
 use Exception;

-use Laminas\Filter\ToNull;

 

 abstract class FieldAbstract implements FieldInterface

 {

src/Field/FieldInterface.php

Doc Comments +11 added lines, -1 removed lines

@@ -15,6 +15,7 @@ 
 {
     /**
      * @param mixed $value
+     * @return void
      */
     public function setValue($value): void;
 
@@ -25,6 +26,7 @@ 
 
     /**
      * @param string $label
+     * @return void
      */
     public function setLabel(string $label): void;
 
@@ -55,6 +57,7 @@ 
 
     /**
      * @param ValidatorInterface $validator
+     * @return void
      */
     public function addValidator(ValidatorInterface $validator): void;
 
@@ -65,6 +68,7 @@ 
 
     /**
      * @param FilterInterface $filter
+     * @return void
      */
     public function addFilter(FilterInterface $filter): void;
 
@@ -74,7 +78,7 @@ 
     public function getFilters(): FilterCollection;
 
     /**
-     * @param FilterInterface $filter
+     * @return void
      */
     public function setTransformer(TransformerInterface $transformer): void;
 
@@ -101,6 +105,7 @@ 
 
     /**
      * @param string $message
+     * @return void
      */
     public function setCustomErrorMessage(string $message): void;
 
@@ -122,11 +127,13 @@ 
     /**
      * @param $key
      * @param $value
+     * @return void
      */
     public function setAttribute(string $key, $value): void;
 
     /**
      * @param array $attributes
+     * @return void
      */
     public function setAttributes(array $attributes): void;
 
@@ -142,6 +149,7 @@ 
 
     /**
      * @param FieldRendererInterface $renderer
+     * @return void
      */
     public function setRenderer(FieldRendererInterface $renderer): void;
 
@@ -149,12 +157,14 @@ 
 
     /**
      * @param bool $required
+     * @return void
      */
     public function setRequired(bool $required): void;
 
     /**
      * @param FormInterface $field
      * @param $triggerValue
+     * @return void
      */
     public function addDynamicForm(FormInterface $field, string $triggerValue): void;
 

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use Del\Form\Collection\ValidatorCollection;
 use Del\Form\Filter\FilterInterface;
 use Del\Form\FormInterface;
-use Del\Form\Renderer\Field\SelectRender;
 use Del\Form\Validator\ValidatorInterface;
 use Del\Form\Renderer\Field\FieldRendererInterface;
 use Exception;

src/Renderer/Field/CheckboxRender.php

Doc Comments +1 added lines

@@ -63,6 +63,7 @@
      * @param FieldInterface $field
      * @param $value
      * @param $labelText
+     * @param boolean $inline
      * @return DOMElement
      */
     private function processOption(FieldInterface $field, $value, $labelText, $inline)

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 use DOMDocumentFragment;
 use DOMElement;
 use DOMNode;
-use DOMText;
 use InvalidArgumentException;
 use LogicException;
 

Spacing +3 added lines, -3 removed lines

@@ -97,9 +97,9 @@ 
      */
     private function getLabel(FieldInterface $field, string $labelText): DOMElement
     {
-        $this->counter ++;
+        $this->counter++;
         $label = $this->getDom()->createElement('label');
-        $label->setAttribute('for', $field->getId() . $this->counter);
+        $label->setAttribute('for', $field->getId().$this->counter);
         $label->setAttribute('class', 'form-check-label');
         $text = $this->createText($labelText);
         $label->appendChild($text);
@@ -118,7 +118,7 @@ 
         $checkbox = $this->getDom()->createElement('input');
         $checkbox->setAttribute('class', 'form-check-input');
         $checkbox->setAttribute('type', 'checkbox');
-        $fieldName = $this->isMultiCheckbox ? $field->getName() . '[]' : $field->getName();
+        $fieldName = $this->isMultiCheckbox ? $field->getName().'[]' : $field->getName();
         $checkbox->setAttribute('name', $fieldName);
         $checkbox->setAttribute('value', $value);
         $fieldValue = $field->getValue();

src/Renderer/Field/RadioRender.php

Unused Use Statements +6 added lines, -7 removed lines

@@ -2,13 +2,12 @@
 

 namespace Del\Form\Renderer\Field;

 

-use Del\Form\Field\FieldInterface;

-use Del\Form\Field\Radio;

-use DOMDocumentFragment;

-use DOMElement;

-use DOMNode;

-use DOMText;

-use InvalidArgumentException;

+use Del\Form\Field\FieldInterface;
+use Del\Form\Field\Radio;
+use DOMDocumentFragment;
+use DOMElement;
+use DOMNode;
+use InvalidArgumentException;
 use LogicException;

 

 class RadioRender extends AbstractFieldRender implements FieldRendererInterface

Doc Comments +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
      * @param FieldInterface $field

      * @param $value

      * @param $labelText

-     * @param $inline

+     * @param boolean $inline

      * @return DOMElement

      */

     private function renderRadio(FieldInterface $field, $value, $labelText, $inline)

Spacing +2 added lines, -2 removed lines

@@ -78,9 +78,9 @@
      */

     private function getLabel(FieldInterface $field, string $labelText): DOMElement

     {

-        $this->counter ++;

+        $this->counter++;

         $label = $this->getDom()->createElement('label');

-        $label->setAttribute('for', $field->getId() . $this->counter);

+        $label->setAttribute('for', $field->getId().$this->counter);

         $label->setAttribute('class', 'form-check-label');

         $text = $this->createText($labelText);

         $label->appendChild($text);

src/Validator/FileExtensionValidator.php

Spacing +3 added lines, -3 removed lines

@@ -36,12 +36,12 @@
     {
         $validExtensions = $this->validExtensions;
         $last = array_pop($validExtensions);
-        $extensions = 'The file must be a ' . $last . ' file.';
+        $extensions = 'The file must be a '.$last.' file.';
 
         if (count($validExtensions) > 0) {
             $extensions = implode(', ', $validExtensions);
-            $extensions .= ' or ' . $last;
-            $extensions = 'The file must be either a ' . $extensions . ' file.';
+            $extensions .= ' or '.$last;
+            $extensions = 'The file must be either a '.$extensions.' file.';
         }
 
         return [$extensions];