Merge branch 'multiple_assignees_review' of gitlab.com:gitlab-org/gitlab-ee...

Merge branch 'multiple_assignees_review' of gitlab.com:gitlab-org/gitlab-ee into fix_specs_for_multiple_assignees

app/assets/javascripts/blob/template_selectors/template_selector.js

@@ -24,7 +24,7 @@ export default class TemplateSelector {
       search: {
         fields: ['name'],
       },
-      clicked: (item, el, e) => this.fetchFileTemplate(item, el, e),
+      clicked: options => this.fetchFileTemplate(options),
       text: item => item.name,
     });
   }
@@ -51,7 +51,10 @@ export default class TemplateSelector {
     return this.$dropdownContainer.removeClass('hidden');
   }
 
-  fetchFileTemplate(item, el, e) {
+  fetchFileTemplate(options) {
+    const { e } = options;
+    const item = options.selectedObj;
+
     e.preventDefault();
     return this.requestFile(item);
   }

app/assets/javascripts/boards/components/new_list_dropdown.js

@@ -52,7 +52,9 @@
         filterable: true,
         selectable: true,
         multiSelect: true,
-        clicked (label, $el, e) {
+        clicked (options) {
+          const { e } = options;
+          const label = options.selectedObj;
           e.preventDefault();
 
           if (!Store.findList('title', label.title)) {

app/assets/javascripts/members.js

@@ -43,7 +43,9 @@
 
             return $el.text();
           },
-          clicked: (selected, $link) => {
+          clicked: (options) => {
+            const $link = options.$el;
+
             if (!$link.data('revert')) {
               this.formSubmit(null, $link);
             } else {

app/assets/javascripts/namespace_select.js

@@ -58,7 +58,8 @@
       });
     }
 
-    NamespaceSelect.prototype.onSelectItem = function(item, el, e) {
+    NamespaceSelect.prototype.onSelectItem = function(options) {
+      const { e } = options;
       return e.preventDefault();
     };
 

app/assets/javascripts/project.js

@@ -119,7 +119,8 @@ import Cookies from 'js-cookie';
           toggleLabel: function(obj, $el) {
             return $el.text().trim();
           },
-          clicked: function(selected, $el, e) {
+          clicked: function(options) {
+            const { e } = options;
             e.preventDefault();
             if ($('input[name="ref"]').length) {
               var $form = $dropdown.closest('form');

app/assets/javascripts/protected_branches/protected_branch_access_dropdown.js

@@ -53,7 +53,10 @@
             onHide();
           }
         },
-        clicked(item, $el, e) {
+        clicked(opts) {
+          const { $el, e } = opts;
+          const item = opts.selectedObj;
+
           e.preventDefault();
 
           if ($el.is('.is-active')) {

app/assets/javascripts/protected_branches/protected_branch_dropdown.js

@@ -35,7 +35,8 @@ class ProtectedBranchDropdown {
         return _.escape(protectedBranch.id);
       },
       onFilter: this.toggleCreateNewButton.bind(this),
-      clicked: (item, $el, e) => {
+      clicked: (options) => {
+        const { $el, e } = options;
         e.preventDefault();
         this.onSelect();
       }

app/assets/javascripts/sidebar/components/assignees/assignees.js

@@ -16,6 +16,10 @@ export default {
       type: Array,
       required: true,
     },
+    editable: {
+      type: Boolean,
+      required: true,
+    },
   },
   computed: {
     firstUser() {
@@ -142,14 +146,17 @@ export default {
       <div class="value hide-collapsed">
         <template v-if="hasNoUsers">
           <span class="assign-yourself no-value">
-            No assignee -
-            <button
-              type="button"
-              class="btn-link"
-              @click="assignSelf"
-            >
-              assign yourself
-            </button>
+            No assignee
+            <template v-if="editable">
+             -
+              <button
+                type="button"
+                class="btn-link"
+                @click="assignSelf"
+              >
+                assign yourself
+              </button>
+            </template>
           </span>
         </template>
         <template v-else-if="hasOneUser">

app/assets/javascripts/sidebar/components/assignees/sidebar_assignees.js

@@ -76,6 +76,7 @@ export default {
         class="value"
         :root-path="store.rootPath"
         :users="store.assignees"
+        :editable="store.editable"
         @assign-self="assignSelf"
       />
     </div>

app/assets/javascripts/sidebar/stores/sidebar_store.js

@@ -27,7 +27,7 @@ export default class SidebarStore {
     this.timeEstimate = data.time_estimate;
     this.totalTimeSpent = data.total_time_spent;
     this.humanTimeEstimate = data.human_time_estimate;
-    this.humanTimeSpent = data.human_time_spent;
+    this.humanTotalTimeSpent = data.human_total_time_spent;
   }
 
   addAssignee(assignee) {

app/views/projects/boards/components/sidebar/_assignee.html.haml

@@ -5,6 +5,7 @@
       ":editable" => can?(current_user, :admin_issue, @project) }
     %assignees.value{ "root-path" => "#{root_url}",
       ":users" => "issue.assignees",
+      ":editable" => can?(current_user, :admin_issue, @project),
       "@assign-self" => "assignSelf" }
 
   - if can?(current_user, :admin_issue, @project)