From a1d7d60685b45a85611d3ea1c78b61215c900f13 Mon Sep 17 00:00:00 2001
From: "John Molakvoæ (skjnldsv)" <skjnldsv@protonmail.com>
Date: Wed, 25 Jul 2018 12:20:46 +0200
Subject: Fix search on public pages
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
---
 core/search/js/search.js         |  7 +++++--
 core/search/js/searchprovider.js | 21 +++++++++++----------
 2 files changed, 16 insertions(+), 12 deletions(-)

(limited to 'core')

diff --git a/core/search/js/search.js b/core/search/js/search.js
index 461c963c471..167d7c7a53d 100644
--- a/core/search/js/search.js
+++ b/core/search/js/search.js
@@ -52,10 +52,13 @@
 			var self = this;
 
 			if (typeof searchCallback !== 'function') {
-				throw 'searchCallback must be a function';
+				throw new Error('searchCallback must be a function');
 			}
 			if (typeof resetCallback !== 'function') {
-				throw 'resetCallback must be a function';
+				throw new Error('resetCallback must be a function');
+			}
+			if (!document.getElementById('searchbox')) {
+				throw new Error('searchBox not available');
 			}
 
 			this.searchCallback = searchCallback;
diff --git a/core/search/js/searchprovider.js b/core/search/js/searchprovider.js
index 4a484050385..f5a499fab60 100644
--- a/core/search/js/searchprovider.js
+++ b/core/search/js/searchprovider.js
@@ -413,26 +413,27 @@
 
 $(document).ready(function() {
 	var $searchResults = $('#searchresults');
-	if ($searchResults.length > 0) {
+	var $searchBox = $('#searchbox');
+	if ($searchResults.length > 0 && $searchBox.length > 0) {
 		$searchResults.addClass('hidden');
-		$('#app-content')
-			.find('.viewcontainer')
-			.css('min-height', 'initial');
 		$searchResults.load(
 			OC.webroot + '/core/search/templates/part.results.html',
 			function() {
 				OC.Search = new OCA.Search.Core(
-					$('#searchbox'),
-					$('#searchresults')
+					$searchBox,
+					$searchResults
 				);
 			}
 		);
 	} else {
+		// check again later
 		_.defer(function() {
-			OC.Search = new OCA.Search.Core(
-				$('#searchbox'),
-				$('#searchresults')
-			);
+			if ($searchResults.length > 0 && $searchBox.length > 0) {
+				OC.Search = new OCA.Search.Core(
+					$searchBox,
+					$searchResults
+				);
+			}
 		});
 	}
 });
-- 
cgit v1.2.3