Improve

src/App/Helper/ConstraintPayloadDocHelper.php

@@ -22,7 +22,7 @@ class ConstraintPayloadDocHelper
      * @param TypeDoc $doc
      * @param Constraint $constraint
      */
-    public function appendPayloadDoc(TypeDoc $doc, Constraint $constraint)
+    public function appendPayloadDoc(TypeDoc $doc, Constraint $constraint) : void
     {
         if (!$this->hasPayloadDoc($constraint)) {
             return;

src/App/Helper/DocTypeHelper.php

@@ -51,7 +51,7 @@ public function guess(array $constraintList) : TypeDoc
      *
      * @return TypeDoc|null
      */
-    protected function getDocFromTypeConstraintOrPayloadDocIfExist(array $constraintList)
+    protected function getDocFromTypeConstraintOrPayloadDocIfExist(array $constraintList) : ?TypeDoc
     {
         $doc = null;
         // Check if a Type constraint exist or if a constraint have a type documentation
@@ -87,7 +87,7 @@ protected function getDocFromTypeConstraintOrPayloadDocIfExist(array $constraint
      *
      * @return TypeDoc|null
      */
-    private function normalizeType(string $type)
+    private function normalizeType(string $type) : ?TypeDoc
     {
         if ('scalar' === $type) {
             return new ScalarDoc();

src/App/Helper/MinMaxHelper.php

@@ -17,7 +17,7 @@ class MinMaxHelper
      * @param TypeDoc    $doc
      * @param Constraint $constraint
      */
-    public function append(TypeDoc $doc, Constraint $constraint)
+    public function append(TypeDoc $doc, Constraint $constraint) : void
     {
         if ($doc instanceof StringDoc) {
             $this->appendStringDoc($doc, $constraint);
@@ -32,7 +32,7 @@ public function append(TypeDoc $doc, Constraint $constraint)
      * @param StringDoc  $doc
      * @param Constraint $constraint
      */
-    private function appendStringDoc(StringDoc $doc, Constraint $constraint)
+    private function appendStringDoc(StringDoc $doc, Constraint $constraint) : void
     {
         if ($constraint instanceof Assert\Length) {
             if (null !== $constraint->min) {
@@ -54,7 +54,7 @@ private function appendStringDoc(StringDoc $doc, Constraint $constraint)
      * @param NumberDoc  $doc
      * @param Constraint $constraint
      */
-    private function appendNumberDoc(NumberDoc $doc, Constraint $constraint)
+    private function appendNumberDoc(NumberDoc $doc, Constraint $constraint) : void
     {
         $this->appendNumberMinMax($doc, $constraint);
 
@@ -69,7 +69,7 @@ private function appendNumberDoc(NumberDoc $doc, Constraint $constraint)
      * @param CollectionDoc $doc
      * @param Constraint    $constraint
      */
-    private function appendCollectionDoc(CollectionDoc $doc, Constraint $constraint)
+    private function appendCollectionDoc(CollectionDoc $doc, Constraint $constraint) : void
     {
         if ($constraint instanceof Assert\Choice) {
             if (null !== $constraint->min) {
@@ -112,7 +112,7 @@ private function appendCollectionDoc(CollectionDoc $doc, Constraint $constraint)
      * @param NumberDoc $doc
      * @param Constraint $constraint
      */
-    private function appendNumberMinMax(NumberDoc $doc, Constraint $constraint)
+    private function appendNumberMinMax(NumberDoc $doc, Constraint $constraint) : void
     {
         if ($constraint instanceof Assert\Range) {
             if (null !== $constraint->min) {

src/App/Helper/StringDocHelper.php

@@ -17,7 +17,7 @@ class StringDocHelper
      *
      * @throws \ReflectionException
      */
-    public function append(TypeDoc $doc, Constraint $constraint)
+    public function append(TypeDoc $doc, Constraint $constraint) : void
     {
         // If format already defined or type is defined and is not a string nor scalar => give up
         if (!$doc instanceof StringDoc) {

src/App/Helper/TypeGuesser.php

@@ -22,7 +22,7 @@ class TypeGuesser
      *
      * @return TypeDoc|null
      */
-    public function guessTypeFromConstraintList(array $constraintList)
+    public function guessTypeFromConstraintList(array $constraintList) : ?TypeDoc
     {
         $doc = $abstractTypeFound = null;
         foreach ($constraintList as $constraint) {
@@ -50,7 +50,7 @@ public function guessTypeFromConstraintList(array $constraintList)
      *
      * @return TypeDoc|null
      */
-    protected function guessTypeFromConstraint(Constraint $constraint)
+    protected function guessTypeFromConstraint(Constraint $constraint) : ?TypeDoc
     {
         if (null !== ($type = $this->guessPrimaryTypeFromConstraint($constraint))) {
             return $type;
@@ -86,7 +86,7 @@ private function isAbstractType(TypeDoc $doc) : bool
      *
      * @return null|ArrayDoc|BooleanDoc|ObjectDoc|ScalarDoc|StringDoc
      */
-    private function guessPrimaryTypeFromConstraint(Constraint $constraint)
+    private function guessPrimaryTypeFromConstraint(Constraint $constraint) : ?TypeDoc
     {
         static $stringConstraintClassList = [
             Assert\Length::class, // << Applied on string only
@@ -140,7 +140,7 @@ private function guessPrimaryTypeFromConstraint(Constraint $constraint)
      *
      * @return ScalarDoc|StringDoc
      */
-    private function guessDateTimeType(Assert\DateTime $constraint)
+    private function guessDateTimeType(Assert\DateTime $constraint) : TypeDoc
     {
         if ('U' === $constraint->format) {
             return new ScalarDoc();// Don't know if value will be an number as string or as integer
@@ -154,7 +154,7 @@ private function guessDateTimeType(Assert\DateTime $constraint)
      *
      * @return ArrayDoc|ObjectDoc
      */
-    private function guestCollectionType(Assert\Collection $constraint)
+    private function guestCollectionType(Assert\Collection $constraint) : TypeDoc
     {
         // If only integer => array, else object
         $integerKeyList = array_filter(array_keys($constraint->fields), 'is_int');

src/Infra/Transformer/ConstraintToParamsDocTransformer.php

@@ -230,7 +230,7 @@ class_uses($object)
         return count(array_intersect($actualClassList, $classList)) > 0;
     }
 
-    private function addToAllowedValueListIfNotExist(TypeDoc $doc, $value)
+    private function addToAllowedValueListIfNotExist(TypeDoc $doc, $value) : void
     {
         if (!in_array($value, $doc->getAllowedValueList(), true)) {
             $doc->addAllowedValue($value);