From ba6e8294f7bd70cc88df87093259d5ff1b449623 Mon Sep 17 00:00:00 2001
From: Mudzakkir <707089+qaharmdz@users.noreply.github.com>
Date: Mon, 26 Jul 2021 23:07:28 +0700
Subject: [PATCH] Fixed invalid tags break the parseDocBlock()

---
 CHANGELOG.md              |  3 +-
 examples/Library/Acme.php |  2 +
 src/DocTracer.php         | 88 ++++++++++++++++++++++++---------------
 3 files changed, 59 insertions(+), 34 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index f83b51c..22594be 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -15,7 +15,8 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 ---
 
 ## [Unreleased]
-
+### Fixed
+- Invalid tags break the parseDocBlock()
 
 ## [v1.0.0] - 2021-07-25
 ### Added
diff --git a/examples/Library/Acme.php b/examples/Library/Acme.php
index 812a534..a9c78ea 100644
--- a/examples/Library/Acme.php
+++ b/examples/Library/Acme.php
@@ -29,6 +29,8 @@
  * @author    Another Author <delta@example.com>
  * @copyright 2021 (c) Acme Corporation
  * @license   [https://opensource.org/licenses/MIT](https://opensource.org/licenses/MIT)
+ *
+ * @return    [type]    Show invalid tags as is
  */
 class Acme extends ParentClass implements \Countable
 {
diff --git a/src/DocTracer.php b/src/DocTracer.php
index 081e91e..15f95c6 100644
--- a/src/DocTracer.php
+++ b/src/DocTracer.php
@@ -5,6 +5,7 @@
 namespace Mdz;
 
 use phpDocumentor\Reflection\DocBlockFactory;
+use phpDocumentor\Reflection\DocBlock\Tags\InvalidTag;
 use Michelf\MarkdownExtra;
 
 /**
@@ -375,26 +376,28 @@ protected function parseDocBlock(\Reflector $ref): array
 
         $i = 0;
         foreach ($docBlock->getTags() as $docTags) {
-            if ($docTags->getName() === 'param') {
-                $tags[$docTags->getName()][$i] = [
-                    'name'        => '@' . $docTags->getName(),
-                    'type'        => (string)$docTags->getType(),
-                    'variable'    => '$' . $docTags->getVariableName(),
-                    'description' => $docTags->getDescription()->render(),
-                ];
-            } elseif (in_array($docTags->getName(), ['var', 'return', 'throws'])) {
-                $tags[$docTags->getName()][$i] = [
-                    'name'        => '@' . $docTags->getName(),
-                    'type'        => (string)$docTags->getType(),
-                    'description' => $docTags->getDescription()->render(),
-                ];
-            } else {
-                $tags[$docTags->getName()][$i] = [
-                    'name'   => '@' . $docTags->getName(),
-                ];
+            if ($docTags && !$docTags instanceof InvalidTag) {
+                if ($docTags->getName() === 'param') {
+                    $tags[$docTags->getName()][$i] = [
+                        'name'        => '@' . $docTags->getName(),
+                        'type'        => (string)$docTags->getType(),
+                        'variable'    => '$' . $docTags->getVariableName(),
+                        'description' => $docTags->getDescription()->render(),
+                    ];
+                } elseif (in_array($docTags->getName(), ['var', 'return', 'throws'])) {
+                    $tags[$docTags->getName()][$i] = [
+                        'name'        => '@' . $docTags->getName(),
+                        'type'        => (string)$docTags->getType(),
+                        'description' => $docTags->getDescription()->render(),
+                    ];
+                } else {
+                    $tags[$docTags->getName()][$i] = [
+                        'name'   => '@' . $docTags->getName(),
+                    ];
+                }
             }
 
-            $tags[$docTags->getName()][$i]['render'] = trim(str_replace('@' . $docTags->getName(), '', $docTags->render()));
+            $tags[$docTags->getName()][$i]['render'] = trim($docTags->render());
 
             $i++;
         }
@@ -469,23 +472,42 @@ protected function formatDocBlock(array $docBlock): string
 
                 foreach ($tags as $tag) {
                     if ($name === 'param') {
-                        $docs .= '<tr>';
-                        $docs .= '<td class="dt-doc-tag-name">' . $tag['name'] . '</td>';
-                        $docs .= '<td class="dt-doc-tag-type">' . $this->wordbreak($tag['type']) . '</td>';
-                        $docs .= '<td class="dt-doc-tag-variable">' . $tag['variable'] . '</td>';
-                        $docs .= '<td class="dt-doc-tag-description">' . $this->markdown($tag['description']) . '</td>';
-                        $docs .= '</tr>';
+                        if (isset($tag['name'])) {
+                            $docs .= '<tr>';
+                            $docs .= '<td class="dt-doc-tag-name">' . $tag['name'] . '</td>';
+                            $docs .= '<td class="dt-doc-tag-type">' . $this->wordbreak($tag['type']) . '</td>';
+                            $docs .= '<td class="dt-doc-tag-variable">' . $tag['variable'] . '</td>';
+                            $docs .= '<td class="dt-doc-tag-description">' . $this->markdown($tag['description']) . '</td>';
+                            $docs .= '</tr>';
+                        } else {
+                            $docs .= '<tr>';
+                            $docs .= '<td class="dt-doc-tag-render" colspan="4">' . $this->markdown($tag['render']) . '</td>';
+                            $docs .= '</tr>';
+                        }
                     } elseif (in_array($name, ['var', 'return', 'throws'])) {
-                        $docs .= '<tr>';
-                        $docs .= '<td class="dt-doc-tag-name">' . $tag['name'] . '</td>';
-                        $docs .= '<td class="dt-doc-tag-type">' . $this->wordbreak($tag['type']) . '</td>';
-                        $docs .= '<td class="dt-doc-tag-description">' . $this->markdown($tag['description']) . '</td>';
-                        $docs .= '</tr>';
+                        if (isset($tag['name'])) {
+                            $docs .= '<tr>';
+                            $docs .= '<td class="dt-doc-tag-name">' . $tag['name'] . '</td>';
+                            $docs .= '<td class="dt-doc-tag-type">' . $this->wordbreak($tag['type']) . '</td>';
+                            $docs .= '<td class="dt-doc-tag-description">' . $this->markdown($tag['description']) . '</td>';
+                            $docs .= '</tr>';
+                        } else {
+                            $docs .= '<tr>';
+                            $docs .= '<td class="dt-doc-tag-render" colspan="3">' . $this->markdown($tag['render']) . '</td>';
+                            $docs .= '</tr>';
+                        }
                     } else {
-                        $docs .= '<tr>';
-                        $docs .= '<td class="dt-doc-tag-name">' . $tag['name'] . '</td>';
-                        $docs .= '<td class="dt-doc-tag-render">' . $this->markdown($tag['render']) . '</td>';
-                        $docs .= '</tr>';
+                        if (isset($tag['name'])) {
+                            $render = str_replace('@' . $tag['name'], '', $tag['render']);
+                            $docs .= '<tr>';
+                            $docs .= '<td class="dt-doc-tag-name">' . $tag['name'] . '</td>';
+                            $docs .= '<td class="dt-doc-tag-render">' . $this->markdown($render) . '</td>';
+                            $docs .= '</tr>';
+                        } else {
+                            $docs .= '<tr>';
+                            $docs .= '<td class="dt-doc-tag-render" colspan="2">' . $this->markdown($tag['render']) . '</td>';
+                            $docs .= '</tr>';
+                        }
                     }
                 }
                 $docs .= '</table>';