From 77f30af0179164fe3e893df616eb60beb8a1d9fe Mon Sep 17 00:00:00 2001
From: Sean McGivern <sean@gitlab.com>
Date: Mon, 6 Jun 2016 15:40:35 +0100
Subject: [PATCH] Tidy up Ruby style in templates

---
 app/views/projects/labels/index.html.haml | 9 ++++-----
 app/views/shared/_label_row.html.haml     | 2 +-
 2 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/app/views/projects/labels/index.html.haml b/app/views/projects/labels/index.html.haml
index 0450f51204..5deb84ad41 100644
--- a/app/views/projects/labels/index.html.haml
+++ b/app/views/projects/labels/index.html.haml
@@ -13,9 +13,8 @@
 .labels
   - if can?(current_user, :admin_label, @project)
     -# Only show it in the first page
-    - if (params[:page].present? and params[:page] != '1') or @project.labels.blank? or (params[:page] == nil and @project.labels.blank?)
-      - hide_class = 'hide'
-    .prioritized-labels{ class: hide_class }
+    - hide = @project.labels.empty? || (params[:page].present? && params[:page] != '1')
+    .prioritized-labels{ class: ('hide' if hide) }
       %h5 Prioritized Labels
       %ul.content-list.manage-labels-list.js-prioritized-labels{ "data-url" => set_priorities_namespace_project_labels_path(@project.namespace, @project) }
         - if @prioritized_labels.present?
@@ -24,14 +23,14 @@
           %p.empty-message No prioritized labels yet
   .other-labels
     - if can?(current_user, :admin_label, @project)
-      %h5{ class: hide_class } Other Labels
+      %h5{ class: ('hide' if hide } Other Labels
     - if @labels.present?
       %ul.content-list.manage-labels-list.js-other-labels
         = render @labels
       = paginate @labels, theme: 'gitlab'
     - else
       .nothing-here-block
-        - if can? current_user, :admin_label, @project
+        - if can?(current_user, :admin_label, @project)
           Create a label or #{link_to 'generate a default set of labels', generate_namespace_project_labels_path(@project.namespace, @project), method: :post}.
         - else
           No labels created
diff --git a/app/views/shared/_label_row.html.haml b/app/views/shared/_label_row.html.haml
index c53f32d97e..d315a3fe93 100644
--- a/app/views/shared/_label_row.html.haml
+++ b/app/views/shared/_label_row.html.haml
@@ -1,5 +1,5 @@
 %span.label-row
-  - if can? current_user, :admin_label, @project
+  - if can?(current_user, :admin_label, @project)
     .js-toggle-priority.toggle-priority{ data: { url: remove_priority_namespace_project_label_path(@project.namespace, @project, label),
       dom_id: dom_id(label) } }
       %button.add-priority.btn.has-tooltip{ title: 'Prioritize', :'data-placement' => 'top' }