Merge pull request #14333 from nextcloud/feature/sharee-explicit-lookup

Search sharees on lookup server when explicitly requested by user
nextcloud · Feb 26, 2019 · 73b8b56 · 73b8b56
2 parents a3d936f + e930a0c
commit 73b8b56
Show file tree

Hide file tree

Showing 11 changed files with 174 additions and 71 deletions.
diff --git a/apps/files_sharing/js/additionalScripts.js b/apps/files_sharing/js/additionalScripts.js
diff --git a/apps/files_sharing/js/additionalScripts.js.map b/apps/files_sharing/js/additionalScripts.js.map
diff --git a/apps/files_sharing/lib/Controller/ShareesAPIController.php b/apps/files_sharing/lib/Controller/ShareesAPIController.php
@@ -92,6 +92,7 @@ class ShareesAPIController extends OCSController {
 		'lookup' => [],
 		'circles' => [],
 		'rooms' => [],
+		'lookupEnabled' => false,
 	];
 
 	protected $reachedEndFor = [];
@@ -212,6 +213,7 @@ public function search(string $search = '', string $itemType = null, int $page =
 			$result['exact'] = array_merge($this->result['exact'], $result['exact']);
 		}
 		$this->result = array_merge($this->result, $result);
+		$this->result['lookupEnabled'] = $this->config->getAppValue('files_sharing', 'lookupServerEnabled', 'yes') === 'yes';
 		$response = new DataResponse($this->result);
 
 		if ($hasMoreResults) {

diff --git a/apps/files_sharing/src/style/sharetabview.scss b/apps/files_sharing/src/style/sharetabview.scss
@@ -38,6 +38,11 @@
 		opacity: .7;
 		margin-right: 7px;
 	}
