Merge pull request #152 from ashnazg/strict

Strict Updates

Author: ashnazg

src/DocBlock.php

@@ -133,7 +133,7 @@ public function isTemplateEnd(): bool
      *
      * @return Tag[]
      */
-    public function getTags()
+    public function getTags(): array
     {
         return $this->tags;
     }
@@ -146,7 +146,7 @@ public function getTags()
      *
      * @return Tag[]
      */
-    public function getTagsByName(string $name)
+    public function getTagsByName(string $name): array
     {
         $result = [];
 

src/DocBlock/Description.php

@@ -72,7 +72,7 @@ public function __construct(string $bodyTemplate, array $tags = [])
      *
      * @return Tag[]
      */
-    public function getTags()
+    public function getTags(): array
     {
         return $this->tags;
     }

src/DocBlock/DescriptionFactory.php

@@ -61,7 +61,7 @@ public function create(string $contents, ?TypeContext $context = null): Descript
      *
      * @return string[] A series of tokens of which the description text is composed.
      */
-    private function lex(string $contents)
+    private function lex(string $contents): array
     {
         $contents = $this->removeSuperfluousStartingWhitespace($contents);
 

src/DocBlock/StandardTagFactory.php

@@ -163,7 +163,7 @@ public function registerTagHandler(string $tagName, string $handler): void
      *
      * @return string[]
      */
-    private function extractTagParts(string $tagLine)
+    private function extractTagParts(string $tagLine): array
     {
         $matches = [];
         if (! preg_match('/^@(' . self::REGEX_TAGNAME . ')(?:\s*([^\s].*)|$)/us', $tagLine, $matches)) {
@@ -222,7 +222,7 @@ private function findHandlerClassName(string $tagName, TypeContext $context): st
      * @return mixed[] A series of values that can be passed to the Factory Method of the tag whose parameters
      *     is provided with this method.
      */
-    private function getArgumentsForParametersFromWiring($parameters, $locator)
+    private function getArgumentsForParametersFromWiring($parameters, $locator): array
     {
         $arguments = [];
         foreach ($parameters as $index => $parameter) {
@@ -251,7 +251,7 @@ private function getArgumentsForParametersFromWiring($parameters, $locator)
      *
      * @return \ReflectionParameter[]
      */
-    private function fetchParametersForHandlerFactoryMethod(string $handlerClassName)
+    private function fetchParametersForHandlerFactoryMethod(string $handlerClassName): array
     {
         if (! isset($this->tagHandlerParameterCache[$handlerClassName])) {
             $methodReflection = new \ReflectionMethod($handlerClassName, 'create');
@@ -271,7 +271,7 @@ private function fetchParametersForHandlerFactoryMethod(string $handlerClassName
      *
      * @return mixed[]
      */
-    private function getServiceLocatorWithDynamicParameters(TypeContext $context, string $tagName, string $tagBody)
+    private function getServiceLocatorWithDynamicParameters(TypeContext $context, string $tagName, string $tagBody): array
     {
         $locator = array_merge(
             $this->serviceLocator,

src/DocBlock/Tags/Covers.php

@@ -47,7 +47,7 @@ public static function create(
         ?DescriptionFactory $descriptionFactory = null,
         ?FqsenResolver $resolver = null,
         ?TypeContext $context = null
-    ) {
+    ): self {
         Assert::notEmpty($body);
 
         $parts = preg_split('/\s+/Su', $body, 2);

src/DocBlock/Tags/Deprecated.php

@@ -59,7 +59,7 @@ public static function create(
         ?string $body,
         ?DescriptionFactory $descriptionFactory = null,
         ?TypeContext $context = null
-    ) {
+    ): self {
         if (empty($body)) {
             return new static();
         }

src/DocBlock/Tags/Example.php

@@ -13,7 +13,6 @@
 
 namespace phpDocumentor\Reflection\DocBlock\Tags;
 
-use phpDocumentor\Reflection\DocBlock\Description;
 use phpDocumentor\Reflection\DocBlock\Tag;
 use Webmozart\Assert\Assert;
 
@@ -81,7 +80,7 @@ public function getContent()
     /**
      * {@inheritdoc}
      */
-    public static function create(string $body)
+    public static function create(string $body): ?Tag
     {
         // File component: File path in quotes or File URI / Source information
         if (! preg_match('/^(?:\"([^\"]+)\"|(\S+))(?:\s+(.*))?$/sux', $body, $matches)) {

src/DocBlock/Tags/Generic.php

@@ -49,7 +49,7 @@ public static function create(
         string $name = '',
         ?DescriptionFactory $descriptionFactory = null,
         ?TypeContext $context = null
-    ) {
+    ): self {
         Assert::stringNotEmpty($name);
         Assert::notNull($descriptionFactory);
 

src/DocBlock/Tags/Method.php

@@ -92,12 +92,12 @@ public static function create(
                 )?
                 # Return type
                 (?:
-                    (   
+                    (
                         (?:[\w\|_\\\\]*\$this[\w\|_\\\\]*)
                         |
                         (?:
                             (?:[\w\|_\\\\]+)
-                            # array notation           
+                            # array notation
                             (?:\[\])*
                         )*
                     )
@@ -170,7 +170,7 @@ public function getMethodName(): string
     /**
      * @return string[]
      */
-    public function getArguments()
+    public function getArguments(): array
     {
         return $this->arguments;
     }
@@ -204,7 +204,7 @@ public function __toString(): string
             . ($this->description ? ' ' . $this->description->render() : ''));
     }
 
-    private function filterArguments($arguments)
+    private function filterArguments(array $arguments = []): array
     {
         foreach ($arguments as &$argument) {
             if (is_string($argument)) {
@@ -227,7 +227,7 @@ private function filterArguments($arguments)
         return $arguments;
     }
 
-    private static function stripRestArg($argument)
+    private static function stripRestArg(string $argument): string
     {
         if (strpos($argument, '...') === 0) {
             $argument = trim(substr($argument, 3));

src/DocBlock/Tags/Param.php

@@ -53,7 +53,7 @@ public static function create(
         ?TypeResolver $typeResolver = null,
         ?DescriptionFactory $descriptionFactory = null,
         ?TypeContext $context = null
-    ) {
+    ): self {
         Assert::stringNotEmpty($body);
         Assert::allNotNull([$typeResolver, $descriptionFactory]);