refactor: removed deprecated method and moved layout code to template

thorsten · Jan 12, 2025 · eb70a4e · eb70a4e
1 parent f319250
commit eb70a4e
Show file tree

Hide file tree

Showing 5 changed files with 54 additions and 45 deletions.
diff --git a/phpmyfaq/assets/templates/default/faq.twig b/phpmyfaq/assets/templates/default/faq.twig
@@ -1,22 +1,22 @@
 {% extends 'index.twig' %}
 
 {% block richSnippets %}
-<script type="application/ld+json">
-  {
-    "@context": "https://schema.org",
-    "@type": "FAQPage",
-    "mainEntity": [
-      {
-        "@type": "Question",
-        "name": "{{ question }}",
+  <script type="application/ld+json">
+    {
+      "@context": "https://schema.org",
+      "@type": "FAQPage",
+      "mainEntity": [
+        {
+          "@type": "Question",
+          "name": "{{ question }}",
         "acceptedAnswer": {
           "@type": "Answer",
           "text": "{{ answer | striptags }}"
         }
       }
     ]
   }
-</script>
+  </script>
 {% endblock %}
 
 {% block content %}
@@ -26,6 +26,18 @@
 
       <article class="pmf-faq-body pb-4 mb-4 border-bottom">{{ answer|raw }}</article>
 
+      {% if attachmentList is not empty %}
+        <p>{{ 'msgAttachedFiles' | translate }}:</p>
+        <ul class="pb-4 mb-4 border-bottom">
+          {% for attachment in attachmentList %}
+            <li>
+              <i class="bi bi-{{ attachment.icon }}" aria-hidden="true"></i>
+              <a href="{{ attachment.url }}">{{ attachment.filename|e }}</a>
+            </li>
+          {% endfor %}
+        </ul>
+      {% endif %}
+
       <p class="d-print-none">{{ writeCommentMsg|raw }}</p>
       <div id="pmf-comment-add-success"></div>
       <div class="d-print-none" id="comments">{{ renderComments|raw }}</div>
@@ -67,12 +79,12 @@
                   {{ msgAddBookmark }}
                 </a>
               </li>
-            <li class="list-group-item bg-transparent">
-              <i aria-hidden="true" class="bi bi-file-pdf"></i>
-              <a target="_blank" href="{{ linkToPdf }}" rel="noopener" class="text-decoration-none">
-                {{ msgPdf }}
-              </a>
-            </li>
+              <li class="list-group-item bg-transparent">
+                <i aria-hidden="true" class="bi bi-file-pdf"></i>
+                <a target="_blank" href="{{ linkToPdf }}" rel="noopener" class="text-decoration-none">
+                  {{ msgPdf }}
+                </a>
+              </li>
             {% endif %}
             {% if enableSendToFriend == true %}
               <li class="list-group-item bg-transparent">

diff --git a/phpmyfaq/faq.php b/phpmyfaq/faq.php
@@ -165,7 +165,7 @@
 if ($faqConfig->get('records.disableAttachments') && 'yes' == $faq->faqRecord['active']) {
     try {
         $attList = AttachmentFactory::fetchByRecordId($faqConfig, $faqId);
-        $answer .= $attachmentHelper->renderAttachmentList($attList);
+        $attachmentList = $attachmentHelper->getAttachmentList($attList);
     } catch (AttachmentException) {
         // handle exception
     }
@@ -318,6 +318,7 @@
     'solutionIdLink' => Link::getSystemRelativeUri() . '?solution_id=' . $faq->faqRecord['solution_id'],
     'question' => $question,
     'answer' => $answer,
+    'attachmentList' => $attachmentList,
     'faqDate' => $date->format($faq->faqRecord['date']),
     'faqAuthor' => Strings::htmlentities($author),
     'msgPdf' => Translation::get('msgPDF'),

diff --git a/phpmyfaq/src/phpMyFAQ/Helper/AttachmentHelper.php b/phpmyfaq/src/phpMyFAQ/Helper/AttachmentHelper.php
@@ -18,8 +18,6 @@
 namespace phpMyFAQ\Helper;
 
 use phpMyFAQ\Attachment\AbstractAttachment;
-use phpMyFAQ\Strings;
-use phpMyFAQ\Translation;
 
 /**
  * Class AttachmentHelper
@@ -28,29 +26,23 @@
 class AttachmentHelper
 {
     /**
-     * Returns an HTML list of attached files.
+     * Returns a list of attached files.
      *
      * @param AbstractAttachment[] $attachmentList
-     * @deprecated Rewrite this method to use Twig, will be removed in v4.1
      */
-    public function renderAttachmentList(array $attachmentList): string
+    public function getAttachmentList(array $attachmentList): array
     {
         if ($attachmentList === []) {
-            return '';
+            return [];
         }
 
-        $html = sprintf('<p>%s:</p><ul>', Translation::get('msgAttachedFiles'));
-
-        foreach ($attachmentList as $attachment) {
-            $html .= sprintf(
-                '<li><i class="bi bi-%s" aria-hidden="true"></i> <a href="%s">%s</a></li>',
-                $this->mapMimeTypeToIcon($attachment->getMimeType()),
-                $attachment->buildUrl(),
-                Strings::htmlentities($attachment->getFilename())
-            );
-        }
-
-        return $html . '</ul>';
+        return array_map(function ($attachment) {
+            return [
+                'icon' => $this->mapMimeTypeToIcon($attachment->getMimeType()),
+                'url' => $attachment->buildUrl(),
+                'filename' => $attachment->getFilename(),
+            ];
+        }, $attachmentList);
     }
 
     private function mapMimeTypeToIcon(string $mimeType): string

diff --git a/phpmyfaq/src/phpMyFAQ/Helper/CommentHelper.php b/phpmyfaq/src/phpMyFAQ/Helper/CommentHelper.php
@@ -31,7 +31,7 @@ class CommentHelper extends AbstractHelper
      * Returns all user comments (HTML formatted) from a record by type.
      *
      * @param Comment[] $comments
-     * @deprecated Rewrite this method to use Twig
+     * @deprecated Rewrite this method to use Twig, will be removed in v4.1
      */
     public function getComments(array $comments): string
     {

diff --git a/tests/phpMyFAQ/Helper/AttachmentHelperTest.php b/tests/phpMyFAQ/Helper/AttachmentHelperTest.php
@@ -22,28 +22,32 @@ protected function setUp(): void
         $this->attachmentHelper = new AttachmentHelper();
     }
 
-    public function testRenderAttachmentListEmpty()
+    public function testGetAttachmentListEmpty()
     {
         $attachmentList = [];
-        $result = $this->attachmentHelper->renderAttachmentList($attachmentList);
-        $this->assertEquals('', $result);
+        $result = $this->attachmentHelper->getAttachmentList($attachmentList);
+        $this->assertEquals([], $result);
     }
 
-    public function testRenderAttachmentListWithAttachments()
+    public function testGetAttachmentListWithAttachments()
     {
         $attachmentMock = $this->createMock(AbstractAttachment::class);
         $attachmentMock->method('getMimeType')->willReturn('application/pdf');
-        $attachmentMock->method('buildUrl')->willReturn('http://example.com/file.pdf');
+        $attachmentMock->method('buildUrl')->willReturn('https://example.com/file.pdf');
         $attachmentMock->method('getFilename')->willReturn('file.pdf');
 
         $attachmentList = [$attachmentMock];
 
-        $result = $this->attachmentHelper->renderAttachmentList($attachmentList);
+        $result = $this->attachmentHelper->getAttachmentList($attachmentList);
 
-        $expectedHtml = '<p>Attached files:</p><ul>';
-        $expectedHtml .= '<li><i class="bi bi-file-pdf-o" aria-hidden="true"></i> <a href="http://example.com/file.pdf">file&period;pdf</a></li>';
-        $expectedHtml .= '</ul>';
+        $expectedResult = [
+            [
+                'icon' => 'file-pdf-o',
+                'url' => 'https://example.com/file.pdf',
+                'filename' => 'file.pdf',
+            ],
+        ];
 
-        $this->assertEquals($expectedHtml, $result);
+        $this->assertEquals($expectedResult, $result);
     }
 }