From 8e77fe38607bf19e3f7d19c4a4f3cabf4946ee19 Mon Sep 17 00:00:00 2001
From: luccioman <luccioman@users.noreply.github.com>
Date: Wed, 8 Mar 2017 10:27:18 +0100
Subject: [PATCH] Fixed unresolved pattern case in search results progress bar.

This is a fix for mantis 715 (http://mantis.tokeek.de/view.php?id=715).

A possible path scenario that could leading to this case :
 - YaCy is running low in memory
 - a search is requested
 - before the end of search results rendering, the cleanup job runs and
deletes the running search event from the cache because of short memory
 - then yacysearchitem renders with "-UNRESOLVED_PATTERN-" parameter
values passed to the statistics() JavaScript function
---
 htroot/yacysearchitem.html |  9 +++++----
 htroot/yacysearchitem.java | 22 ++++++++++++----------
 2 files changed, 17 insertions(+), 14 deletions(-)

diff --git a/htroot/yacysearchitem.html b/htroot/yacysearchitem.html
index 73d08678c..d16e0d7fa 100644
--- a/htroot/yacysearchitem.html
+++ b/htroot/yacysearchitem.html
@@ -55,7 +55,8 @@
   #(item)#::<tr class="#(col)#TableCellLight::TableCellDark#(/col)#"><td>#[name]#</td><td><a href="#[href]#" target="#[target]#" #(noreferrer)#::rel="noreferrer"#(/noreferrer)#>#[hrefshort]#</a></tr>#(/item)#
 #(/content)#
 
-<script type="text/javascript">
-statistics("#[offset]#", "#[itemscount]#", "#[itemsperpage]#", "#[totalcount]#", "#[localResourceSize]#", "#[remoteResourceSize]#", "#[remoteIndexCount]#", "#[remotePeerCount]#", "#[navurlBase]#");
-</script>
-
+#(statistics)#::
+  <script type="text/javascript">
+	statistics("#[offset]#", "#[itemscount]#", "#[itemsperpage]#", "#[totalcount]#", "#[localResourceSize]#", "#[remoteResourceSize]#", "#[remoteIndexCount]#", "#[remotePeerCount]#", "#[navurlBase]#", #[localQuery]#);
+  </script>
+#(/statistics)#
diff --git a/htroot/yacysearchitem.java b/htroot/yacysearchitem.java
index 21038f64c..b43430019 100644
--- a/htroot/yacysearchitem.java
+++ b/htroot/yacysearchitem.java
@@ -102,6 +102,7 @@ public class yacysearchitem {
         prop.put("rssreferences", "0");
         prop.put("dynamic", "0");
         prop.put("localQuery", "0");
+        prop.put("statistics", "0");
 
         // find search event
         final SearchEvent theSearch = SearchEventCache.getEvent(eventID);
@@ -111,16 +112,17 @@ public class yacysearchitem {
         }
 
         // dynamically update count values
-        prop.put("offset", theSearch.query.neededResults() - theSearch.query.itemsPerPage() + 1);
-        prop.put("itemscount", Formatter.number(Math.min((item < 0) ? theSearch.query.neededResults() : item + 1, theSearch.getResultCount())));
-        prop.put("itemsperpage", Formatter.number(theSearch.query.itemsPerPage));
-        prop.put("totalcount", Formatter.number(theSearch.getResultCount(), true));
-        prop.put("localResourceSize", Formatter.number(theSearch.local_rwi_stored.get() + theSearch.local_solr_stored.get(), true));
-        prop.put("remoteResourceSize", Formatter.number(theSearch.remote_rwi_stored.get() + theSearch.remote_solr_stored.get(), true));
-        prop.put("remoteIndexCount", Formatter.number(theSearch.remote_rwi_available.get() + theSearch.remote_solr_available.get(), true));
-        prop.put("remotePeerCount", Formatter.number(theSearch.remote_rwi_peerCount.get() + theSearch.remote_solr_peerCount.get(), true));
-        prop.put("navurlBase", QueryParams.navurlBase(RequestHeader.FileType.HTML, theSearch.query, null, false).toString());
-        prop.put("localQuery", theSearch.query.isLocal() ? "1" : "0");
+        prop.put("statistics", "1");
+        prop.put("statistics_offset", theSearch.query.neededResults() - theSearch.query.itemsPerPage() + 1);
+        prop.put("statistics_itemscount", Formatter.number(Math.min((item < 0) ? theSearch.query.neededResults() : item + 1, theSearch.getResultCount())));
+        prop.put("statistics_itemsperpage", Formatter.number(theSearch.query.itemsPerPage));
+        prop.put("statistics_totalcount", Formatter.number(theSearch.getResultCount(), true));
+        prop.put("statistics_localResourceSize", Formatter.number(theSearch.local_rwi_stored.get() + theSearch.local_solr_stored.get(), true));
+        prop.put("statistics_remoteResourceSize", Formatter.number(theSearch.remote_rwi_stored.get() + theSearch.remote_solr_stored.get(), true));
+        prop.put("statistics_remoteIndexCount", Formatter.number(theSearch.remote_rwi_available.get() + theSearch.remote_solr_available.get(), true));
+        prop.put("statistics_remotePeerCount", Formatter.number(theSearch.remote_rwi_peerCount.get() + theSearch.remote_solr_peerCount.get(), true));
+        prop.put("statistics_navurlBase", QueryParams.navurlBase(RequestHeader.FileType.HTML, theSearch.query, null, false).toString());
+        prop.put("statistics_localQuery", theSearch.query.isLocal() ? "1" : "0");
         final String target_special_pattern = sb.getConfig(SwitchboardConstants.SEARCH_TARGET_SPECIAL_PATTERN, "");
         final boolean noreferrer = sb.getConfigBool(SwitchboardConstants.SEARCH_RESULT_NOREFERRER, SwitchboardConstants.SEARCH_RESULT_NOREFERRER_DEFAULT);