summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSean McGivern <sean@gitlab.com>2019-06-13 12:21:51 +0000
committerSean McGivern <sean@gitlab.com>2019-06-13 12:21:51 +0000
commit0a70ba177e4589733659dd5af85402d0a3081026 (patch)
treedd6398b122b1f180386beb13402f012c4c2f17ab
parent0c4059efc146c869d3ffd1bc075c67c8b4ae921d (diff)
parent0a2c334cd558dc4f6e822bac1077f480e9fffc7c (diff)
downloadgitlab-ce-0a70ba177e4589733659dd5af85402d0a3081026.tar.gz
Merge branch 'ce-12091-remove-elasticsearch-lite-project' into 'master'
Backport changes from 'Partially revert d9cb907c: "Avoid loading objects from DB in ES results"' See merge request gitlab-org/gitlab-ce!29597
-rw-r--r--app/controllers/search_controller.rb1
-rw-r--r--app/models/project_feature.rb2
-rw-r--r--app/services/search_service.rb4
-rw-r--r--app/views/search/_results.html.haml2
-rw-r--r--lib/gitlab/search_results.rb4
5 files changed, 1 insertions, 12 deletions
diff --git a/app/controllers/search_controller.rb b/app/controllers/search_controller.rb
index cb25548c83f..a80ab3bcd28 100644
--- a/app/controllers/search_controller.rb
+++ b/app/controllers/search_controller.rb
@@ -25,7 +25,6 @@ class SearchController < ApplicationController
@show_snippets = search_service.show_snippets?
@search_results = search_service.search_results
@search_objects = search_service.search_objects
- @display_options = search_service.display_options
render_commits if @scope == 'commits'
eager_load_user_status if @scope == 'users'
diff --git a/app/models/project_feature.rb b/app/models/project_feature.rb
index 6bcb051bff6..0542581c6e0 100644
--- a/app/models/project_feature.rb
+++ b/app/models/project_feature.rb
@@ -72,8 +72,6 @@ class ProjectFeature < ApplicationRecord
default_value_for :wiki_access_level, value: ENABLED, allows_nil: false
default_value_for :repository_access_level, value: ENABLED, allows_nil: false
- scope :for_project_id, -> (project) { where(project: project) }
-
def feature_available?(feature, user)
# This feature might not be behind a feature flag at all, so default to true
return false unless ::Feature.enabled?(feature, user, default_enabled: true)
diff --git a/app/services/search_service.rb b/app/services/search_service.rb
index 302510341ac..e0cbfac2420 100644
--- a/app/services/search_service.rb
+++ b/app/services/search_service.rb
@@ -52,10 +52,6 @@ class SearchService
@search_objects ||= search_results.objects(scope, params[:page])
end
- def display_options
- @display_options ||= search_results.display_options(scope)
- end
-
private
def search_service
diff --git a/app/views/search/_results.html.haml b/app/views/search/_results.html.haml
index 12eb8d7fa81..cb8a8a24be8 100644
--- a/app/views/search/_results.html.haml
+++ b/app/views/search/_results.html.haml
@@ -21,7 +21,7 @@
.search-results
- if @scope == 'projects'
.term
- = render 'shared/projects/list', { projects: @search_objects, pipeline_status: false }.merge(@display_options)
+ = render 'shared/projects/list', projects: @search_objects, pipeline_status: false
- else
- locals = { projects: blob_projects(@search_objects) } if %w[blobs wiki_blobs].include?(@scope)
= render partial: "search/results/#{@scope.singularize}", collection: @search_objects, locals: locals
diff --git a/lib/gitlab/search_results.rb b/lib/gitlab/search_results.rb
index 3daa03d01d6..7c1e6b1baff 100644
--- a/lib/gitlab/search_results.rb
+++ b/lib/gitlab/search_results.rb
@@ -85,10 +85,6 @@ module Gitlab
UsersFinder.new(current_user, search: query).execute
end
- def display_options(_scope)
- {}
- end
-
private
def projects