From ebc0f974dce00d1d6e0cd0f564a285e730293888 Mon Sep 17 00:00:00 2001
From: Sebijk <git@sebijk.net>
Date: Tue, 6 Jun 2023 16:55:13 +0200
Subject: [PATCH] fix undefined variable

---
 src/templates/modern/li/search.details.tpl | 10 +++++-----
 src/templates/modern/li/search.results.tpl |  2 +-
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/templates/modern/li/search.details.tpl b/src/templates/modern/li/search.details.tpl
index 24229dd..54727da 100644
--- a/src/templates/modern/li/search.details.tpl
+++ b/src/templates/modern/li/search.details.tpl
@@ -50,19 +50,19 @@
 			<input type="checkbox" name="items[{$resultCat.name}][]" id="checkbox_{$resultCatID}_{$resultID}" value="{$result.id}"{if !$resultCat.massActions} disabled="disabled"{/if} onchange="toggleResultMassActions(document.forms.f1, {$resultCatID});" />
 		</td>
 		<td width="24">
-			<i class="fa {if $result.icon}{$result.icon}{else}{$resultCat.icon}{/if}" aria-hidden="true"></i>
+			<i class="fa {if !empty($result.icon)}{$result.icon}{else}{$resultCat.icon}{/if}" aria-hidden="true"></i>
 		</td>
 		<td nowrap="nowrap" style="padding:4px;">
 			<a title="{text value=$result.title}" href="{if $result.extLink}{$result.extLink}{else}{$result.link}sid={$sid}{/if}"{if $result.extLink} target="_blank"{/if}>
 				<div style="text-overflow:ellipsis;overflow:hidden;{if $result.bold}font-weight:bold;{/if}{if $result.strike}text-decoration:line-through;{/if}">
 					{text value=$result.title}
 				</div>
-				{if $result.excerpt}<div style="color:#777;text-overflow:ellipsis;overflow:hidden;">{$result.excerpt}</div>{/if}
+				{if !empty($result.excerpt)}<div style="color:#777;text-overflow:ellipsis;overflow:hidden;">{$result.excerpt}</div>{/if}
 			</a>
 		</td>
-		<td width="130"><span style="{if $result.bold}font-weight:bold;{/if}{if $result.strike}text-decoration:line-through;{/if}">{if $result.date}{date timestamp=$result.date nice=true}{/if}</span></td>
-		<td width="75"><span style="{if $result.bold}font-weight:bold;{/if}{if $result.strike}text-decoration:line-through;{/if}">{if $result.size||$result.size===0}{size bytes=$result.size}{/if}</span></td>
-		<td style="text-align:center;">{if $result.score}{$result.score} %{else}-{/if}</td>
+		<td width="130"><span style="{if !empty($result.bold)}font-weight:bold;{/if}{if $result.strike}text-decoration:line-through;{/if}">{if $result.date}{date timestamp=$result.date nice=true}{/if}</span></td>
+		<td width="75"><span style="{if !empty($result.bold)}font-weight:bold;{/if}{if $result.strike}text-decoration:line-through;{/if}">{if $result.size||$result.size===0}{size bytes=$result.size}{/if}</span></td>
+		<td style="text-align:center;">{if !empty($result.score)}{$result.score} %{else}-{/if}</td>
 	</tr>
 	{/foreach}
 	{if $resultCat.massActions}
diff --git a/src/templates/modern/li/search.results.tpl b/src/templates/modern/li/search.results.tpl
index 2f48c72..f6029ed 100644
--- a/src/templates/modern/li/search.results.tpl
+++ b/src/templates/modern/li/search.results.tpl
@@ -5,7 +5,7 @@
 {foreach from=$resultCat.results item=result}
 	<tr>
 		<td class="resultLeft" height="20" width="90">{if $first}{assign var=first value=false}{text value=$resultCat.title}{else}&nbsp;{/if}</td>
-		<td class="resultRight" onclick="{if !empty($result.extLink)}window.open('{$result.extLink}');{else}document.location.href='{$result.link}sid={$sid}';{/if}parent.hideSearchPopup(true);" title="{text value=$result.excerpt allowEmpty=true stripTags=true}">
+		<td class="resultRight" onclick="{if !empty($result.extLink)}window.open('{$result.extLink}');{else}document.location.href='{$result.link}sid={$sid}';{/if}parent.hideSearchPopup(true);" title="{if isset($result.excerpt)}{text value=$result.excerpt allowEmpty=true stripTags=true}{/if}">
 			<i class="fa {if !empty($result.icon)}{$result.icon}{else}{$resultCat.icon}{/if}" aria-hidden="true"></i>
 			{text value=$result.title cut=25}
 		</td>