From b4701356280c6ee9be31037628195877df477bda Mon Sep 17 00:00:00 2001
From: vincent <vincent@enhydra.fr>
Date: Sat, 26 Mar 2022 15:36:49 +0100
Subject: [PATCH] wip #5181 @0.5

---
 resources/js/search.js                      | 31 ++++++++-------------
 resources/views/components/search.blade.php | 10 +++++--
 2 files changed, 20 insertions(+), 21 deletions(-)

diff --git a/resources/js/search.js b/resources/js/search.js
index 311427e..1e77fff 100644
--- a/resources/js/search.js
+++ b/resources/js/search.js
@@ -28,30 +28,23 @@ export default () => ({
         this.medialibrary = this.$el.dataset.medialibrary;
     },
 
-    makeResultLink(result, classes) {
-        // Result links have differing attributes, so we need this function to be able to generate the HTML correctly
-        // This can't be done directly in the x-for template loop
-        let URL = result.url;
-        let click = 'return true;';
-
-        // Media items get displayed on the media library page (simpler this way so extra JSON object isn't needed)
-        if (result.type === 'video' || result.type === 'audio') {
-            URL = `${this.medialibrary}?player=${result.id}`;
-        }
-
-        // PDFs can be opened directly
-        if (result.type === 'pdf') {
-            click = 'console.log($el.attributes.href.value); openPDF($el.attributes.href.value); closeSearch(); return false;';
-        }
-        return {url: URL, click: click, classes: classes, displayedTitle: result.displayedTitle};
-    },
-
     get results() {
         if (this.query.length < 3) {
             return [];
         }
 
-        var res = this.miniSearch.search(this.query);
+        var results = this.miniSearch.search(this.query);
+        var res = [];
+        for (var i in results) {
+            var r = results[r];
+            let URL = r.url;
+            let click = 'return true;';
+            // Media items get displayed on the media library page (simpler this way so extra JSON object isn't needed)
+            if (r.type === 'video' || r.type === 'audio') {
+                URL = `${this.medialibrary}?player=${r.id}`;
+            }
+            res.push({url: URL, click: click, displayedTitle: r.displayedTitle, id: r.id});
+        }
 
         return res;
     },
diff --git a/resources/views/components/search.blade.php b/resources/views/components/search.blade.php
index 3b68a99..aa001f6 100644
--- a/resources/views/components/search.blade.php
+++ b/resources/views/components/search.blade.php
@@ -70,8 +70,14 @@
             <div class="h-full max-h-full overflow-y-auto">
                 <template x-for="result in results" :key="result.id">
                     <div class="py-8 border-b border-black border-opacity-20">
-                        <div x-model="makeResultLink(result, 'text-2xl text-blue')">
-                            <a x-bind:href="url" x-bind:class="classes" x-text="displayedTitle" @click="click"></a>
+                        <div>
+                            <template x-if="result.type==pdf">
+                                <a x-bind:href="url" class="text-2xl text-blue" x-text="displayedTitle"></a>
+                            </template>
+                            <template x-if="result.type!=pdf">
+                                <a x-bind:href="url" class="text-2xl text-blue" x-text="displayedTitle"
+                                   @click.prevent="console.log($el.attributes.href.value); openPDF($el.attributes.href.value); closeSearch();"></a>
+                            </template>
                         </div>
                         <p x-text="result.text" class="mt-2 line-clamp-1"></p>
                     </div>
-- 
2.39.5