Removed label/milestone helper methods

app/helpers/issuables_helper.rb

@@ -23,29 +23,19 @@ module IssuablesHelper
   def user_dropdown_label(user_id, default_label)
     return "Unassigned" if user_id == "0"
 
-    user = @project.team.users.find_by(id: user_id) if @project
-    user = User.find_by_id(user_id) if !@project
-
-    if user
-      user.name
-    else
-      default_label
-    end
+    if @project
+      member = @project.team.find_member(user_id)
+      if member
+        user = member.user
       end
-
-  def labels_dropdown_label(label_name)
-    if !label_name
-      "Label"
     else
-      label_name
-    end
+      user = User.find(user_id)
     end
 
-  def milestone_dropdown_label(milestone_name)
-    if !milestone_name
-      "Milestone"
+    if user
+      user.name
     else
-      milestone_name
+      default_label
     end
   end
 

app/views/shared/issuable/_filter.html.haml

@@ -21,7 +21,7 @@
         .filter-item.inline.milestone-filter
           - if params[:milestone_title]
             = hidden_field_tag(:milestone_title, params[:milestone_title])
-          = dropdown_tag(milestone_dropdown_label(params[:milestone_title]), options: { title: "Filter by milestone", toggle_class: 'js-milestone-select js-filter-submit', filter: true, dropdown_class: "dropdown-menu-selectable",
+          = dropdown_tag(h(params[:milestone_name] || "Milestone"), options: { title: "Filter by milestone", toggle_class: 'js-milestone-select js-filter-submit', filter: true, dropdown_class: "dropdown-menu-selectable",
             placeholder: "Search milestones", footer_content: true, data: { show_no: true, show_any: true, field_name: "milestone_title", selected: params[:milestone_title], project_id: (@project.id if @project), milestones: (namespace_project_milestones_path(@project.namespace, @project, :js) if @project) } }) do
             - if @project
               %ul.dropdown-footer-list
@@ -42,7 +42,7 @@
           .dropdown
             %button.dropdown-menu-toggle.js-label-select.js-filter-submit{type: "button", data: {toggle: "dropdown", field_name: "label_name", show_no: "true", show_any: "true", selected: params[:label_name], project_id: (@project.id if @project), labels: (namespace_project_labels_path(@project.namespace, @project, :js) if @project)}}
               %span.dropdown-toggle-text
-                = labels_dropdown_label(params[:label_name])
+                = h(params[:label_name] || "Label")
               = icon('chevron-down')
             .dropdown-menu.dropdown-select.dropdown-menu-paging.dropdown-menu-labels.dropdown-menu-selectable
               .dropdown-page-one