From dacb7c85aa7654c70f97c9bd4dc80a52913d842a Mon Sep 17 00:00:00 2001
From: Janos SUTO <sj@acts.hu>
Date: Sun, 2 May 2021 08:19:09 +0200
Subject: [PATCH] Minor gui fixes

Signed-off-by: Janos SUTO <sj@acts.hu>
---
 webui/system/misc.php                         | 31 ++++++++++++++++++-
 .../theme/default/templates/search/helper.tpl |  8 ++---
 2 files changed, 34 insertions(+), 5 deletions(-)

diff --git a/webui/system/misc.php b/webui/system/misc.php
index ae356ed2..ca9598e5 100644
--- a/webui/system/misc.php
+++ b/webui/system/misc.php
@@ -1,5 +1,10 @@
 <?php
 
+unction H($s = '') {
+   print htmlentities($s);
+}
+
+
 function LOGGER($event = '', $username = '') {
    $ipaddr = '';
 
@@ -493,7 +498,31 @@ function fixup_date_condition($field = '', $date1 = 0, $date2 = 0) {
 
 
 function make_short_string($what, $length) {
-   return strlen($what) > $length ? substr($what, 0, $length) . "..." : $what;
+   if($length < 1) { return ''; }
+
+   if(strlen($what) <= $length) { return $what; }
+
+   $arr = preg_split("/\s/", $what);
+   $s = '';
+
+   $i = 0;
+   foreach($arr as $a) {
+      if($i == 0) {
+         if($length > 0 && strlen($a) > $length) {
+            return substr($a, 0, $length) . '...';
+         }
+      }
+
+      if(strlen($s) + strlen($a) <= $length) {
+         $s .= $a . ' ';
+      } else {
+         break;
+      }
+
+      $i++;
+   }
+
+   return $s . '...';
 }
 
 
diff --git a/webui/view/theme/default/templates/search/helper.tpl b/webui/view/theme/default/templates/search/helper.tpl
index f02d19ed..dfeaec96 100644
--- a/webui/view/theme/default/templates/search/helper.tpl
+++ b/webui/view/theme/default/templates/search/helper.tpl
@@ -43,7 +43,7 @@
       </thead>
       <tbody>
     <?php $i=0; foreach ($messages as $message) { ?>
-            
+
          <tr onmouseover="Piler.current_message_id = <?php print $message['id']; ?>; return false;" id="e_<?php print $message['id']; ?>" class="resultrow new <?php if($message['deleted'] == 1) { ?>xxx<?php } ?>" onclick="Piler.view_message_by_pos(<?php print $i; ?>);">
             <td id="c1_r<?php print $i; ?>" class="resultcell restore" onclick="Piler.stop_propagation(event);"><input type="checkbox" id="r_<?php print $message['id']; ?>" name="r_<?php print $message['id']; ?>" value="iiii" <?php if(SEARCH_RESULT_CHECKBOX_CHECKED == 1) { ?>checked="checked"<?php } ?> class="restorebox" /></td>
             <td id="c2_r<?php print $i; ?>" class="resultcell id"><?php print ($page*$page_len) + $i + 1; ?></td>
@@ -67,7 +67,7 @@
 
     <?php $i++; } ?>
       </tbody>
-      
+
       <tfoot>
         <tr>
           <th id="restore-header"><input type="checkbox" id="bulkcheck2" name="bulkcheck2" value="1" <?php if(SEARCH_RESULT_CHECKBOX_CHECKED == 1) { ?>checked="checked"<?php } ?> class="restorebox" onclick="Piler.toggle_bulk_check('2');" /></th>
@@ -106,7 +106,7 @@
      <?php } ?>
        </tr>
       </tfoot>
-      
+
     </table>
 
     <?php } else if($n == 0) { ?>
@@ -135,7 +135,7 @@
             &nbsp;
 
          <?php if(Registry::get('auditor_user') == 1 && $session->get("sphx_query")) { ?>
-            <span style="margin-left: 30px;"><a href="#" onclick="Piler.show_message('messagebox1', '<?php print $session->get("sphx_query"); ?>', 5);">sphinx</a></span>
+            <span style="margin-left: 30px;"><a href="#" onclick="Piler.show_message('messagebox1', '<?php H($session->get("sphx_query")); ?>', 5);">sphinx</a></span>
          <?php } ?>
 
     <?php } else { print $text_none_found; } ?>