+	.icon.search-globally {
+		width: 32px;
+		height: 32px;
+		margin-right: 0;
+	}
 }
 
 .shareTabView {

diff --git a/apps/files_sharing/tests/Controller/ShareesAPIControllerTest.php b/apps/files_sharing/tests/Controller/ShareesAPIControllerTest.php
@@ -236,12 +236,13 @@ public function testSearch($getData, $apiSetting, $enumSetting, $remoteSharingEn
 
 		/** @var IConfig|\PHPUnit_Framework_MockObject_MockObject $config */
 		$config = $this->createMock(IConfig::class);
-		$config->expects($this->exactly(2))
+		$config->expects($this->exactly(3))
 			->method('getAppValue')
-			->with('core', $this->anything(), $this->anything())
+			->with($this->anything(), $this->anything(), $this->anything())
 			->willReturnMap([
 				['core', 'shareapi_only_share_with_group_members', 'no', $apiSetting],
 				['core', 'shareapi_allow_share_dialog_user_enumeration', 'yes', $enumSetting],
+				['files_sharing', 'lookupServerEnabled', 'yes', 'yes'],
 			]);
 
 		$this->shareManager->expects($itemType === 'file' || $itemType === 'folder' ? $this->once() : $this->never())

diff --git a/core/js/dist/share_backend.js b/core/js/dist/share_backend.js
diff --git a/core/js/dist/share_backend.js.map b/core/js/dist/share_backend.js.map
diff --git a/core/js/sharedialogview.js b/core/js/sharedialogview.js
@@ -32,6 +32,10 @@
 		/** @type {boolean} **/
 		_showLink: true,
 
+		_lookup: false,
+
+		_lookupAllowed: false,
+
 		/** @type {string} **/
 		tagName: 'div',
 
@@ -125,24 +129,28 @@
 
 		/* trigger search after the field was re-selected */
 		onShareWithFieldFocus: function() {
-			this.$el.find('.shareWithField').autocomplete("search");
+			var $shareWithField = this.$el.find('.shareWithField');
+			$shareWithField.autocomplete("search", $shareWithField.val());
 		},
 
-		_getSuggestions: function(searchTerm, perPage, model) {
+		_getSuggestions: function(searchTerm, perPage, model, lookup) {
 			if (this._lastSuggestions &&
 				this._lastSuggestions.searchTerm === searchTerm &&
+				this._lastSuggestions.lookup === lookup &&
 				this._lastSuggestions.perPage === perPage &&
 				this._lastSuggestions.model === model) {
 				return this._lastSuggestions.promise;
 			}
 
 			var deferred = $.Deferred();
+			var view = this;
 
 			$.get(
 				OC.linkToOCS('apps/files_sharing/api/v1') + 'sharees',
 				{
 					format: 'json',
 					search: searchTerm,
+					lookup: lookup,
 					perPage: perPage,
 					itemType: model.get('itemType')
 				},
@@ -300,6 +308,7 @@
 						var remotes = result.ocs.data.remotes;
 						var remoteGroups = result.ocs.data.remote_groups;
 						var lookup = result.ocs.data.lookup;
+						var lookupEnabled = result.ocs.data.lookupEnabled;
 						var emails = [];
 						if (typeof(result.ocs.data.emails) !== 'undefined') {
 							emails = result.ocs.data.emails;
@@ -365,8 +374,17 @@
 										lookup.length
 									)
 							);
+						if (!view._lookup && lookupEnabled) {
+							result.push(
+								{
+									label: t('core', 'Search globally'),
+									value: {},
+									lookup: true
+								}
+							)
+						}
 
-						deferred.resolve(result, exactMatches, moreResultsAvailable);
+						deferred.resolve(result, exactMatches, moreResultsAvailable, lookupEnabled);
 					} else {
 						deferred.reject(result.ocs.meta.message);
 					}
@@ -377,6 +395,7 @@
 
 			this._lastSuggestions = {
 				searchTerm: searchTerm,
+				lookup: lookup,
 				perPage: perPage,
 				model: model,
 				promise: deferred.promise()
@@ -641,14 +660,8 @@
 			var $shareWithField = $('.shareWithField');
 			this._getRecommendations(
 				view.model
-			).done(function(suggestions, exactMatches) {
-				view._pendingOperationsCount--;
-				if (view._pendingOperationsCount === 0) {
-					$loading.addClass('hidden');
-					$loading.removeClass('inlineblock');
-					$confirm.removeClass('hidden');
-				}
-
+			).done(function(suggestions) {
+				console.info('recommendations', suggestions);
 				if (suggestions.length > 0) {
 					$shareWithField
 						.autocomplete("option", "autoFocus", true);
@@ -659,13 +672,6 @@
 					response();
 				}
 			}).fail(function(message) {
-				view._pendingOperationsCount--;
-				if (view._pendingOperationsCount === 0) {
-					$loading.addClass('hidden');
-					$loading.removeClass('inlineblock');
-					$confirm.removeClass('hidden');
-				}
-
 				console.error('could not load recommendations', message)
 			});
 		},
@@ -674,6 +680,7 @@
 			// If nothing is entered we show recommendations instead of search
 			// results
 			if (search.term.length === 0) {
+				console.info(search.term, 'empty search term -> using recommendations');
 				this.recommendationHandler(response);
 				return;
 			}
@@ -716,7 +723,8 @@
 			this._getSuggestions(
 				search.term.trim(),
 				perPage,
-				view.model
+				view.model,
+				view._lookup
 			).done(function(suggestions, exactMatches, moreResultsAvailable) {
 				view._pendingOperationsCount--;
 				if (view._pendingOperationsCount === 0) {
@@ -747,7 +755,7 @@
 						.attr('data-original-title', title)
 						.tooltip('hide')
 						.tooltip({
-							placement: 'bottom',
+							placement: 'top',
 							trigger: 'manual'
 						})
 						.tooltip('fixTitle')
@@ -818,6 +826,10 @@
 				insert.addClass('merged');
 				text = item.value.shareWith;
 				description = type;
+			} else if (item.lookup) {
+				text = item.label;
+				icon = false;
+				insert.append('<span class="icon icon-search search-globally"></span>');
 			} else {
 				var avatar = $("<div class='avatardiv'></div>").appendTo(insert);
 				if (item.value.shareType === OC.Share.SHARE_TYPE_USER || item.value.shareType === OC.Share.SHARE_TYPE_CIRCLE) {
@@ -843,7 +855,9 @@
 				)
 				.appendTo(insert);
 			insert.attr('title', item.value.shareWith);
-			insert.append('<span class="icon '+icon+'" title="' + text + '"></span>');
+			if (icon) {
+				insert.append('<span class="icon ' + icon + '" title="' + text + '"></span>');
+			}
 			insert = $("<a>")
 				.append(insert);
 			return $("<li>")
@@ -869,6 +883,18 @@
 				return false;
 			}
 
+			if (s.item.lookup) {
+				// Retrigger search but with global lookup this time
+				this._lookup = true;
+				var $shareWithField = this.$el.find('.shareWithField');
+				var val = $shareWithField.val();
+				setTimeout(function() {
+					console.debug('searching again, but globally. search term: ' + val);
+					$shareWithField.autocomplete("search", val);
+				}, 0);
+				return false;
+			}
+
 			e.preventDefault();
 			// Ensure that the keydown handler for the input field is not
 			// called; otherwise it would try to add the recipient again, which
@@ -947,12 +973,11 @@
 			};
 
 			var perPage = parseInt(oc_config['sharing.maxAutocompleteResults'], 10) || 200;
-			var onlyExactMatches = true;
 			this._getSuggestions(
 				$shareWithField.val(),
 				perPage,
 				this.model,
-				onlyExactMatches
+				this._lookup
 			).done(function(suggestions, exactMatches) {
 				if (suggestions.length === 0) {
 					restoreUI();